diff options
author | Josh Cooper <josh@puppet.com> | 2018-06-27 22:57:38 -0700 |
---|---|---|
committer | Josh Cooper <josh@puppet.com> | 2018-06-27 23:09:19 -0700 |
commit | df20c08345fba0b43833eef47c4da2fdd78523e9 (patch) | |
tree | eccd5c48492f4240a8e6a269567c2439070d0719 /spec/acceptance/services.rb | |
parent | 1294290d6045fdd383a17cb45039b671ab954f05 (diff) | |
download | puppet-augeas_core-df20c08345fba0b43833eef47c4da2fdd78523e9.tar.gz puppet-augeas_core-df20c08345fba0b43833eef47c4da2fdd78523e9.tar.bz2 |
Automatic update of rubocop violations
Add exclusions for:
Layout/IndentHeredoc,
Metrics/BlockNesting:
Metrics/LineLength:
Style/AccessorMethodName:
Style/DoubleNegation:
Style/MultipleComparison:
Style/NumericPredicate:
Style/PredicateName:
Style/SignalException:
Style/VariableName:
SignalException is disabled because Puppet::Type#fail overrides Kernel#fail,
and we really do want fail and not raise.
Diffstat (limited to 'spec/acceptance/services.rb')
-rw-r--r-- | spec/acceptance/services.rb | 33 |
1 files changed, 16 insertions, 17 deletions
diff --git a/spec/acceptance/services.rb b/spec/acceptance/services.rb index ec87316..2e48af0 100644 --- a/spec/acceptance/services.rb +++ b/spec/acceptance/services.rb @@ -1,23 +1,22 @@ -test_name "Augeas services file" do - +test_name 'Augeas services file' do tag 'risk:medium', 'audit:medium', 'audit:acceptance', - 'audit:refactor' # move to types test dir - # use single manifest/apply + 'audit:refactor' # move to types test dir + # use single manifest/apply skip_test 'requires augeas which is included in AIO' if @options[:type] != 'aio' - confine :except, :platform => 'windows' - confine :except, :platform => 'osx' - confine :to, {}, hosts.select { |host| ! host[:roles].include?('master') } + confine :except, platform: 'windows' + confine :except, platform: 'osx' + confine :to, {}, hosts.reject { |host| host[:roles].include?('master') } - step "Backup the services file" do - on hosts, "cp /etc/services /tmp/services.bak" + step 'Backup the services file' do + on hosts, 'cp /etc/services /tmp/services.bak' end begin - step "Add an entry to the services file" do + step 'Add an entry to the services file' do manifest = <<EOF augeas { 'add_services_entry': context => '/files/etc/services', @@ -32,11 +31,11 @@ augeas { 'add_services_entry': } EOF - on hosts, puppet_apply('--verbose'), :stdin => manifest + on hosts, puppet_apply('--verbose'), stdin: manifest on hosts, "fgrep 'Doom 666/udp' /etc/services" end - step "Change the protocol to udp" do + step 'Change the protocol to udp' do manifest = <<EOF augeas { 'change_service_protocol': context => '/files/etc/services', @@ -48,11 +47,11 @@ augeas { 'change_service_protocol': } EOF - on hosts, puppet_apply('--verbose'), :stdin => manifest + on hosts, puppet_apply('--verbose'), stdin: manifest on hosts, "fgrep 'Doom 666/tcp' /etc/services" end - step "Remove the services entry" do + step 'Remove the services entry' do manifest = <<EOF augeas { 'del_service_entry': context => '/files/etc/services', @@ -64,10 +63,10 @@ augeas { 'del_service_entry': } EOF - on hosts, puppet_apply('--verbose'), :stdin => manifest - on hosts, "fgrep 'Doom 666/tcp' /etc/services", :acceptable_exit_codes => [1] + on hosts, puppet_apply('--verbose'), stdin: manifest + on hosts, "fgrep 'Doom 666/tcp' /etc/services", acceptable_exit_codes: [1] end ensure - on hosts, "mv /tmp/services.bak /etc/services" + on hosts, 'mv /tmp/services.bak /etc/services' end end |