Handle default expected_roundtrip more explicitly for clarity

puma · Nov 6, 2020 · 0bb2df3 · 0bb2df3
1 parent 1015bad
commit 0bb2df3
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/test/test_json.rb b/test/test_json.rb
@@ -93,13 +93,14 @@ def test_generate_raises_error_for_unexpected_value_type
 
   private
 
-  def assert_puma_json_generates_string(expected_output, value_to_serialize, expected_roundtrip: value_to_serialize)
+  def assert_puma_json_generates_string(expected_output, value_to_serialize, expected_roundtrip: nil)
     actual_output = Puma::JSON.generate(value_to_serialize)
     assert_equal expected_output, actual_output
 
     if value_to_serialize.nil?
       assert_nil ::JSON.parse(actual_output)
     else
+      expected_roundtrip ||= value_to_serialize
       assert_equal expected_roundtrip, ::JSON.parse(actual_output)
     end
   end