Make updates based on review by @kotp at tommyschaefer#1

Author: tommyschaefer

lib/wordy_cases.rb

@@ -1,6 +1,6 @@
 class WordyCase < OpenStruct
   def test_name
-    'test_%s' % description.downcase.tr_s(' ', '_')
+    'test_%s' % description.downcase.tr(' ', '_')
   end
 
   def workload
@@ -17,9 +17,7 @@ def skipped
   private
 
   def indent(size, lines)
-    lines.split("\n").map do |line|
-      ' ' * size + line
-    end.join("\n")
+    lines.lines.each_with_object('') { |line, obj| obj << ' ' * size + line }
   end
 
   def assertion
@@ -48,13 +46,13 @@ def message_assertion
 
 class WordyCase::PreProcessor
   class << self
-    def call(row:)
-      row.merge('message' => message_for(row: row))
+    def call(row)
+      row.merge('message' => message_for(row))
     end
 
     private
 
-    def message_for(row:)
+    def message_for(row)
       return unless row['input'] == 'What is -3 plus 7 multiplied by -2?'
 
       'You should ignore order of precedence. -3 + 7 * -2 = -8, not %s'
@@ -64,6 +62,6 @@ def message_for(row:)
 
 WordyCases = proc do |data|
   JSON.parse(data)['cases'].map.with_index do |row, i|
-    WordyCase.new(WordyCase::PreProcessor.call(row: row).merge(index: i))
+    WordyCase.new(WordyCase::PreProcessor.call(row).merge(index: i))
   end
 end

test/wordy_cases_test.rb

@@ -66,14 +66,14 @@ def test_workload_without_expected
 class WordyCasePrProcessorTest < Minitest::Test
   def test_call_as_non_special_case
     row = { 'input' => '' }
-    processed_row = WordyCase::PreProcessor.call(row: row)
+    processed_row = WordyCase::PreProcessor.call(row)
 
     assert_equal({ 'input' => '', 'message' => nil }, processed_row)
   end
 
   def test_call_as_special_case
     row = { 'input' => 'What is -3 plus 7 multiplied by -2?' }
-    processed_row = WordyCase::PreProcessor.call(row: row)
+    processed_row = WordyCase::PreProcessor.call(row)
     expected_row = {
       'input' => 'What is -3 plus 7 multiplied by -2?',
       'message' => 'You should ignore order of precedence. -3 + 7 * -2 = -8, not %s',