Run Rubocop on Truffle specs and update to RuboCop 0.66

PullRequest: truffleruby/710

Author: eregon

.rubocop.yml

@@ -1,5 +1,5 @@
 AllCops:
-  TargetRubyVersion: 2.3
+  TargetRubyVersion: 2.6
   DisabledByDefault: true
   DisplayCopNames: true
   ExtraDetails: true
@@ -9,9 +9,9 @@ AllCops:
     - 'lib/truffle/socket/mri.rb' # taken from MRI
     - 'lib/truffle/ffi/library.rb' # taken from FFI gem
 
-# Type 'Style' (166):
+# Type 'Layout' (166):
 # Supports --auto-correct
-Style/TrailingBlankLines:
+Layout/TrailingBlankLines:
   Description: Checks trailing blank lines and final newline.
   StyleGuide: "#newline-eof"
   Enabled: true
@@ -21,7 +21,7 @@ Style/TrailingBlankLines:
   - final_blank_line
 
 # Supports --auto-correct
-Style/TrailingWhitespace:
+Layout/TrailingWhitespace:
   Description: Avoid trailing whitespace.
   StyleGuide: "#no-trailing-whitespace"
   Enabled: true
@@ -44,7 +44,7 @@ Lint/AmbiguousRegexpLiteral:
   Enabled: true
 
 # Supports --auto-correct
-Lint/BlockAlignment:
+Layout/BlockAlignment:
   Description: Align block ends correctly.
   Enabled: true
   EnforcedStyleAlignWith: either
@@ -58,7 +58,7 @@ Lint/CircularArgumentReference:
     should not refer back to the name of the argument.
   Enabled: true
 
-Lint/ConditionPosition:
+Layout/ConditionPosition:
   Description: Checks for condition placed in a confusing position relative to the keyword.
   StyleGuide: "#same-line-condition"
   Enabled: true
@@ -69,7 +69,7 @@ Lint/Debugger:
   Enabled: true
 
 # Supports --auto-correct
-Lint/DefEndAlignment:
+Layout/DefEndAlignment:
   Description: Align ends corresponding to defs correctly.
   Enabled: true
   EnforcedStyleAlignWith: start_of_line
@@ -122,7 +122,7 @@ Lint/EmptyWhen:
   Enabled: false
 
 # Supports --auto-correct
-Lint/EndAlignment:
+Layout/EndAlignment:
   Description: Align ends correctly.
   Enabled: true
   EnforcedStyleAlignWith: keyword
@@ -175,15 +175,11 @@ Lint/InheritException:
   Exclude:
   - src/main/ruby/core/exception.rb
 
-Lint/InvalidCharacterLiteral:
-  Description: Checks for invalid character literals with a non-escaped whitespace character.
-  Enabled: true
-
-Lint/LiteralInCondition:
+Lint/LiteralAsCondition:
   Description: Checks of literals used in conditions.
   Details: >-
       Pattern `while true` is allowed, disable the cop with
-      `while true # rubocop:disable Lint/LiteralInCondition` in this case.
+      `while true # rubocop:disable Lint/LiteralAsCondition` in this case.
   Enabled: true
 
 # Supports --auto-correct
@@ -277,7 +273,7 @@ Lint/UnifiedInteger:
   Enabled: false
 
 # Supports --auto-correct
-Lint/UnneededDisable:
+Lint/UnneededCopDisableDirective:
   Description: 'Checks for rubocop:disable comments that can be removed. Note: this
     cop is not disabled when disabling all cops. It must be explicitly disabled.'
   Enabled: true
@@ -333,22 +329,22 @@ Lint/UselessSetterCall:
 
 Lint/Void:
   Description: Possible use of operator/literal/variable in void context.
-  Enabled: true
+  Enabled: false # Triggers for any return value for #initialize and #[]=
 
-#################### Style ###########################
+#################### Layout ###########################
 
 # Supports --auto-correct
 Style/BlockDelimiters:
   Enabled: true
 
-Style/CaseIndentation:
+Layout/CaseIndentation:
   Enabled: true
   EnforcedStyle: 'end'
 
-Style/IndentationConsistency:
+Layout/IndentationConsistency:
   Enabled: true
 
-Style/IndentationWidth:
+Layout/IndentationWidth:
   Enabled: true
 
 # Supports --auto-correct

lib/truffle/java/java.rb

@@ -41,8 +41,7 @@ def hash
     def eql?(another)
       Truffle::Interop::Java.java_eql?(@value, @value)
     end
-
-    alias_method :eql?, :==
+    alias_method :==, :eql?
 
     def equal?(another)
       nil != @value and JavaObject === another and @value.equals(another.value)

lib/truffle/java/java_utilities.rb

@@ -367,7 +367,7 @@ def toString
     a_proxy = Module.new do
       define_singleton_method(:included, included_method)
       class << self
-        attr_accessor :java_class
+        attr_writer :java_class
         def java_class
           JavaUtilities::wrap_java_value(@java_class)
         end

lib/truffle/java/method_selector.rb

@@ -334,7 +334,7 @@ def initialize(value_class, boxed_class, wide_type=nil)
   class Parameters
     def initialize(params, var_args)
       @params = params.map { |t| Parameter.for_type(t) }
-      @var_args
+      @var_args = var_args
     end
 
     def [](index)

lib/truffle/socket/socket.rb

@@ -238,7 +238,7 @@ def self.unpack_sockaddr_in(sockaddr)
     _, address, port = Truffle::Socket::Foreign
       .unpack_sockaddr_in(sockaddr, false)
 
-    return port, address
+    [port, address]
   rescue SocketError => e
     if e.message =~ /ai_family not supported/
       raise ArgumentError, 'not an AF_INET/AF_INET6 sockaddr'

lib/truffle/socket/truffle/foreign.rb

@@ -166,7 +166,7 @@ def self.getaddrinfo(host, service = nil, family = nil, socktype = nil,
           res = Addrinfo.new(res[:ai_next])
         end
 
-        return addrinfos
+        addrinfos
       ensure
         hints.free if hints
 
@@ -282,7 +282,7 @@ def self.pack_sockaddr_in(host, port, family = ::Socket::AF_UNSPEC,
 
         res = Addrinfo.new(res_p.read_pointer)
 
-        return res[:ai_addr].read_string(res[:ai_addrlen])
+        res[:ai_addr].read_string(res[:ai_addrlen])
       ensure
         hints.free if hints
 

spec/truffle/.rubocop.yml

@@ -0,0 +1,4 @@
+inherit_from: ../../.rubocop.yml
+
+Style/StringLiterals:
+  Enabled: false

spec/truffle/array/fixtures/tarball_entry.tar

@@ -33,7 +33,6 @@ def capacity(ary)
 
   it "creates arrays with the requested capacity with a value block" do
     a = Array.new(17) { 1 }
-    capacity(a).should == 17    
+    capacity(a).should == 17
   end
 end
-

spec/truffle/array/new_spec.rb

@@ -22,14 +22,14 @@ def check_pack_equal_bytes(expected, actual)
   it "should pack expressions with loops" do
     ([1] * 1024).pack('l' * 1024).sum.should == 1024
   end
-  
+
   it "should be able to pack a tarball entry" do
     ary = ["metadata.gz", "0000444", "0000000", "0000000", "00000001244", "00000000044", "        ", " ", "0", nil,
            "ustar", "00", "wheel", "wheel", "0000000", "0000000", ""]
     pack_format = "a100a8a8a8a12a12a7aaa100a6a2a32a32a8a8a155"
     packed = ary.pack(pack_format)
-    expected = "metadata.gz\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x000000444\x000000000\x000000000\x0000000001244\x0000000000044\x00        0\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00ustar\x0000wheel\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00wheel\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x000000000\x000000000\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
-    packed.should == expected
+    path = fixture __FILE__, "tarball_entry.tar"
+    packed.should == File.binread(path)
   end
 
   # Real-world usage of Array#pack, gathered by Brandon Fish, not designed