Fix spec tests and rubocop errors

7 files changed, 11 insertions, 13 deletions

diff --git a/lib/puppet/type/mailalias.rb b/lib/puppet/type/mailalias.rb
index fde34be..59c88ef 100644
--- a/lib/puppet/type/mailalias.rb
+++ b/lib/puppet/type/mailalias.rb
@@ -2,13 +2,13 @@
 module Puppet
   Type.newtype(:mailalias) do
     desc <<-DESC
-Creates an email alias in the local alias database.
+  Creates an email alias in the local alias database.
 
-@example using mailalias to redirect mail for the ftp account to root's mailbox
-mailalias { 'ftp':
-  ensure => present,
-  recipient => 'root',
-}
+  @example using mailalias to redirect mail for the ftp account to root's mailbox
+  mailalias { 'ftp':
+    ensure => present,
+    recipient => 'root',
+  }
     DESC
 
     ensurable
diff --git a/spec/acceptance/tests/destroy_spec.rb b/spec/acceptance/tests/destroy_spec.rb
index 499b613..4a740db 100644
--- a/spec/acceptance/tests/destroy_spec.rb
+++ b/spec/acceptance/tests/destroy_spec.rb
@@ -12,7 +12,7 @@ RSpec.context 'should delete an email alias' do
       on(agent, "echo '#{name}: foo,bar,baz' >> /etc/aliases")
 
       # (setup) verify the alias exists
-      on(agent, 'cat /etc/aliases')  do |res|
+      on(agent, 'cat /etc/aliases') do |res|
         assert_match(%r{#{name}:.*foo,bar,baz}, res.stdout, 'mailalias not in aliases file')
       end
     end
diff --git a/spec/acceptance/tests/modify_spec.rb b/spec/acceptance/tests/modify_spec.rb
index b3937dc..f4ea7bf 100644
--- a/spec/acceptance/tests/modify_spec.rb
+++ b/spec/acceptance/tests/modify_spec.rb
@@ -13,7 +13,7 @@ RSpec.context 'should modify an email alias' do
       on(agent, "echo '#{name}: foo,bar,baz' >> /etc/aliases")
 
       # (setup) verify the alias exists
-      on(agent, 'cat /etc/aliases')  do |res|
+      on(agent, 'cat /etc/aliases') do |res|
         assert_match(%r{#{name}:.*foo,bar,baz}, res.stdout, 'mailalias not in aliases file')
       end
     end
@@ -34,7 +34,7 @@ RSpec.context 'should modify an email alias' do
     end
 
     it 'verifies the updated alias is present' do
-      on(agent, 'cat /etc/aliases')  do |res|
+      on(agent, 'cat /etc/aliases') do |res|
         assert_match(%r{#{name}:.*foo,bar,baz,blarvitz}, res.stdout, 'updated mailalias not in aliases file')
       end
     end
diff --git a/spec/acceptance/tests/query_spec.rb b/spec/acceptance/tests/query_spec.rb
index c888a0b..1a7ab42 100644
--- a/spec/acceptance/tests/query_spec.rb
+++ b/spec/acceptance/tests/query_spec.rb
@@ -12,7 +12,7 @@ RSpec.context 'should be able to find an exisitng email alias' do
       on(agent, "echo '#{name}: foo,bar,baz' >> /etc/aliases")
 
       # (setup) verify the alias exists
-      on(agent, 'cat /etc/aliases')  do |res|
+      on(agent, 'cat /etc/aliases') do |res|
         assert_match(%r{#{name}:.*foo,bar,baz}, res.stdout, 'mailalias not in aliases file')
       end
     end
diff --git a/spec/integration/provider/mailalias/aliases_spec.rb b/spec/integration/provider/mailalias/aliases_spec.rb
index 29098f8..ae9dc10 100755..100644
--- a/spec/integration/provider/mailalias/aliases_spec.rb
+++ b/spec/integration/provider/mailalias/aliases_spec.rb
@@ -1,4 +1,3 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 require 'shared_behaviours/all_parsedfile_providers'
 
diff --git a/spec/spec_helper_acceptance.rb b/spec/spec_helper_acceptance.rb
index 218a34d..0adde24 100644
--- a/spec/spec_helper_acceptance.rb
+++ b/spec/spec_helper_acceptance.rb
@@ -10,7 +10,7 @@ def beaker_opts
 end
 
 def non_windows_agents
-  agents.select { |agent| !agent['platform'].include?('windows') }
+  agents.reject { |agent| agent['platform'].include?('windows') }
 end
 
 RSpec.configure do |c|
diff --git a/spec/unit/type/mailalias_spec.rb b/spec/unit/type/mailalias_spec.rb
index 2a079ba..1fb195e 100755..100644
--- a/spec/unit/type/mailalias_spec.rb
+++ b/spec/unit/type/mailalias_spec.rb
@@ -1,4 +1,3 @@
-#! /usr/bin/env ruby
 require 'spec_helper'
 
 describe Puppet::Type.type(:mailalias) do