8000 Fix Naming/AccessorMethodName in the tests. · rubyzip/rubyzip@ce17c57 · GitHub
[go: up one dir, main page]

Skip to content

Commit ce17c57

Browse files
committed
Fix Naming/AccessorMethodName in the tests.
This was kind of a misfire of this cop, but no bother to change.
1 parent c30d9df commit ce17c57

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

.rubocop_todo.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ Naming/AccessorMethodName:
3636
- 'lib/zip/filesystem.rb'
3737
- 'lib/zip/input_stream.rb'
3838
- 'lib/zip/streamable_stream.rb'
39-
- 'test/file_permissions_test.rb'
4039

4140
# Offense count: 7
4241
# Configuration parameters: EnforcedStyle.

test/file_permissions_test.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,23 +15,23 @@ def test_current_umask
1515
end
1616

1717
def test_umask_000
18-
set_umask(0o000) do
18+
apply_umask(0o000) do
1919
create_files
2020
end
2121

2222
assert_matching_permissions FILENAME, ZIPNAME
2323
end
2424

2525
def test_umask_066
26-
set_umask(0o066) do
26+
apply_umask(0o066) do
2727
create_files
2828
end
2929

3030
assert_matching_permissions FILENAME, ZIPNAME
3131
end
3232

3333
def test_umask_027
34-
set_umask(0o027) do
34+
apply_umask(0o027) do
3535
create_files
3636
end
3737

@@ -56,7 +56,7 @@ def create_files
5656
end
5757

5858
# If anything goes wrong, make sure the umask is restored.
59-
def set_umask(umask)
59+
def apply_umask(umask)
6060
saved_umask = ::File.umask(umask)
6161
yield
6262
ensure

0 commit comments

Comments
 (0)
0