Fix to accepting unquoted strings

Author: s3cur3

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v0.3.1
+
+Bug fix to accept more unquoted strings, including those that have Elixir delimiters in them like quotes, parentheses, etc.
+
 ## v0.3.0
 
 ### New features

lib/parameterized_test.ex

@@ -260,12 +260,12 @@ defmodule ParameterizedTest do
   end
 
   defp eval_cell(cell, row, _context) do
-    case Code.eval_string(cell) do
+    case Code.eval_string(cell, [], log: false) do
       {val, []} -> val
       _ -> raise "Failed to evaluate example cell `#{cell}` in row `#{row}`}"
     end
   rescue
-    _e in [SyntaxError, CompileError] ->
+    _e in [SyntaxError, CompileError, TokenMissingError] ->
       String.trim(cell)
 
     e ->

mix.exs

@@ -6,7 +6,7 @@ defmodule ParameterizedTest.MixProject do
   def project do
     [
       app: :parameterized_test,
-      version: "0.3.0",
+      version: "0.3.1",
       elixir: "~> 1.14",
       elixirc_paths: elixirc_paths(Mix.env()),
       start_permanent: Mix.env() == :prod,

test/parameterized_test_test.exs

@@ -155,18 +155,30 @@ defmodule ParameterizedTestTest do
 
   param_test "interprets otherwise unparseable values as strings",
              """
-             |  value  | unquoted string key |
-             |---------|---------------------|
-             |    1    | foo, bar            |
-             |  2 3 4  |                     |
+             |  value  | unquoted string key          |
+             |---------|------------------------------|
+             |    1    | foo, bar & baz               |
+             |  2 3 4  |                              |
+             |  5      | Won't error out              |
+             |  6      | a !@#$%^&*():"_;',./][}{<> z |
+             |  7      |  %{shoes: 19_99,  |
              """,
              %{value: value, "unquoted string key": unquoted} do
     case value do
       1 ->
-        assert unquoted == "foo, bar"
+        assert unquoted == "foo, bar & baz"
 
       "2 3 4" ->
         assert is_nil(unquoted)
+
+      5 ->
+        assert unquoted == "Won't error out"
+
+      6 ->
+        assert unquoted == "a !@#$%^&*():\"_;',./][}{<> z"
+
+      7 ->
+        assert unquoted == "%{shoes: 19_99,"
     end
   end
 
@@ -298,28 +310,10 @@ defmodule ParameterizedTestTest do
     end
   end
 
-  test "fails to compile Markdown rows with invalid Elixir in them" do
-    assert_raise RuntimeError, fn ->
-      defmodule FailToEvaluateBadMarkdownTest do
-        use ExUnit.Case, async: true
-
-        import ParameterizedTest
-
-        param_test "test with invalid Elixir in a cell",
-                   """
-                   | spending_by_category  |
-                   | %{shoes: 19_99,       |
-                   """,
-                   %{gets_free_shipping?: gets_free_shipping?} do
-          assert gets_free_shipping?
-        end
-      end
-    end
-  end
-
   test "fails to compile when the keys in a handrolled list don't all match" do
     assert_raise RuntimeError, fn ->
-      defmodule FailToEvaluateBadMarkdownTest do
+      defmodule FailToEvaluateMismatchedKeys do
+        @moduledoc false
         use ExUnit.Case, async: true
 
         import ParameterizedTest