Fix disapprove format

iHiD · iHiD · commit a1f5549b6391 · 2019-08-18T23:10:21.000+01:00
diff --git a/lib/analyzers/two_fer/analyze.rb b/lib/analyzers/two_fer/analyze.rb
@@ -173,10 +173,10 @@ def refer_to_mentor!
       raise FinishedFlowControlException
     end
 
-    def disapprove!(msg, *msg_args)
+    def disapprove!(msg, params = {})
       self.status = :disapprove
-      if msg_args.length > 0
-        self.comments << [MESSAGES[msg], *msg_args]
+      if params.length > 0
+        self.comments << {comment: MESSAGES[msg], params: params}
       else
         self.comments << MESSAGES[msg]
       end
diff --git a/test/exercises/two_fer_test.rb b/test/exercises/two_fer_test.rb
@@ -140,7 +140,8 @@ def self.two_fer(*foos)
     }
     results = TwoFer::Analyze.(source)
     assert_equal :disapprove, results[:status]
-    assert_equal [["ruby.two-fer.splat_args", name_variable: :foos]], results[:comments]
+    assert_equal [{comment: "ruby.two-fer.splat_args", params: {name_variable: :foos}}], results[:comments]
+
   end
 
   # ### 

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,8 @@ def self.two_fer(*foos)`
`140`	`140`	`}`
`141`	`141`	`results = TwoFer::Analyze.(source)`
`142`	`142`	`assert_equal :disapprove, results[:status]`
`143`		`- assert_equal [["ruby.two-fer.splat_args", name_variable: :foos]], results[:comments]`
	`143`	`+ assert_equal [{comment: "ruby.two-fer.splat_args", params: {name_variable: :foos}}], results[:comments]`
	`144`	`+`
`144`	`145`	`end`
`145`	`146`
`146`	`147`	`# ###`