fix: don't clear fields on nil result

chore: add tests for destroy

lib/graphql/resolver.ex

@@ -346,13 +346,14 @@ defmodule AshGraphql.Graphql.Resolver do
      }}
   end
 
+  defp clear_fields(nil, _, _), do: nil
+
   defp clear_fields(result, resource, resolution) do
     resolution
     |> fields(true)
     |> Enum.map(fn field ->
       Ash.Resource.Info.aggregate(resource, field.schema_node.identifier) ||
-        Ash.Resource.Info.calculation(resource, field.schema_node.identifier) ||
-        Ash.Resource.Info.attribute(resource, field.schema_node.identifier)
+        Ash.Resource.Info.calculation(resource, field.schema_node.identifier)
     end)
     |> Enum.filter(& &1)
     |> Enum.map(& &1.name)

test/destroy_test.exs

@@ -0,0 +1,43 @@
+defmodule AshGraphql.DestroyTest do
+  use ExUnit.Case, async: false
+
+  setup do
+    on_exit(fn ->
+      try do
+        ETS.Set.delete(ETS.Set.wrap_existing!(AshGraphql.Test.Post))
+        ETS.Set.delete(ETS.Set.wrap_existing!(AshGraphql.Test.Comment))
+      rescue
+        _ ->
+          :ok
+      end
+    end)
+  end
+
+  test "a destroy works" do
+    post = AshGraphql.Test.Api.create!(Ash.Changeset.new(AshGraphql.Test.Post, text: "foobar"))
+
+    resp =
+      """
+      mutation DeletePost($id: ID) {
+        deletePost(id: $id) {
+          result{
+            text
+          }
+          errors{
+            message
+          }
+        }
+      }
+      """
+      |> Absinthe.run(AshGraphql.Test.Schema,
+        variables: %{
+          "id" => post.id
+        }
+      )
+
+    assert {:ok, result} = resp
+
+    refute Map.has_key?(result, :errors)
+    assert %{data: %{"deletePost" => %{"result" => %{"text" => "foobar"}}}} = result
+  end
+end

test/support/resources/post.ex

@@ -17,6 +17,7 @@ defmodule AshGraphql.Test.Post do
       create :create_post, :create_confirm
       create :upsert_post, :upsert, upsert?: true
       update :update_post, :update
+      destroy :delete_post, :destroy
     end
   end