Remove TODO about duplicated named group Regex

makenowjust · ysbaddaden · commit fd55e8da92e1 · 2018-01-22T11:01:29.000+01:00
diff --git a/spec/std/match_data_spec.cr b/spec/std/match_data_spec.cr
@@ -49,8 +49,7 @@ describe "Regex::MatchData" do
     end
 
     it "captures duplicated named group" do
-      # TODO: replace with regexp literal after next version release
-      re = Regex.new "(?:(?<g1>foo)|(?<g1>bar))*"
+      re = /(?:(?<g1>foo)|(?<g1>bar))*/
 
       ("foo" =~ re).should eq(0)
       $~["g1"].should eq("foo")
@@ -116,8 +115,7 @@ describe "Regex::MatchData" do
     end
 
     it "captures duplicated named group" do
-      # TODO: replace with regexp literal after next version release
-      re = Regex.new "(?:(?<g1>foo)|(?<g1>bar))*"
+      re = /(?:(?<g1>foo)|(?<g1>bar))*/
 
       ("foo" =~ re).should eq(0)
       $~["g1"]?.should eq("foo")
@@ -203,8 +201,7 @@ describe "Regex::MatchData" do
     end
 
     it "gets a hash of named captures with duplicated name" do
-      # TODO: replace with regexp literal after next version release
-      "Crystal".match(Regex.new "(?<name>Cr)y(?<name>s)").not_nil!.named_captures.should eq({"name" => "s"})
+      "Crystal".match(/(?<name>Cr)y(?<name>s)/).not_nil!.named_captures.should eq({"name" => "s"})
     end
   end
 
@@ -252,8 +249,7 @@ describe "Regex::MatchData" do
     end
 
     it "converts into a hash with duplicated names" do
-      # TODO: replace with regexp literal after next version release
-      "Crystal".match(Regex.new "(Cr)(?<name>s)?(yst)?(?<name>al)?").not_nil!.to_h.should eq({
+      "Crystal".match(/(Cr)(?<name>s)?(yst)?(?<name>al)?/).not_nil!.to_h.should eq({
              0 => "Crystal",
              1 => "Cr",
         "name" => "al",
