| Total Complexity | 2 |
| Total Lines | 26 |
| Duplicated Lines | 100 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 0 |
Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
| 1 | # frozen_string_literal: true |
||
| 11 | class Remove < Action |
||
| 12 | attr_accessor :account |
||
| 13 | attr_accessor :key |
||
| 14 | |||
| 15 | # @param [AMA::Chef::User::Model::Account] account |
||
| 16 | # @param [AMA::Chef::User::Model::PrivateKey] key |
||
| 17 | def initialize(account, key) |
||
| 18 | @account = account |
||
| 19 | @key = key |
||
| 20 | end |
||
| 21 | |||
| 22 | def apply(resource_factory) |
||
| 23 | account = @account |
||
| 24 | key = @key |
||
| 25 | name = "#{account.id}:#{key.owner}:#{key.id}" |
||
| 26 | content = (key.content || '').to_s |
||
| 27 | resource_factory.ssh_private_key name do |
||
| 28 | id key.id.to_s |
||
| 29 | user account.id.to_s |
||
| 30 | # TODO: remove when https://github.com/ama-team/cookbook-ssh-private-keys/issues/2 |
||
| 31 | # is resolved |
||
| 32 | content content |
||
| 33 | action :remove |
||
| 34 | end |
||
| 35 | end |
||
| 36 | end |
||
| 37 | end |
||
| 43 |