Fix Naming/AccessorMethodName in the tests.

This was kind of a misfire of this cop, but no bother to change.
rubyzip · Feb 19, 2020 · 4916556 · 4916556
1 parent d63e2fd
commit 4916556
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -36,7 +36,6 @@ Naming/AccessorMethodName:
     - 'lib/zip/filesystem.rb'
     - 'lib/zip/input_stream.rb'
     - 'lib/zip/streamable_stream.rb'
-    - 'test/file_permissions_test.rb'
 
 # Offense count: 7
 # Configuration parameters: EnforcedStyle.

diff --git a/test/file_permissions_test.rb b/test/file_permissions_test.rb
@@ -15,23 +15,23 @@ def test_current_umask
   end
 
   def test_umask_000
-    set_umask(0o000) do
+    apply_umask(0o000) do
       create_files
     end
 
     assert_matching_permissions FILENAME, ZIPNAME
   end
 
   def test_umask_066
-    set_umask(0o066) do
+    apply_umask(0o066) do
       create_files
     end
 
     assert_matching_permissions FILENAME, ZIPNAME
   end
 
   def test_umask_027
-    set_umask(0o027) do
+    apply_umask(0o027) do
       create_files
     end
 
@@ -56,7 +56,7 @@ def create_files
   end
 
   # If anything goes wrong, make sure the umask is restored.
-  def set_umask(umask)
+  def apply_umask(umask)
     saved_umask = ::File.umask(umask)
     yield
   ensure