@contember/engine-content-api 1.2.0-rc.4 → 1.3.0-alpha.1

package/tests/cases/integration/graphQlRequests/update/oneHasOneInverse/delete.test.ts

@@ -1,5 +1,5 @@
 import { test } from 'vitest'
-import { execute, sqlDeferred, sqlTransaction } from '../../../../../src/test'
+import { execute, failedTransaction, sqlTransaction } from '../../../../../src/test'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
 import { siteSettingSchema } from './schema'
@@ -27,17 +27,16 @@ test('delete', async () => {
 					parameters: [testUuid(2)],
 					response: { rows: [{ id: testUuid(1) }] },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."site"
-              where "id" in (select "root_"."id"
-                             from "public"."site" as "root_"
-                             where "root_"."id" = ?)
-              returning "id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(1) }] },
-					},
-				]),
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`delete from "public"."site" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: {  },
+				},
 			]),
 		],
 		return: {
@@ -50,3 +49,98 @@ test('delete', async () => {
 	})
 })
 
+
+test('delete', async () => {
+	await execute({
+		schema: siteSettingSchema,
+		query: GQL`mutation {
+        updateSiteSetting(
+            by: {id: "${testUuid(2)}"},
+            data: {site: {delete: true}}
+          ) {
+          ok
+        }
+      }`,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: SQL`SELECT "root_"."id"
+							 FROM "public"."site_setting" AS "root_"
+							 WHERE "root_"."id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`SELECT "root_"."id"
+							 FROM "public"."site" AS "root_"
+							 WHERE "root_"."setting_id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`SELECT "root_"."id" AS "id", TRUE AS "allowed"
+							 FROM "public"."site" AS "root_"
+							 WHERE "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`DELETE
+							 FROM "public"."site"
+							 WHERE "id" IN (?)`,
+					parameters: [testUuid(1)],
+					response: {},
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSiteSetting: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+
+
+test('delete - denied', async () => {
+	await execute({
+		schema: siteSettingSchema,
+		query: GQL`mutation {
+        updateSiteSetting(
+            by: {id: "${testUuid(2)}"},
+            data: {site: {delete: true}}
+          ) {
+          ok
+        }
+      }`,
+		executes: [
+			...failedTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."setting_id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: false }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSiteSetting: {
+					ok: false,
+				},
+			},
+		},
+	})
+})
+

package/tests/cases/integration/graphQlRequests/update/oneHasOneInverse/disconnect.test.ts

@@ -5,11 +5,13 @@ import { testUuid } from '../../../../../src/testUuid'
 import { siteSettingSchema } from './schema'
 
 test('disconnect', async () => {
+	const settingId = testUuid(2)
+	const siteId = testUuid(1)
 	await execute({
 		schema: siteSettingSchema,
 		query: GQL`mutation {
         updateSiteSetting(
-            by: {id: "${testUuid(2)}"},
+            by: {id: "${settingId}"},
             data: {site: {disconnect: true}}
           ) {
           ok
@@ -19,25 +21,29 @@ test('disconnect', async () => {
 			...sqlTransaction([
 				{
 					sql: SQL`select "root_"."id" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
-					parameters: [testUuid(2)],
-					response: { rows: [{ id: testUuid(2) }] },
+					parameters: [settingId],
+					response: { rows: [{ id: settingId }] },
 				},
 				{
 					sql: SQL`select "root_"."id"
                        from "public"."site" as "root_"
                        where "root_"."setting_id" = ?`,
-					parameters: [testUuid(2)],
+					parameters: [settingId],
 					response: {
-						rows: [{ id: testUuid(1) }],
+						rows: [{ id: siteId }],
 					},
 				},
 				{
-					sql: SQL`
-						with "newData_" as
-						(select ? :: uuid as "setting_id", "root_"."id", "root_"."name"  from "public"."site" as "root_"  where "root_"."id" = ?)
-						update  "public"."site" set  "setting_id" =  "newData_"."setting_id"
-						from "newData_"  where "site"."id" = "newData_"."id"`,
-					parameters: [null, testUuid(1)],
+					sql: SQL`with "newData_" as
+              (select
+                 ? :: uuid as "setting_id",
+                 "root_"."id",
+                 "root_"."name"
+               from "public"."site" as "root_"
+               where "root_"."id" = ?) update "public"."site"
+              set "setting_id" = "newData_"."setting_id" from "newData_"
+              where "site"."id" = "newData_"."id"`,
+					parameters: [null, siteId],
 					response: { rowCount: 1 },
 				},
 			]),

package/tests/cases/integration/graphQlRequests/update/oneHasOneInverse/schema.ts

@@ -13,3 +13,16 @@ export const siteSettingSchema = new SchemaBuilder()
 			),
 	)
 	.buildSchema()
+
+export const siteSettingSchemaWithOrphanRemoval = new SchemaBuilder()
+	.entity('Site', entity =>
+		entity
+			.column('name', c => c.type(Model.ColumnType.String))
+			.oneHasOne('setting', r =>
+				r
+					.inversedBy('site')
+					.target('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+					.removeOrphan(),
+			),
+	)
+	.buildSchema()

package/tests/cases/integration/graphQlRequests/update/oneHasOneInverse/upsert.test.ts

@@ -53,11 +53,13 @@ test('upsert - exists', async () => {
 	})
 })
 test('upsert - not exists', async () => {
+	const settingId = testUuid(2)
+	const siteId = testUuid(1)
 	await execute({
 		schema: siteSettingSchema,
 		query: GQL`mutation {
         updateSiteSetting(
-            by: {id: "${testUuid(2)}"},
+            by: {id: "${settingId}"},
            data: {site: {upsert: {update: {name: "Mangoweb"}, create: {name: "Mgw"}}}}
           ) {
           ok
@@ -67,14 +69,14 @@ test('upsert - not exists', async () => {
 			...sqlTransaction([
 				{
 					sql: SQL`select "root_"."id" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
-					parameters: [testUuid(2)],
-					response: { rows: [{ id: testUuid(2) }] },
+					parameters: [settingId],
+					response: { rows: [{ id: settingId }] },
 				},
 				{
 					sql: SQL`select "root_"."id"
                        from "public"."site" as "root_"
                        where "root_"."setting_id" = ?`,
-					parameters: [testUuid(2)],
+					parameters: [settingId],
 					response: {
 						rows: [],
 					},
@@ -86,8 +88,8 @@ test('upsert - not exists', async () => {
 							select "root_"."id", "root_"."name", "root_"."setting_id"
               from "root_"
 							returning "id"`,
-					parameters: [testUuid(1), 'Mgw', testUuid(2)],
-					response: { rows: [{ id: testUuid(1) }] },
+					parameters: [siteId, 'Mgw', settingId],
+					response: { rows: [{ id: siteId }] },
 				},
 			]),
 		],

package/tests/cases/integration/graphQlRequests/update/oneHasOneOwning/connect.test.ts

@@ -2,7 +2,7 @@ import { test } from 'vitest'
 import { execute, sqlTransaction } from '../../../../../src/test'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
-import { siteSettingSchema } from './schema'
+import { siteSettingSchema, siteSettingSchemaWithOrphanRemoval } from './schema'
 
 test('connect - same owner', async () => {
 	await execute({
@@ -47,6 +47,7 @@ test('connect - same owner', async () => {
 		},
 	})
 })
+
 test('connect - no owner', async () => {
 	await execute({
 		schema: siteSettingSchema,
@@ -103,6 +104,7 @@ test('connect - no owner', async () => {
 		},
 	})
 })
+
 test('connect - different owner', async () => {
 	await execute({
 		schema: siteSettingSchema,
@@ -173,3 +175,80 @@ test('connect - different owner', async () => {
 	})
 })
 
+test('connect - orphan removal', async () => {
+	const settingId = testUuid(1)
+	const siteId = testUuid(2)
+	const settingId2 = testUuid(3)
+	await execute({
+		schema: siteSettingSchemaWithOrphanRemoval,
+		query: GQL`mutation {
+        updateSite(
+            by: {id: "${siteId}"},
+            data: {setting: {connect: {id: "${settingId}"}}}
+          ) {
+          ok
+        }
+      }`,
+		executes: [
+			...sqlTransaction([
+				// the owner
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [siteId],
+					response: { rows: [{ id: siteId }] },
+				},
+				// new inverse
+				{
+					sql: SQL`select "root_"."id" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
+					parameters: [settingId],
+					response: { rows: [{ id: settingId }] },
+				},
+				// current inverse
+				{
+					sql: 'select "root_"."setting_id"  from "public"."site" as "root_"   where "root_"."id" = ?',
+					parameters: [siteId],
+					response: { rows: [{ setting_id: settingId2 }] },
+				},
+				// owner of new inverse
+				{
+					sql: 'select "root_"."id"  from "public"."site" as "root_"   where "root_"."setting_id" = ?',
+					parameters: [settingId],
+					response: { rows: [] },
+				},
+				{
+					sql: 'with "newData_" as (select ? :: uuid as "setting_id", "root_"."id", "root_"."name"  from "public"."site" as "root_"   where "root_"."id" = ?) update  "public"."site" set  "setting_id" =  "newData_"."setting_id"   from "newData_"   where "site"."id" = "newData_"."id"',
+					parameters: [settingId, siteId],
+					response: { rowCount: 1 },
+				},
+				{
+					sql: 'select "root_"."id"  from "public"."site_setting" as "root_"   where "root_"."id" = ?',
+					parameters: [settingId2],
+					response: { rows: [{ id: settingId2 }] },
+				},
+				{
+					sql: 'select "root_"."id" as "id", true as "allowed" from "public"."site_setting" as "root_" where "root_"."id" = ?',
+					parameters: [settingId2],
+					response: { rows: [{ id: settingId2, allowed: true }] },
+				},
+				{
+					sql: 'select "root_"."id" as "id", "root_"."setting_id" as "ref" from "public"."site" as "root_" where "setting_id" in (?)',
+					parameters: [settingId2],
+					response: { rows: [] },
+				},
+				{
+					sql: 'DELETE FROM "public"."site_setting" WHERE "id" IN (?)',
+					parameters: [settingId2],
+					response: { rowCount: 1 },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSite: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+

package/tests/cases/integration/graphQlRequests/update/oneHasOneOwning/connectOrCreate.test.ts

@@ -0,0 +1,51 @@
+import { test } from 'vitest'
+import { execute, sqlTransaction } from '../../../../../src/test'
+import { GQL, SQL } from '../../../../../src/tags'
+import { testUuid } from '../../../../../src/testUuid'
+import { siteSettingSchema } from './schema'
+
+test('connect or create', async () => {
+	await execute({
+		schema: siteSettingSchema,
+		query: GQL`mutation {
+        updateSite(
+            by: {id: "${testUuid(2)}"},
+            data: {setting: {connectOrCreate: {connect: {url: "xyz"}, create: {url: "abcd"}}}}
+          ) {
+          ok
+        }
+      }`,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" from "public"."site_setting" as "root_" where "root_"."url" = ?`,
+					parameters: ['xyz'],
+					response: { rows: [{ id: testUuid(3) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."setting_id" = ?`,
+					parameters: [testUuid(3)],
+					response: { rows: [] },
+				},
+				{
+					sql: `with "newData_" as (select ? :: uuid as "setting_id", "root_"."id", "root_"."name"  from "public"."site" as "root_"  where "root_"."id" = ?)
+						update  "public"."site" set  "setting_id" =  "newData_"."setting_id"   from "newData_"  where "site"."id" = "newData_"."id"`,
+					parameters: [testUuid(3), testUuid(2)],
+					response: { rowCount: 1 },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSite: {
+					ok: true,
+				},
+			},
+		},
+	})
+})

package/tests/cases/integration/graphQlRequests/update/oneHasOneOwning/create.test.ts

@@ -2,7 +2,7 @@ import { test } from 'vitest'
 import { execute, sqlTransaction } from '../../../../../src/test'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
-import { siteSettingSchema } from './schema'
+import { siteSettingSchema, siteSettingSchemaWithOrphanRemoval } from './schema'
 
 test('create', async () => {
 	await execute({
@@ -67,3 +67,83 @@ test('create', async () => {
 	})
 })
 
+test('create - orphan removal', async () => {
+	const siteId = testUuid(2)
+	const currentSettingId = testUuid(3)
+	await execute({
+		schema: siteSettingSchemaWithOrphanRemoval,
+		query: GQL`mutation {
+        updateSite(
+            by: {id: "${siteId}"},
+            data: {setting: {create: {url: "http://mangoweb.cz"}}}
+          ) {
+          ok
+        }
+      }`,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [siteId],
+					response: { rows: [{ id: siteId }] },
+				},
+				// get current inverse side
+				{
+					sql: 'select "root_"."setting_id"  from "public"."site" as "root_"   where "root_"."id" = ?',
+					parameters: [siteId],
+					response: { rows: [{ setting_id: currentSettingId }] },
+				},
+				{
+					sql: SQL`with "root_" as
+							(select ? :: uuid as "id", ? :: text as "url")
+							insert into "public"."site_setting" ("id", "url")
+							select "root_"."id", "root_"."url"
+              from "root_"
+							returning "id"`,
+					parameters: [testUuid(1), 'http://mangoweb.cz'],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`with "newData_" as
+              (select
+                 ? :: uuid as "setting_id",
+                 "root_"."id",
+                 "root_"."name"
+               from "public"."site" as "root_"
+               where "root_"."id" = ?) update "public"."site"
+              set "setting_id" = "newData_"."setting_id" from "newData_"
+              where "site"."id" = "newData_"."id"`,
+					parameters: [testUuid(1), siteId],
+					response: { rowCount: 1 },
+				},
+				{
+					sql: 'select "root_"."id"  from "public"."site_setting" as "root_"   where "root_"."id" = ?',
+					parameters: [currentSettingId],
+					response: { rows: [{ id: currentSettingId }] },
+				},
+				{
+					sql: 'select "root_"."id" as "id", true as "allowed" from "public"."site_setting" as "root_" where "root_"."id" = ?',
+					parameters: [currentSettingId],
+					response: { rows: [{ id: currentSettingId, allowed: true }] },
+				},
+				{
+					sql: 'select "root_"."id" as "id", "root_"."setting_id" as "ref" from "public"."site" as "root_" where "setting_id" in (?)',
+					parameters: [currentSettingId],
+					response: { rows: [] },
+				},
+				{
+					sql: 'DELETE FROM "public"."site_setting" WHERE "id" IN (?)',
+					parameters: [currentSettingId],
+					response: { rowCount: 1 },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSite: {
+					ok: true,
+				},
+			},
+		},
+	})
+})

package/tests/cases/integration/graphQlRequests/update/oneHasOneOwning/delete.test.ts

@@ -1,5 +1,5 @@
 import { test } from 'vitest'
-import { execute, sqlDeferred, sqlTransaction } from '../../../../../src/test'
+import { execute, failedTransaction, sqlTransaction } from '../../../../../src/test'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
 import { siteSettingSchema } from './schema'
@@ -31,31 +31,21 @@ test('delete', async () => {
 						rows: [{ setting_id: testUuid(1) }],
 					},
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."site_setting"
-              where "id" in (select "root_"."id"
-                             from "public"."site_setting" as "root_"
-                             where "root_"."id" = ?)
-              returning "id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(1) }] },
-					},
-					{
-						sql: SQL`
-								with "newdata_" as
-								(select ? :: uuid as "setting_id", "root_"."id", "root_"."name"
-									from "public"."site" as "root_"
-									where "root_"."setting_id" in (?))
-								update "public"."site"
-								set "setting_id" = "newdata_"."setting_id" from "newdata_"
-								where "site"."id" = "newdata_"."id"`,
-						parameters: [null, testUuid(1)],
-						response: {
-							rowCount: 1,
-						},
-					},
-				]),
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."setting_id" as "ref", true as "allowed" from "public"."site" as "root_" where "setting_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`delete from "public"."site_setting" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
 			]),
 		],
 		return: {
@@ -68,3 +58,56 @@ test('delete', async () => {
 	})
 })
 
+
+test('delete denied', async () => {
+	await execute({
+		schema: siteSettingSchema,
+		query: GQL`mutation {
+        updateSite(
+            by: {id: "${testUuid(2)}"},
+            data: {setting: {delete: true}}
+          ) {
+          ok
+          errorMessage
+        }
+      }`,
+		executes: [
+			...failedTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."site" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`select "root_"."setting_id"
+                       from "public"."site" as "root_"
+                       where "root_"."id" = ?`,
+					parameters: [testUuid(2)],
+					response: {
+						rows: [{ setting_id: testUuid(1) }],
+					},
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."site_setting" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."setting_id" as "ref", true as "allowed" from "public"."site" as "root_" where "setting_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1), allowed: false }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateSite: {
+					ok: false,
+					errorMessage: 'Execution has failed:\n'
+					+ 'setting: ForeignKeyConstraintViolation (Cannot delete 123e4567-e89b-12d3-a456-000000000001 row(s) of entity SiteSetting, because it is still referenced from 123e4567-e89b-12d3-a456-000000000002 row(s) of entity Site in relation setting. OnDelete behaviour of this relation is set to "set null". This is possibly caused by ACL denial.)',
+				},
+			},
+		},
+	})
+})
+

package/tests/cases/integration/graphQlRequests/update/oneHasOneOwning/schema.ts

@@ -8,8 +8,21 @@ export const siteSettingSchema = new SchemaBuilder()
 			.oneHasOne('setting', r =>
 				r
 					.inversedBy('site')
-					.target('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+					.target('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String).unique()))
 					.onDelete(Model.OnDelete.setNull),
 			),
 	)
 	.buildSchema()
+
+export const siteSettingSchemaWithOrphanRemoval = new SchemaBuilder()
+	.entity('Site', entity =>
+		entity
+			.column('name', c => c.type(Model.ColumnType.String))
+			.oneHasOne('setting', r =>
+				r
+					.inversedBy('site')
+					.target('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+					.removeOrphan(),
+			),
+	)
+	.buildSchema()