Merge pull request #211 from covermymeds/ignore-non-string-dsn

Add check_dsn: false to source_code_path_pattern

Author: mitchellhenke

lib/sentry/config.ex

@@ -70,7 +70,7 @@ defmodule Sentry.Config do
   end
 
   def source_code_path_pattern do
-    get_config(:source_code_path_pattern, default: @default_path_pattern)
+    get_config(:source_code_path_pattern, default: @default_path_pattern, check_dsn: false)
   end
 
   def source_code_exclude_patterns do

test/config_test.exs

@@ -20,4 +20,21 @@ defmodule Sentry.ConfigTest do
 
     assert "my_server" == Config.server_name()
   end
+
+  describe "source_code_path_pattern" do
+    test "retrieves from environment" do
+      modify_env(:sentry, source_code_path_pattern: "**/*test.ex")
+      assert "**/*test.ex" == Config.source_code_path_pattern()
+    end
+
+    test "returns default when not configured" do
+      assert "**/*.ex" == Config.source_code_path_pattern()
+    end
+
+    test "does not retrieve from DSN" do
+      dsn = "https://public:[email protected]/2?source_code_path_pattern=test"
+      modify_env(:sentry, dsn: dsn)
+      refute "test" == Config.source_code_path_pattern()
+    end
+  end
 end