Use guard clause and avoid assignment in conditional

3 files changed, 3 insertions, 3 deletions

diff --git a/spec/acceptance/hosts.rb b/spec/acceptance/hosts.rb
index dabec5a..297d923 100644
--- a/spec/acceptance/hosts.rb
+++ b/spec/acceptance/hosts.rb
@@ -11,7 +11,7 @@ test_name 'Augeas hosts file' do
     'windows',
     'cisco_ios', # PUP-7380
   ]
-  confine :to, {}, hosts.reject { |host| host[:roles].include?('master') }
+  confine(:to, {}, hosts.reject { |host| host[:roles].include?('master') })
 
   step 'Backup the hosts file' do
     on hosts, 'cp /etc/hosts /tmp/hosts.bak'
diff --git a/spec/acceptance/puppet.rb b/spec/acceptance/puppet.rb
index 048a3db..2c17238 100644
--- a/spec/acceptance/puppet.rb
+++ b/spec/acceptance/puppet.rb
@@ -7,7 +7,7 @@ test_name 'Augeas puppet configuration' do
   skip_test 'requires augeas which is included in AIO' if @options[:type] != 'aio'
 
   confine :except, platform: 'windows'
-  confine :to, {}, hosts.reject { |host| host[:roles].include?('master') }
+  confine(:to, {}, hosts.reject { |host| host[:roles].include?('master') })
 
   teardown do
     agents.each do |agent|
diff --git a/spec/acceptance/services.rb b/spec/acceptance/services.rb
index 2e48af0..523b953 100644
--- a/spec/acceptance/services.rb
+++ b/spec/acceptance/services.rb
@@ -9,7 +9,7 @@ test_name 'Augeas services file' do
 
   confine :except, platform: 'windows'
   confine :except, platform: 'osx'
-  confine :to, {}, hosts.reject { |host| host[:roles].include?('master') }
+  confine(:to, {}, hosts.reject { |host| host[:roles].include?('master') })
 
   step 'Backup the services file' do
     on hosts, 'cp /etc/services /tmp/services.bak'