Move MatchData to rely less on shared examples

Author: Earlopain

core/matchdata/captures_spec.rb

@@ -1,6 +1,13 @@
 require_relative '../../spec_helper'
-require_relative 'shared/captures'
+require_relative 'fixtures/classes'
 
 describe "MatchData#captures" do
-  it_behaves_like :matchdata_captures, :captures
+  it "returns an array of the match captures" do
+    /(.)(.)(\d+)(\d)/.match("THX1138.").captures.should == ["H","X","113","8"]
+  end
+
+  it "returns instances of String when given a String subclass" do
+    str = MatchDataSpecs::MyString.new("THX1138: The Movie")
+    /(.)(.)(\d+)(\d)/.match(str).captures.each { |c| c.should.instance_of?(String) }
+  end
 end

core/matchdata/deconstruct_spec.rb

@@ -1,6 +1,7 @@
 require_relative '../../spec_helper'
-require_relative 'shared/captures'
 
 describe "MatchData#deconstruct" do
-  it_behaves_like :matchdata_captures, :deconstruct
+  it "is an alias of MatchData#captures" do
+    MatchData.instance_method(:deconstruct).should == MatchData.instance_method(:captures)
+  end
 end

core/matchdata/eql_spec.rb

@@ -1,6 +1,26 @@
 require_relative '../../spec_helper'
-require_relative 'shared/eql'
 
 describe "MatchData#eql?" do
-  it_behaves_like :matchdata_eql, :eql?
+  it "returns true if both operands have equal target strings, patterns, and match positions" do
+    a = 'haystack'.match(/hay/)
+    b = 'haystack'.match(/hay/)
+    a.eql?(b).should == true
+  end
+
+  it "returns false if the operands have different target strings" do
+    a = 'hay'.match(/hay/)
+    b = 'haystack'.match(/hay/)
+    a.eql?(b).should == false
+  end
+
+  it "returns false if the operands have different patterns" do
+    a = 'haystack'.match(/h.y/)
+    b = 'haystack'.match(/hay/)
+    a.eql?(b).should == false
+  end
+
+  it "returns false if the argument is not a MatchData object" do
+    a = 'haystack'.match(/hay/)
+    a.eql?(Object.new).should == false
+  end
 end

core/matchdata/equal_value_spec.rb

@@ -1,6 +1,7 @@
 require_relative '../../spec_helper'
-require_relative 'shared/eql'
 
 describe "MatchData#==" do
-  it_behaves_like :matchdata_eql, :==
+  it "is an alias of MatchData#eql?" do
+    MatchData.instance_method(:==).should == MatchData.instance_method(:eql?)
+  end
 end

core/matchdata/length_spec.rb

@@ -1,6 +1,7 @@
 require_relative '../../spec_helper'
-require_relative 'shared/length'
 
 describe "MatchData#length" do
-  it_behaves_like :matchdata_length, :length
+  it "is an alias of MatchData#size" do
+    MatchData.instance_method(:length).should == MatchData.instance_method(:size)
+  end
 end

core/matchdata/shared/captures.rb

@@ -1,6 +1,7 @@
 require_relative '../../spec_helper'
-require_relative 'shared/length'
 
 describe "MatchData#size" do
-  it_behaves_like :matchdata_length, :size
+  it "should return the number of elements in the match array" do
+    /(.)(.)(\d+)(\d)/.match("THX1138.").size.should == 5
+  end
 end