Issue #168: trailing content on an sobelow_ignore line silently breaks parsing

lib/sobelow.ex

@@ -291,6 +291,10 @@ defmodule Sobelow do
     Application.get_env(:sobelow, key)
   end
 
+  def put_env(key, value) do
+    Application.put_env(:sobelow, key, value)
+  end
+
   defp print_banner do
     """
     ##############################################

lib/sobelow/parse.ex

@@ -58,7 +58,7 @@ defmodule Sobelow.Parse do
     if Sobelow.get_env(:skip) do
       String.replace(
         content,
-        ~r/#\s?sobelow_skip (\[(\"[^"]+\"(,|, )?)+\])/,
+        ~r/#\s?sobelow_skip (\[(\"[^"]+?\"(,|, )?)+?\]).*/,
         "@sobelow_skip \\g{1}"
       )
     else

test/fixtures/parser/parse_skip.ex

@@ -0,0 +1,8 @@
+defmodule Mod do
+  @moduledoc false
+
+  # sobelow_skip ["Foo"] (no foo violations here)
+  def func() do
+    :ok
+  end
+end

test/parser_test.exs

@@ -2,6 +2,7 @@ defmodule SobelowTest.ParserTest do
   use ExUnit.Case
   import ExUnit.CaptureIO
   alias Sobelow.RCE.CodeModule
+  alias Sobelow.Parse
 
   @metafile %{filename: "test.ex", controller?: true}
 
@@ -27,4 +28,15 @@ defmodule SobelowTest.ParserTest do
 
     assert capture_io(run_test) =~ "Code Execution in `Code.eval_string` - Medium Confidence"
   end
+
+  test "Remainder of line after sobelow_skip expression is ignored" do
+    Sobelow.put_env(:skip, true)
+    file = "./test/fixtures/parser/parse_skip.ex"
+
+    %{def_funs: defs} = Parse.ast(file)
+                        |> Parse.get_meta_funs()
+    assert not Enum.empty?(defs)
+  after
+    Sobelow.put_env(:skip, false)
+   end 
 end