@contember/engine-content-api 1.1.0-rc.3 → 1.2.0-alpha.3

package/tests/cases/integration/graphQlRequests/query/acl/fieldRestrictedByLocale.test.ts

@@ -45,7 +45,12 @@ test('querying id and title', async () => {
 		schema: schema,
 		permissions: permissions,
 		variables: {
-			localeVariable: ['cs'],
+			localeVariable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['cs'] }],
+			},
 		},
 		query: GQL`
         query {
@@ -121,7 +126,12 @@ test('querying with extra where', async () => {
 		schema: schema,
 		permissions: permissions,
 		variables: {
-			localeVariable: ['cs'],
+			localeVariable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['cs'] }],
+			},
 		},
 		query: GQL`
         query {
@@ -167,7 +177,12 @@ test('querying only id', async () => {
 		schema: schema,
 		permissions: permissions,
 		variables: {
-			localeVariable: ['cs'],
+			localeVariable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['cs'] }],
+			},
 		},
 		query: GQL`
         query {
@@ -239,7 +254,12 @@ test('querying on nested field', async () => {
 		schema: schema,
 		permissions: permissions,
 		variables: {
-			localeVariable: ['cs'],
+			localeVariable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['cs'] }],
+			},
 		},
 		query: GQL`
         query {

package/tests/cases/integration/graphQlRequests/query/acl/nestedPredicate.test.ts

@@ -58,7 +58,12 @@ test('querying post + author name', async () => {
 		schema: schema,
 		permissions: permissions,
 		variables: {
-			countryVariable: ['Czechia'],
+			countryVariable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['Czechia'] }],
+			},
 		},
 		query: GQL`
         query {

package/tests/cases/integration/graphQlRequests/update/acl/aclUpdate.test.ts

@@ -1,7 +1,7 @@
 import { test } from 'vitest'
 import { execute, failedTransaction, sqlTransaction } from '../../../../../src/test'
 import { SchemaBuilder } from '@contember/schema-definition'
-import { Model } from '@contember/schema'
+import { Acl, Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
 
@@ -35,7 +35,12 @@ test('update name', async () => {
 			},
 		},
 		variables: {
-			name_variable: ['John', 'Jack'],
+			name_variable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['John', 'Jack'] }],
+			},
 		},
 		executes: [
 			...sqlTransaction([
@@ -98,7 +103,12 @@ test('update name - denied', async () => {
 			},
 		},
 		variables: {
-			name_variable: ['John', 'Jack'],
+			name_variable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['John', 'Jack'] }],
+			},
 		},
 		executes: [
 			...failedTransaction([
@@ -176,8 +186,18 @@ test('update m:n', async () => {
 			},
 		},
 		variables: {
-			post_name_variable: ['Lorem ipsum', 'Dolor sit'],
-			category_name_variable: ['foo', 'bar'],
+			post_name_variable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['Lorem ipsum', 'Dolor sit'] }],
+			},
+			category_name_variable: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ in: ['foo', 'bar'] }],
+			},
 		},
 		query: GQL`mutation  {
           updatePost(by: {id: "${testUuid(1)}"}, data: {categories: [

package/tests/cases/integration/graphQlSchema/graphQlSchemaBuilder.test.ts

@@ -30,20 +30,20 @@ const testSchema = async (test: Test) => {
 	const graphQlSchemaBuilder = schemaFactory.create(schemaWithAcl, authorizator)
 	const graphQlSchema = graphQlSchemaBuilder.build()
 
-	const result = await graphql(
-		graphQlSchema,
-		`
+	const result = await graphql({
+		schema: graphQlSchema,
+		source: `
 			{
 				_info {
 					description
 				}
 			}
 		`,
-	)
+	})
 	const errors = (result.errors || []).map(it => it.message)
 	assert.deepEqual(errors, [])
 
-	const textSchema = printSchema(graphQlSchema)
+	const textSchema = printSchema(graphQlSchema) + '\n'
 
 	const filename = path.join(__dirname, test.graphQlSchemaFile)
 	let expectedSchema: string

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

@@ -5,13 +5,14 @@ import { testUuid } from '@contember/engine-api-tester'
 
 describe('create acl variables', () => {
 
+	const definition: Acl.EntityVariable = { type: Acl.VariableType.entity, entityName: 'Author' }
 	it('prefix from memberships', () => {
 		assert.deepStrictEqual(createAclVariables({
 			roles: {
 				author: {
 					stages: '*',
 					variables: {
-						authorID: { type: Acl.VariableType.entity, entityName: 'Author' },
+						authorID: definition,
 					},
 					entities: {
 					},
@@ -22,7 +23,7 @@ describe('create acl variables', () => {
 			personId: null,
 			memberships: [{ role: 'author', variables: [{ name: 'authorID', values: [testUuid(2)] }] }],
 		}), {
-			author__authorID: [testUuid(2)],
+			author__authorID: { definition, value: [testUuid(2)] },
 		})
 	})
 
@@ -33,7 +34,7 @@ describe('create acl variables', () => {
 				author: {
 					stages: '*',
 					variables: {
-						authorID: { type: Acl.VariableType.entity, entityName: 'Author' },
+						authorID: definition,
 					},
 					entities: {},
 				},
@@ -43,18 +44,19 @@ describe('create acl variables', () => {
 			personId: null,
 			memberships: [{ role: 'author', variables: [] }],
 		}), {
-			author__authorID: [],
+			author__authorID: { definition, value: [] },
 		})
 	})
 
 
 	it('predefined', () => {
+		const definition: Acl.PredefinedVariable = { type: Acl.VariableType.predefined, value: 'identityID' }
 		assert.deepStrictEqual(createAclVariables({
 			roles: {
 				author: {
 					stages: '*',
 					variables: {
-						authorIdentity: { type: Acl.VariableType.predefined, value: 'identityID' },
+						authorIdentity: definition,
 					},
 					entities: {},
 				},
@@ -64,7 +66,7 @@ describe('create acl variables', () => {
 			personId: null,
 			memberships: [{ role: 'author', variables: [] }],
 		}), {
-			author__authorIdentity: [testUuid(1)],
+			author__authorIdentity: { definition, value: [testUuid(1)] },
 		})
 	})
 })

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

@@ -44,10 +44,18 @@ const permissions: Acl.Permissions = {
 
 describe('Predicates injector', () => {
 
+	const variables: Acl.VariablesMap = {
+		localeVariable: {
+			definition: {
+				type: Acl.VariableType.condition,
+			},
+			value: [{ in: ['cs'] }],
+		},
+	}
 	it('injects predicate', () => {
 		const injector = new PredicatesInjector(
 			schema,
-			new PredicateFactory(permissions, new VariableInjector(schema, { localeVariable: ['cs'] })),
+			new PredicateFactory(permissions, new VariableInjector(schema, variables)),
 		)
 		const result = injector.inject(schema.entities['PostLocale'], {})
 
@@ -59,7 +67,7 @@ describe('Predicates injector', () => {
 	it('merges predicate with explicit where', () => {
 		const injector = new PredicatesInjector(
 			schema,
-			new PredicateFactory(permissions, new VariableInjector(schema, { localeVariable: ['cs'] })),
+			new PredicateFactory(permissions, new VariableInjector(schema, variables)),
 		)
 		const result = injector.inject(schema.entities['PostLocale'], { id: { in: [1, 2] } })
 
@@ -71,9 +79,7 @@ describe('Predicates injector', () => {
 							id: { in: [1, 2] },
 						},
 						{
-							locale: {
-								in: ['cs'],
-							},
+							locale: { in: ['cs'] },
 						},
 					],
 				},
@@ -87,7 +93,7 @@ describe('Predicates injector', () => {
 	it('injects predicate to where', () => {
 		const injector = new PredicatesInjector(
 			schema,
-			new PredicateFactory(permissions, new VariableInjector(schema, { localeVariable: ['cs'] })),
+			new PredicateFactory(permissions, new VariableInjector(schema, variables)),
 		)
 
 		const result = injector.inject(schema.entities['PostLocale'], { title: { eq: 'abc' } })

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

@@ -1,7 +1,7 @@
 import { VariableInjector } from '../../../src/acl'
 import { SchemaBuilder } from '@contember/schema-definition'
-import { Model } from '@contember/schema'
-import { describe, it, assert } from 'vitest'
+import { Acl, Model } from '@contember/schema'
+import { assert, describe, it } from 'vitest'
 
 describe('Variable injector', () => {
 
@@ -17,9 +17,21 @@ describe('Variable injector', () => {
 			)
 			.buildSchema()
 
+		const ids = ['91528a5c-839b-4114-9952-d661914a607f', '1e2f3937-a0b1-4105-8aae-6da8ac9fc146']
 		const injector = new VariableInjector(schema, {
-			site: [1, 2],
-			locale: 'cs',
+			site: {
+				definition: {
+					type: Acl.VariableType.entity,
+					entityName: 'Site',
+				},
+				value: ids,
+			},
+			locale: {
+				definition: {
+					type: Acl.VariableType.condition,
+				},
+				value: [{ eq: 'cs' }],
+			},
 		})
 		const result = injector.inject(schema.entities['PostLocale'], {
 			or: [
@@ -53,7 +65,7 @@ describe('Variable injector', () => {
 					and: [
 						{
 							post: {
-								site: { id: { in: [1, 2] } },
+								site: { id: { in: ids } },
 							},
 						},
 						{

package/tests/src/testGraphql.ts

@@ -10,10 +10,17 @@ export interface Test {
 }
 
 export const executeGraphQlTest = async (test: Test) => {
-	const rawResponse = await graphql(test.schema, test.query, null, test.context, test.queryVariables)
+	const rawResponse = await graphql({
+		schema: test.schema,
+		source: test.query,
+		contextValue: test.context,
+		variableValues: test.queryVariables,
+	})
 	const response = JSON.parse(JSON.stringify(rawResponse))
 	if ('errors' in rawResponse) {
-		console.error((rawResponse.errors as any)[0])
+		if (rawResponse.errors?.length === 1 && 'originalError' in (rawResponse.errors as any)[0]) {
+			throw (rawResponse.errors as any)[0].originalError
+		}
 		rawResponse.errors = (rawResponse.errors as any).map(({ message }: any) => ({ message }))
 	}
 	assert.deepStrictEqual(response, test.return)

package/jasmine.json

@@ -1,7 +0,0 @@
-{
-	"spec_files": [
-		"dist/tests/cases/**/*.js"
-	],
-	"stopSpecOnExpectationFailure": false,
-	"random": false
-}