Add more tests and align them

Author: corka149

test/jsonpatch/operation/add_test.exs

@@ -53,13 +53,20 @@ defmodule Jsonpatch.Operation.AddTest do
     assert %{"a" => [0, 1, 3]} = Operation.apply_op(patch, target)
   end
 
-  test "Add a value to an empty array" do
+  test "Add a value to an empty array with binary key" do
     patch = %Add{path: "/a/0", value: 3}
     target = %{"a" => []}
 
     assert %{"a" => [3]} = Operation.apply_op(patch, target)
   end
 
+  test "Add a value to an empty array with atom key" do
+    patch = %Add{path: "/a/0", value: 3}
+    target = %{a: []}
+
+    assert %{a: [3]} = Operation.apply_op(patch, target, keys: :atoms)
+  end
+
   test "Add a value to an array with invalid index" do
     patch = %Add{path: "/a/b", value: 3}
     target = %{"a" => [0, 1, 2]}

test/jsonpatch/operation/remove_test.exs

@@ -54,7 +54,15 @@ defmodule RemoveTest do
     }
 
     remove_patch = %Remove{path: "/nameX"}
-    assert {:error, :invalid_path, "nameX"} = Jsonpatch.apply_patch(remove_patch, target)
+    assert {:error, :invalid_path, "nameX"} = Operation.apply_op(remove_patch, target)
+  end
+
+  test "Remove element in map with atom keys" do
+    target = %{name: "Ceasar", age: 66}
+
+    remove_patch = %Remove{path: "/age"}
+
+    assert %{name: "Ceasar"} = Operation.apply_op(remove_patch, target, keys: :atoms)
   end
 
   test "Remove element by invalid index" do
@@ -66,7 +74,7 @@ defmodule RemoveTest do
     }
 
     remove_patch = %Remove{path: "/hobbies/a"}
-    assert {:error, :invalid_index, "a"} = Jsonpatch.apply_patch(remove_patch, target)
+    assert {:error, :invalid_index, "a"} = Operation.apply_op(remove_patch, target)
 
     # Longer path
     target = %{
@@ -77,11 +85,11 @@ defmodule RemoveTest do
     }
 
     remove_patch = %Remove{path: "/hobbies/b/description"}
-    assert {:error, :invalid_index, "b"} = Jsonpatch.apply_patch(remove_patch, target)
+    assert {:error, :invalid_index, "b"} = Operation.apply_op(remove_patch, target)
 
     # Longer path, numeric - out of
     remove_patch = %Remove{path: "/hobbies/1/description"}
-    assert {:error, :invalid_index, "1"} = Jsonpatch.apply_patch(remove_patch, target)
+    assert {:error, :invalid_index, "1"} = Operation.apply_op(remove_patch, target)
   end
 
   test "Return error when patch error was provided to remove operation" do

test/jsonpatch/operation/test_test.exs

@@ -29,6 +29,14 @@ defmodule Jsonpatch.Operation.TestTest do
     assert ^target = Operation.apply_op(test_op, target)
   end
 
+  test "Test with atom as key" do
+    target = %{role: "Developer"}
+
+    test_op = %Test{path: "/role", value: "Developer"}
+
+    assert ^target = Operation.apply_op(test_op, target, keys: :atoms)
+  end
+
   test "Fail to test element with path with multiple indices" do
     target = %{
       "a" => %{