Fix bad merge of SkinnyMethodAdapter spec.

The "let" used on master was getting assigned the result of the "expect" calls merged in from jruby-1_7.
jruby · Jan 20, 2016 · 567509b · 567509b
1 parent 18866ad
commit 567509b
Showing 1 changed file with 3 additions and 16 deletions.
diff --git a/spec/compiler/skinnymethodadapter_spec.rb b/spec/compiler/skinnymethodadapter_spec.rb
@@ -13,8 +13,8 @@
 describe "SkinnyMethodAdapter" do
   let(:instance_methods) { SkinnyMethodAdapter.instance_methods.map(&:to_s) }
   let(:insn_opcodes) do
-    insn_opcodes = Opcodes.constants.select do |c|
-      case c.to_s # for 1.9's symbols
+    Opcodes.constants.map(&:to_s).select do |c|
+      case c
 
       when /ACC_/, # access modifiers
            /V1_/, # version identifiers
@@ -36,20 +36,7 @@
       else
         true
       end
-    end.map(&:to_s).map(&:downcase)
-    # 1.9 makes them symbols, so to_s the lot
-    insn_opcodes = insn_opcodes.map(&:to_s)
-    instance_methods = SkinnyMethodAdapter.instance_methods.map(&:to_s)
-
-    insn_opcodes.each do |opcode|
-      opcode = opcode.downcase
-      expect(instance_methods).to include(opcode)
-    end
-
-    expect(instance_methods).to include("go_to")
-    expect(instance_methods).to include("voidreturn")
-    expect(instance_methods).to include("instance_of")
-    expect(instance_methods).to include("newobj")
+    end.map(&:downcase)
   end
 
   it "supports all JVM opcodes" do