Skip to content

Commit 35f652a

Browse files
committed
Fix a RuboCop offense
1 parent 95b2453 commit 35f652a

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

test/integration/default/serverspec/files_spec.rb

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
platform = os[:family].downcase
2323

24-
def has_runuser?
24+
def runuser?
2525
system('which runuser')
2626
end
2727

@@ -40,8 +40,8 @@ def has_runuser?
4040
it { should be_mode 644 }
4141
it { should be_owned_by 'root' }
4242
it { should be_grouped_into 'root' }
43-
it { should be_readable.by_user('spamd') } if has_runuser?
44-
it { should_not be_writable.by_user('spamd') } if has_runuser?
43+
it { should be_readable.by_user('spamd') } if runuser?
44+
it { should_not be_writable.by_user('spamd') } if runuser?
4545
end
4646
end
4747

@@ -50,15 +50,15 @@ def has_runuser?
5050
it { should be_mode 755 }
5151
it { should be_owned_by 'spamd' }
5252
it { should be_grouped_into 'spamd' }
53-
it { should be_readable.by_user('spamd') } if has_runuser?
54-
it { should be_writable.by_user('spamd') } if has_runuser?
53+
it { should be_readable.by_user('spamd') } if runuser?
54+
it { should be_writable.by_user('spamd') } if runuser?
5555
end
5656

5757
describe file('/etc/mail/spamassassin/local.cf') do
5858
it { should be_file }
5959
it { should be_mode 644 }
6060
it { should be_owned_by 'root' }
6161
it { should be_grouped_into 'root' }
62-
it { should be_readable.by_user('spamd') } if has_runuser?
63-
it { should_not be_writable.by_user('spamd') } if has_runuser?
62+
it { should be_readable.by_user('spamd') } if runuser?
63+
it { should_not be_writable.by_user('spamd') } if runuser?
6464
end

0 commit comments

Comments
 (0)