🧪 test: adjust tests

Author: m1212e

test/src/abilities.test.ts

@@ -115,7 +115,7 @@ describe("test rumble abilities", async () => {
 		).toEqual(10);
 	});
 
-	test("deny indirect read with helper implementation on one to one", async () => {
+	test("error indirect read with helper implementation on one to one with error on non nullable relationship", async () => {
 		rumble.abilityBuilder.comments.allow(["read"]);
 
 		const { executor, yogaInstance } = build();
@@ -140,6 +140,26 @@ describe("test rumble abilities", async () => {
 		]);
 	});
 
+	test("deny indirect read with helper implementation on one to one with error on nullable relationship", async () => {
+		rumble.abilityBuilder.comments.allow(["read"]);
+
+		const { executor, yogaInstance } = build();
+		const r = await executor({
+			document: parse(/* GraphQL */ `
+        query {
+          findManyComments {
+            id
+            post {
+              id
+            }
+          }
+        }
+      `),
+		});
+
+		expect((r as any).data.findManyComments.length).toEqual(10);
+	});
+
 	test("allow indirect read with helper implementation on one to one", async () => {
 		rumble.abilityBuilder.comments.allow(["read"]);
 		rumble.abilityBuilder.users.allow(["read"]);

test/src/db/schema.ts

@@ -37,9 +37,12 @@ export const comments = table("comments", {
 	postId: t.text("post_id").references(() => posts.id, {
 		onDelete: "cascade",
 	}),
-	ownerId: t.text("owner_id").references(() => users.id, {
-		onDelete: "cascade",
-	}),
+	ownerId: t
+		.text("owner_id")
+		.notNull()
+		.references(() => users.id, {
+			onDelete: "cascade",
+		}),
 });
 
 export const usersRelations = relations(users, ({ one, many }) => ({