Handle support for both Jason and JSON

We will simply rescue the exception thrown by both libraries and return
a simple `:error` to keep the interface consistent. We will need to keep
an eye on upstream changes in Ecto as support for both `JSON` and `Jason`
evolve.

closes: #159

Author: warmwaffles

lib/ecto/adapters/sqlite3/codec.ex

@@ -10,7 +10,9 @@ defmodule Ecto.Adapters.SQLite3.Codec do
   def bool_decode(v), do: {:ok, v}
 
   def json_decode(v) when is_binary(v) do
-    Application.get_env(:ecto_sqlite3, :json_library, Jason).decode(v)
+    {:ok, Application.get_env(:ecto_sqlite3, :json_library, Jason).decode!(v)}
+  rescue
+    _ -> :error
   end
 
   def json_decode(v), do: {:ok, v}
@@ -89,7 +91,9 @@ defmodule Ecto.Adapters.SQLite3.Codec do
   def json_encode(value) when is_bitstring(value), do: {:ok, value}
 
   def json_encode(value) do
-    Application.get_env(:ecto_sqlite3, :json_library, Jason).encode(value)
+    {:ok, Application.get_env(:ecto_sqlite3, :json_library, Jason).encode!(value)}
+  rescue
+    _err -> :error
   end
 
   def blob_encode(nil), do: {:ok, nil}

test/ecto/adapters/sqlite3/codec_test.exs

@@ -35,10 +35,10 @@ defmodule Ecto.Adapters.SQLite3.CodecTest do
     end
 
     test "handles malformed json" do
-      {:error, _} = Codec.json_decode("")
-      {:error, _} = Codec.json_decode(" ")
-      {:error, _} = Codec.json_decode("{")
-      {:error, _} = Codec.json_decode("[")
+      :error = Codec.json_decode("")
+      :error = Codec.json_decode(" ")
+      :error = Codec.json_decode("{")
+      :error = Codec.json_decode("[")
     end
   end