@contember/engine-content-api 1.2.0-alpha.12 → 1.2.0-alpha.13

package/tests/cases/integration/graphQlRequests/query/graphqlFeatures/alias.test.ts

@@ -77,15 +77,13 @@ test('aliases', async () => {
         }`,
 		executes: [
 			{
-				sql: SQL`select
-                     "root_"."id" as "root_idx",
-                     "root_"."name" as "root_name1",
-                     "root_"."name" as "root_name2",
-                     true as "root_meta_name1_readable1",
-                     true as "root_meta_name1_readable2",
-                     "root_"."name" = ? as "root_meta_name1_updatable1",
-                     "root_"."id" as "root_id"
-                   from "public"."author" as "root_"`,
+				sql: SQL`SELECT
+							 "root_"."id" AS "root_idx",
+							 "root_"."name" AS "root_name1",
+							 "root_"."name" AS "root_name2",
+							 "root_"."name" = ? AS "root___predicate_name_eq",
+							 "root_"."id" AS "root_id"
+						 FROM "public"."author" AS "root_"`,
 				parameters: ['John'],
 				response: {
 					rows: [
@@ -94,9 +92,7 @@ test('aliases', async () => {
 							root_idx: testUuid(1),
 							root_name1: 'John',
 							root_name2: 'John',
-							root_meta_name1_readable1: true,
-							root_meta_name1_readable2: true,
-							root_meta_name1_updatable1: false,
+							root___predicate_name_eq: true,
 						},
 					],
 				},
@@ -111,7 +107,7 @@ test('aliases', async () => {
 							name1: {
 								readable1: true,
 								readable2: true,
-								updatable1: false,
+								updatable1: true,
 							},
 						},
 						name1: 'John',

package/tests/cases/unit/predicatesInjector.test.ts

@@ -5,6 +5,8 @@ import { describe, it, assert } from 'vitest'
 import { PermissionsFactory } from '@contember/engine-system-api/dist/src/model'
 import { WhereOptimizer } from '../../../src/mapper/select/optimizer/WhereOptimizer'
 import { ConditionOptimizer } from '../../../src/mapper/select/optimizer/ConditionOptimizer'
+import { acceptFieldVisitor } from '@contember/schema-utils'
+import { testUuid } from '@contember/engine-api-tester'
 
 
 
@@ -157,14 +159,14 @@ namespace DeepFilterModel {
 
 
 describe('predicates injector elimination', () => {
-	it('eliminates predicates in where', () => {
-		const schema = createSchema(DeepFilterModel)
-		const permissions = new PermissionFactory(schema.model).create(schema.acl, ['reader'])
+	const schema = createSchema(DeepFilterModel)
+	const permissions = new PermissionFactory(schema.model).create(schema.acl, ['reader'])
 
-		const injector = new PredicatesInjector(
-			schema.model,
-			new PredicateFactory(permissions, schema.model, new VariableInjector(schema.model, {})),
-		)
+	const injector = new PredicatesInjector(
+		schema.model,
+		new PredicateFactory(permissions, schema.model, new VariableInjector(schema.model, {})),
+	)
+	it('eliminates predicates in where', () => {
 		const injected = injector.inject(schema.model.entities.Article, {
 			coverPhoto: { image: { tags: { label: { eq: 'foo' } } } },
 		})
@@ -173,4 +175,32 @@ describe('predicates injector elimination', () => {
 
 		assert.deepStrictEqual(result, { and: [{ coverPhoto: { image: { tags: { label: { eq: 'foo' } } } } }, { isPublished: { eq: true } }] })
 	})
+
+	it('eliminate back referencing predicate', () => {
+		const relation = acceptFieldVisitor(schema.model, schema.model.entities.Article, 'coverPhoto', {
+			visitColumn: () => {
+				throw new Error()
+			},
+			visitRelation: ctx => ctx,
+		})
+		const injected = injector.inject(schema.model.entities.ImageUse, {
+			articles: { id: { in: [testUuid(1)] } },
+		}, relation)
+
+		assert.deepStrictEqual(injected, {
+			and: [
+				{
+					articles: {
+						and: [
+							{ id: { in: [testUuid(1)] } },
+							{ id: { always: true } },
+						],
+					},
+				},
+				{
+					articles: { id: { always: true } },
+				},
+			],
+		})
+	})
 })

package/tests/cases/unit/whereOptimizer.test.ts

@@ -200,14 +200,15 @@ describe('where optimized', () => {
 	describe('remove id predicates', () => {
 
 		it('remove ID predicate when traversing from a relation - always', () => {
-			assert.deepStrictEqual(whereOptimizer.optimize({
-				image: { id: { isNull: false } },
-			}, model.entities.Author, [acceptFieldVisitor(model, 'Image', 'authors', {
+			const relationPath = [acceptFieldVisitor(model, 'Image', 'authors', {
 				visitColumn: () => {
 					throw new Error()
 				},
 				visitRelation: context => context,
-			})]), {
+			})]
+			assert.deepStrictEqual(whereOptimizer.optimize({
+				image: { id: { isNull: false } },
+			}, model.entities.Author, { relationPath }), {
 				id: { always: true },
 			})
 		})
@@ -216,12 +217,12 @@ describe('where optimized', () => {
 		it('remove ID predicate when traversing from a relation - never', () => {
 			assert.deepStrictEqual(whereOptimizer.optimize({
 				image: { id: { isNull: true } },
-			}, model.entities.Author, [acceptFieldVisitor(model, 'Image', 'authors', {
+			}, model.entities.Author, { relationPath: [acceptFieldVisitor(model, 'Image', 'authors', {
 				visitColumn: () => {
 					throw new Error()
 				},
 				visitRelation: context => context,
-			})]), {
+			})] }), {
 				id: { never: true },
 			})
 		})
@@ -229,28 +230,27 @@ describe('where optimized', () => {
 		it('remove ID predicate when traversing in where', () => {
 			assert.deepStrictEqual(whereOptimizer.optimize({
 				authors: { image: { id: { isNull: false } } },
-			}, model.entities.Image, []), {
+			}, model.entities.Image), {
 				id: { always: true },
 			})
 		})
 
 		it('do not remove ID predicate when traversing from a has-many relation', () => {
-			assert.deepStrictEqual(whereOptimizer.optimize({
-				authors: { id: { isNull: false } },
-			}, model.entities.Image, [acceptFieldVisitor(model, 'Author', 'image', {
+			const relationPath = [acceptFieldVisitor(model, 'Author', 'image', {
 				visitColumn: () => {
 					throw new Error()
 				},
 				visitRelation: context => context,
-			})]), {
+			})]
+			assert.deepStrictEqual(whereOptimizer.optimize({
+				authors: { id: { isNull: false } },
+			}, model.entities.Image, { relationPath }), {
 				authors: { id: { isNull: false } },
 			})
 		})
 
 		it('do not remove ID predicate when traversing from a nested has-many relation', () => {
-			assert.deepStrictEqual(whereOptimizer.optimize({
-				articles: { author: { id: { isNull: false } } },
-			}, model.entities.Image, [
+			const relationPath = [
 				acceptFieldVisitor(model, 'Author', 'articles', {
 					visitColumn: () => {
 						throw new Error()
@@ -263,7 +263,10 @@ describe('where optimized', () => {
 					},
 					visitRelation: context => context,
 				}),
-			]), {
+			]
+			assert.deepStrictEqual(whereOptimizer.optimize({
+				articles: { author: { id: { isNull: false } } },
+			}, model.entities.Image, { relationPath }), {
 				articles: { author: { id: { isNull: false } } },
 			})
 		})
@@ -271,7 +274,7 @@ describe('where optimized', () => {
 		it('do not remove ID predicate when traversing in where over has-many relation', () => {
 			assert.deepStrictEqual(whereOptimizer.optimize({
 				image: { authors: { id: { isNull: false } } },
-			}, model.entities.Author, []), {
+			}, model.entities.Author), {
 				image: { authors: { id: { isNull: false } } },
 			})
 		})
@@ -280,7 +283,7 @@ describe('where optimized', () => {
 		it('do not remove ID predicate when traversing in where over has-many relation - nested', () => {
 			assert.deepStrictEqual(whereOptimizer.optimize({
 				image: { authors: { image: { id: { isNull: false } } } },
-			}, model.entities.Author, []), {
+			}, model.entities.Author), {
 				image: { authors: { image: { id: { isNull: false } } } },
 			})
 		})