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

package/tests/cases/integration/graphQlRequests/delete/additionalDeleteFilter.test.ts

@@ -1,5 +1,5 @@
 import { test } from 'vitest'
-import { execute, sqlDeferred, sqlTransaction } from '../../../../src/test'
+import { execute, sqlTransaction } from '../../../../src/test'
 import { SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../src/tags'
@@ -13,51 +13,32 @@ test('delete post with additional filter', async () => {
 		query: GQL`
         mutation {
           deletePost(by: {id: "${testUuid(1)}"}, filter: {locale: {eq: "cs"}}) {
-            node {
-              id
-            }
+            ok
           }
         }`,
 		executes: [
 			...sqlTransaction([
 				{
-					sql: SQL`select
-                     "root_"."id" as "root_id"
-                     from "public"."post" as "root_"
-                   where "root_"."id" = ?`,
+					sql: SQL`select "root_"."id" from "public"."post" as "root_" where "root_"."id" = ?`,
 					parameters: [testUuid(1)],
-					response: {
-						rows: [
-							{
-								root_id: testUuid(1),
-							},
-						],
-					},
+					response: { rows: [{ id: testUuid(1) }] },
 				},
 				{
-					sql: SQL`select "root_"."id" from "public"."post" as "root_" where "root_"."id" = ?`,
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."post" as "root_" where "root_"."id" = ? and "root_"."locale" = ?`,
+					parameters: [testUuid(1), 'cs'],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`delete from "public"."post" where "id" in (?)`,
 					parameters: [testUuid(1)],
-					response: { rows: [{ id: testUuid(1) }] },
+					response: {  },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."post"
-            where "id" in (select "root_"."id"
-                           from "public"."post" as "root_"
-                           where "root_"."id" = ? and "root_"."locale" = ?)
-            returning "id"`,
-						parameters: [testUuid(1), 'cs'],
-						response: { rows: [{ id: testUuid(1) }] },
-					},
-				]),
 			]),
 		],
 		return: {
 			data: {
 				deletePost: {
-					node: {
-						id: testUuid(1),
-					},
+					ok: true,
 				},
 			},
 		},

package/tests/cases/integration/graphQlRequests/delete/cascadeDelete.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 { SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../src/tags'
@@ -19,68 +19,89 @@ test('delete author with posts and locales cascade delete', async () => {
 		query: GQL`
         mutation {
           deleteAuthor(by: {id: "${testUuid(1)}"}) {
-            node {
-              id
-            }
+            ok
           }
         }`,
 		executes: [
 			...sqlTransaction([
 				{
-					sql: SQL`select "root_"."id" as "root_id"
-                     from "public"."author" as "root_"
-                     where "root_"."id" = ?`,
+					sql: SQL`select "root_"."id" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref", true as "allowed" from "public"."post" as "root_" where "author_id" in (?)`,
 					parameters: [testUuid(1)],
-					response: {
-						rows: [
-							{
-								root_id: testUuid(1),
-							},
-						],
-					},
+					response: { rows: [{ id: testUuid(2), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."post_id" as "ref", true as "allowed" from "public"."post_locales" as "root_" where "post_id" in (?)`,
+					parameters: [testUuid(2)],
+					response: { rows: [{ id: testUuid(3), allowed: true }] },
 				},
+				{
+					sql: SQL`delete from "public"."author" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { },
+				},
+			]),
+		],
+		return: {
+			data: {
+				deleteAuthor: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+
+
+test('delete author with posts (denied) ', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', entity =>
+				entity.manyHasOne('author', relation => relation.target('Author').onDelete(Model.OnDelete.cascade)),
+			)
+			.entity('Author', entity => entity.column('name', column => column.type(Model.ColumnType.String)))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          deleteAuthor(by: {id: "${testUuid(1)}"}) {
+            ok
+            errorMessage
+          }
+        }`,
+		executes: [
+			...failedTransaction([
 				{
 					sql: SQL`select "root_"."id" from "public"."author" as "root_" where "root_"."id" = ?`,
 					parameters: [testUuid(1)],
 					response: { rows: [{ id: testUuid(1) }] },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."author"
-            where "id" in (select "root_"."id"
-                           from "public"."author" as "root_"
-                           where "root_"."id" = ?)
-            returning "id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(1) }] },
-					},
-					{
-						sql: SQL`delete from "public"."post"
-            where "id" in (select "root_"."id"
-                           from "public"."post" as "root_"
-                           where "root_"."author_id" in (?))
-            returning "id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(2) }, { id: testUuid(3) }] },
-					},
-					{
-						sql: SQL`delete from "public"."post_locales"
-            where "id" in (select "root_"."id"
-                           from "public"."post_locales" as "root_"
-                           where "root_"."post_id" in (?, ?))
-            returning "id"`,
-						parameters: [testUuid(2), testUuid(3)],
-						response: { rows: [{ id: testUuid(4) }, { id: testUuid(5) }] },
-					},
-				]),
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref", true as "allowed" from "public"."post" as "root_" where "author_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1), allowed: false }] },
+				},
 			]),
 		],
 		return: {
 			data: {
 				deleteAuthor: {
-					node: {
-						id: testUuid(1),
-					},
+					ok: false,
+					errorMessage: 'Execution has failed:\n' +
+						'unknown field: ForeignKeyConstraintViolation (Cannot delete 123e4567-e89b-12d3-a456-000000000001 row(s) of entity Author, because it is still referenced from 123e4567-e89b-12d3-a456-000000000002 row(s) of entity Post in relation author. OnDelete behaviour of this relation is set to "cascade". This is possibly caused by ACL denial.)',
 				},
 			},
 		},
@@ -88,3 +109,4 @@ test('delete author with posts and locales cascade delete', async () => {
 })
 
 
+

package/tests/cases/integration/graphQlRequests/delete/deleteById.test.ts

@@ -1,5 +1,5 @@
 import { test } from 'vitest'
-import { execute, sqlDeferred, sqlTransaction } from '../../../../src/test'
+import { execute, sqlTransaction } from '../../../../src/test'
 import { SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../src/tags'
@@ -62,17 +62,16 @@ test('delete post by id', async () => {
 					parameters: [testUuid(1)],
 					response: { rows: [{ id: testUuid(1) }] },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."post"
-            where "id" in (select "root_"."id"
-                           from "public"."post" 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"."post" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`delete from "public"."post" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
 			]),
 		],
 		return: {

package/tests/cases/integration/graphQlRequests/delete/orphanRemoval.test.ts

@@ -1,5 +1,5 @@
 import { test } from 'vitest'
-import { execute, sqlDeferred, sqlTransaction } from '../../../../src/test'
+import { execute, sqlTransaction } from '../../../../src/test'
 import { SchemaBuilder } from '@contember/schema-definition'
 import { GQL, SQL } from '../../../../src/tags'
 import { testUuid } from '../../../../src/testUuid'
@@ -38,26 +38,37 @@ test('delete post and orphaned content', async () => {
 					parameters: [testUuid(1)],
 					response: { rows: [{ id: testUuid(1) }] },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."post"
-            where "id" in (select "root_"."id"
-                           from "public"."post" as "root_"
-                           where "root_"."id" = ?)
-            returning "id", "content_id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(1), content_id: testUuid(2) }] },
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."content_id" as "_content_id", true as "allowed" from "public"."post" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: {
+						rows: [{ id: testUuid(1), _content_id: testUuid(2), allowed: true }],
+					},
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."content" as "root_" where "root_"."id" in (?)`,
+					parameters: [testUuid(2)],
+					response: {
+						rows: [{ id: testUuid(2), allowed: true }],
 					},
-					{
-						sql: SQL`delete from "public"."content"
-            where "id" in (select "root_"."id"
-                           from "public"."content" as "root_"
-                           where "root_"."id" in (?))
-            returning "id"`,
-						parameters: [testUuid(2)],
-						response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."content_id" as "ref" from "public"."post" as "root_" where "content_id" in (?)`,
+					parameters: [testUuid(2)],
+					response: {
+						rows: [{ id: testUuid(1), ref: testUuid(2) }],
 					},
-				]),
+				},
+				{
+					sql: SQL`delete from "public"."post" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { },
+				},
+				{
+					sql: SQL`delete from "public"."content" where "id" in (?)`,
+					parameters: [testUuid(2)],
+					response: { },
+				},
 			]),
 		],
 		return: {

package/tests/cases/integration/graphQlRequests/delete/restrictOnDelete.test.ts

@@ -0,0 +1,102 @@
+import { test } from 'vitest'
+import { execute, failedTransaction, sqlTransaction } from '../../../../src/test'
+import { SchemaBuilder } from '@contember/schema-definition'
+import { Model } from '@contember/schema'
+import { GQL, SQL } from '../../../../src/tags'
+import { testUuid } from '../../../../src/testUuid'
+
+test('delete author and with restrict on posts', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', entity =>
+				entity.manyHasOne('author', relation => relation.target('Author')),
+			)
+			.entity('Author', entity => entity.column('name', column => column.type(Model.ColumnType.String)))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          deleteAuthor(by: {id: "${testUuid(1)}"}) {
+            ok
+          }
+        }`,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref" from "public"."post" as "root_" where "author_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [] },
+				},
+				{
+					sql: SQL`delete from "public"."author" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { },
+				},
+			]),
+		],
+		return: {
+			data: {
+				deleteAuthor: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+
+
+test('delete author with restrict on posts - declined', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', entity =>
+				entity.manyHasOne('author', relation => relation.target('Author')),
+			)
+			.entity('Author', entity => entity.column('name', column => column.type(Model.ColumnType.String)))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          deleteAuthor(by: {id: "${testUuid(1)}"}) {
+            ok
+            errorMessage
+          }
+        }`,
+		executes: [
+			...failedTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref" from "public"."post" as "root_" where "author_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1) }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				deleteAuthor: {
+					ok: false,
+					errorMessage: 'Execution has failed:\n' +
+						'unknown field: ForeignKeyConstraintViolation (Cannot delete 123e4567-e89b-12d3-a456-000000000001 row(s) of entity Author, because it is still referenced from 123e4567-e89b-12d3-a456-000000000002 row(s) of entity Post in relation author. OnDelete behaviour of this relation is set to "restrict". You might consider changing it to "setNull" or "cascade".)',
+				},
+			},
+		},
+	})
+})
+

package/tests/cases/integration/graphQlRequests/delete/setNullOnDelete.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 { SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../src/tags'
@@ -41,28 +41,21 @@ test('delete author and set null on posts', async () => {
 					parameters: [testUuid(1)],
 					response: { rows: [{ id: testUuid(1) }] },
 				},
-				...sqlDeferred([
-					{
-						sql: SQL`delete from "public"."author"
-            where "id" in (select "root_"."id"
-                           from "public"."author" as "root_"
-                           where "root_"."id" = ?)
-            returning "id"`,
-						parameters: [testUuid(1)],
-						response: { rows: [{ id: testUuid(1) }] },
-					},
-					{
-						sql: SQL`with "newData_" as (select
-                                           ? :: uuid as "author_id",
-                                           "root_"."id"
-                                         from "public"."post" as "root_"
-                                         where "root_"."author_id" in (?)) update "public"."post"
-            set "author_id" = "newData_"."author_id" from "newData_"
-            where "post"."id" = "newData_"."id"`,
-						parameters: [null, testUuid(1)],
-						response: { rowCount: 1 },
-					},
-				]),
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref", true as "allowed" from "public"."post" as "root_" where "author_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`delete from "public"."author" where "id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { },
+				},
 			]),
 		],
 		return: {
@@ -77,3 +70,50 @@ test('delete author and set null on posts', async () => {
 	})
 })
 
+
+test('delete author and set null on posts - declined', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', entity =>
+				entity.manyHasOne('author', relation => relation.target('Author').onDelete(Model.OnDelete.setNull)),
+			)
+			.entity('Author', entity => entity.column('name', column => column.type(Model.ColumnType.String)))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          deleteAuthor(by: {id: "${testUuid(1)}"}) {
+            ok
+            errorMessage
+          }
+        }`,
+		executes: [
+			...failedTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", true as "allowed" from "public"."author" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1), allowed: true }] },
+				},
+				{
+					sql: SQL`select "root_"."id" as "id", "root_"."author_id" as "ref", true as "allowed" from "public"."post" as "root_" where "author_id" in (?)`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(2), ref: testUuid(1), allowed: false }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				deleteAuthor: {
+					ok: false,
+					errorMessage: 'Execution has failed:\n' +
+						'unknown field: ForeignKeyConstraintViolation (Cannot delete 123e4567-e89b-12d3-a456-000000000001 row(s) of entity Author, because it is still referenced from 123e4567-e89b-12d3-a456-000000000002 row(s) of entity Post in relation author. OnDelete behaviour of this relation is set to "set null". This is possibly caused by ACL denial.)',
+				},
+			},
+		},
+	})
+})
+

package/tests/cases/integration/graphQlRequests/insert/manyHasOne/connectOrCreate.test.ts

@@ -0,0 +1,106 @@
+import { test } from 'vitest'
+import { execute, sqlTransaction } from '../../../../../src/test'
+import { SchemaBuilder } from '@contember/schema-definition'
+import { Model } from '@contember/schema'
+import { GQL, SQL } from '../../../../../src/tags'
+import { testUuid } from '@contember/engine-api-tester'
+
+test('create posts and connect/create author (exists, many has one)', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', e =>
+				e.column('publishedAt', c => c.type(Model.ColumnType.DateTime)).manyHasOne('author', r => r.target('Author')),
+			)
+			.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String).unique()))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          createPost(data: {
+            publishedAt: "2018-06-11",
+            author: {connectOrCreate: {connect: {name: "John"}, create: {name: "John"}}}
+          }) {
+			ok
+          }
+        }
+      `,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: `select "root_"."id" from "public"."author" as "root_" where "root_"."name" = ?`,
+					parameters: ['John'],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: `with "root_" as (select ? :: uuid as "id", ? :: timestamptz as "published_at", ? :: uuid as "author_id")
+					insert into  "public"."post" ("id", "published_at", "author_id") select "root_"."id", "root_"."published_at", "root_"."author_id"  from "root_"  returning "id"`,
+					parameters: [testUuid(1), '2018-06-11', testUuid(2)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				createPost: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+
+
+test('create posts and connect/create author (not exists, many has one)', async () => {
+	await execute({
+		schema: new SchemaBuilder()
+			.entity('Post', e =>
+				e.column('publishedAt', c => c.type(Model.ColumnType.DateTime)).manyHasOne('author', r => r.target('Author')),
+			)
+			.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String).unique()))
+			.buildSchema(),
+		query: GQL`
+        mutation {
+          createPost(data: {
+            publishedAt: "2018-06-11",
+            author: {connectOrCreate: {connect: {name: "John"}, create: {name: "John"}}}
+          }) {
+			ok
+          }
+        }
+      `,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: `select "root_"."id" from "public"."author" as "root_" where "root_"."name" = ?`,
+					parameters: ['John'],
+					response: { rows: [] },
+				},
+				{
+					sql: SQL`with "root_" as
+						(select ? :: uuid as "id", ? :: text as "name")
+						insert into "public"."author" ("id", "name")
+						select "root_"."id", "root_"."name"
+            from "root_"
+						returning "id"`,
+					parameters: [testUuid(2), 'John'],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: `with "root_" as (select ? :: uuid as "id", ? :: timestamptz as "published_at", ? :: uuid as "author_id")
+					insert into  "public"."post" ("id", "published_at", "author_id") select "root_"."id", "root_"."published_at", "root_"."author_id"  from "root_"  returning "id"`,
+					parameters: [testUuid(1), '2018-06-11', testUuid(2)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				createPost: {
+					ok: true,
+				},
+			},
+		},
+	})
+})
+
+
+