From 69d8a732f0c8037b8f8835e17f76700da91c297e Mon Sep 17 00:00:00 2001
From: Zach Daniel <zach@zachdaniel.dev>
Date: Tue, 7 May 2024 20:01:29 -0400
Subject: [PATCH] fix: don't use `fragment("1")` because ecto requires a proper
 source

---
 lib/expr.ex                        | 2 +-
 test/manual_relationships_test.exs | 7 +++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/lib/expr.ex b/lib/expr.ex
index 984b5dc..f7a099e 100644
--- a/lib/expr.ex
+++ b/lib/expr.ex
@@ -1561,7 +1561,7 @@ defmodule AshPostgres.Expr do
         on_subquery: fn subquery ->
           subquery =
             Ecto.Query.from(row in Ecto.Query.exclude(subquery, :select),
-              select: fragment("1")
+              select: row
             )
             |> Map.put(:__ash_bindings__, subquery.__ash_bindings__)
 
diff --git a/test/manual_relationships_test.exs b/test/manual_relationships_test.exs
index 7aa900a..a1956db 100644
--- a/test/manual_relationships_test.exs
+++ b/test/manual_relationships_test.exs
@@ -16,10 +16,9 @@ defmodule AshPostgres.Test.ManualRelationshipsTest do
     end
 
     test "exists can be used" do
-      post =
-        Post
-        |> Ash.Changeset.new(%{title: "title"})
-        |> Api.create!()
+      Post
+      |> Ash.Changeset.new(%{title: "title"})
+      |> Api.create!()
 
       Comment
       |> Ash.Changeset.new(%{title: "title2"})