@contember/schema-migrations 2.1.0-alpha.3 → 2.1.0-alpha.30

package/tests/cases/integration/patchTrigger.test.ts

@@ -0,0 +1,184 @@
+import { Actions } from '@contember/schema'
+import { emptySchema } from '@contember/schema-utils'
+import { describe } from 'bun:test'
+import { testMigrations } from '../../src/tests'
+import { c, createSchema } from '@contember/schema-definition'
+namespace Schema {
+	export class Post {
+		title = c.stringColumn()
+		description = c.stringColumn()
+		category = c.manyHasOne(Category, 'articles')
+	}
+
+	export class Category {
+		name = c.stringColumn()
+		slug = c.stringColumn()
+		articles = c.oneHasMany(Post, 'category')
+	}
+
+
+}
+
+describe('patch trigger', () => testMigrations({
+	original: {
+		...createSchema(Schema),
+		actions: {
+			triggers: {
+				'post_created': {
+					type: 'basic',
+					name: 'post_created',
+					entity: 'Post',
+					create: true,
+					delete: false,
+					update: false,
+					target: 'webhook',
+					selection: [
+						'id',
+						'title',
+						['category', {}, ['name']],
+					],
+				} satisfies Actions.BasicTrigger,
+			},
+			targets: {
+				'webhook': {
+					type: 'webhook',
+					name: 'webhook',
+					url: 'https://example.com/webhook',
+				} satisfies Actions.WebhookTarget,
+			},
+		},
+	},
+	updated: {
+		...createSchema(Schema),
+		actions: {
+			triggers: {
+				'post_created': {
+					type: 'basic',
+					name: 'post_created',
+					entity: 'Post',
+					create: true,
+					delete: false,
+					update: false,
+					target: 'webhook',
+					selection: [
+						'id',
+						'title',
+						'description', // added field
+						['category', {}, ['name', 'slug']], // added sub-field
+					],
+				} satisfies Actions.BasicTrigger,
+			},
+			targets: {
+				'webhook': {
+					type: 'webhook',
+					name: 'webhook',
+					url: 'https://example.com/webhook',
+				} satisfies Actions.WebhookTarget,
+			},
+		},
+	},
+	diff: [
+		{
+			modification: 'patchTrigger',
+			triggerName: 'post_created',
+			patch: [
+				{
+					op: 'replace',
+					path: '/selection/2',
+					value: 'description',
+				},
+				{
+					op: 'add',
+					path: '/selection/-',
+					value: [
+						'category',
+						{},
+						[
+							'name',
+							'slug',
+						],
+					],
+				},
+			],
+		},
+	],
+	sql: '', // no SQL changes for action modifications
+}))
+
+describe('patch watch trigger', () => testMigrations({
+	original: {
+		...createSchema(Schema),
+		actions: {
+			triggers: {
+				'post_watch': {
+					type: 'watch',
+					name: 'post_watch',
+					entity: 'Post',
+					target: 'webhook',
+					watch: [
+						'title',
+						['category', {}, ['name']],
+					],
+				} satisfies Actions.WatchTrigger,
+			},
+			targets: {
+				'webhook': {
+					type: 'webhook',
+					name: 'webhook',
+					url: 'https://example.com/webhook',
+				} satisfies Actions.WebhookTarget,
+			},
+		},
+	},
+	updated: {
+		...createSchema(Schema),
+		actions: {
+			triggers: {
+				'post_watch': {
+					type: 'watch',
+					name: 'post_watch',
+					entity: 'Post',
+					target: 'webhook',
+					watch: [
+						'title',
+						'description', // added field
+						['category', {}, ['name', 'slug']], // added sub-field
+					],
+				} satisfies Actions.WatchTrigger,
+			},
+			targets: {
+				'webhook': {
+					type: 'webhook',
+					name: 'webhook',
+					url: 'https://example.com/webhook',
+				} satisfies Actions.WebhookTarget,
+			},
+		},
+	},
+	diff: [
+		{
+			modification: 'patchTrigger',
+			triggerName: 'post_watch',
+			patch: [
+				{
+					op: 'replace',
+					path: '/watch/1',
+					value: 'description',
+				},
+				{
+					op: 'add',
+					path: '/watch/-',
+					value: [
+						'category',
+						{},
+						[
+							'name',
+							'slug',
+						],
+					],
+				},
+			],
+		},
+	],
+	sql: '', // no SQL changes for action modifications
+}))

package/tests/cases/integration/updateColumnDefinition.test.ts

@@ -251,5 +251,32 @@ describe('change string to array string', () => testMigrations({
 			},
 		}),
 	],
-	sql: SQL`ALTER TABLE "author" ALTER "email" SET DATA TYPE text[] USING ARRAY["email"]::text[];`,
+	sql: SQL`ALTER TABLE "author" ALTER "email" SET DATA TYPE text[] USING CASE WHEN "email" IS NULL THEN NULL ELSE ARRAY["email"]::text[] END;`,
+}))
+
+describe('change string to array string not null', () => testMigrations({
+	original: createSchema({
+		Author: class Author {
+			email = def.stringColumn().notNull()
+		},
+	}),
+	updated: createSchema({
+		Author: class Author {
+			email = def.stringColumn().list().notNull()
+		},
+	}),
+	diff: [
+		updateColumnDefinitionModification.createModification({
+			entityName: 'Author',
+			fieldName: 'email',
+			definition: {
+				type: Model.ColumnType.String,
+				columnType: 'text',
+				nullable: false,
+				list: true,
+			},
+		}),
+	],
+	sql: SQL`ALTER TABLE "author"
+        ALTER "email" SET DATA TYPE text[] USING CASE WHEN "email" IS NULL THEN ARRAY[]::text[] ELSE ARRAY["email"]::text[] END;`,
 }))

package/tests/cases/integration/updateEnum.test.ts

@@ -1,6 +1,6 @@
 import { describe } from 'bun:test'
 import { testMigrations } from '../../src/tests'
-import { SchemaBuilder } from '@contember/schema-definition'
+import { c, createSchema, SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { SQL } from '../../src/tags'
 
@@ -35,3 +35,45 @@ describe('update enum', () => testMigrations({
 	sql: SQL`ALTER DOMAIN "postStatus" DROP CONSTRAINT poststatus_check;
 	ALTER DOMAIN "postStatus" ADD CONSTRAINT poststatus_check CHECK (VALUE IN('publish','draft'));`,
 }))
+
+
+namespace EnumModelOriginal {
+	export const FooStatus = c.createEnum('foo', 'bar')
+	export class Foo {
+
+		status = c.enumColumn(FooStatus).notNull()
+		statusList = c.enumColumn(FooStatus).notNull().list()
+	}
+}
+
+namespace EnumModelUpdated {
+	export const FooStatus = c.createEnum('foo', 'bar', 'baz')
+
+	export class Foo {
+
+		status = c.enumColumn(FooStatus).notNull()
+		statusList = c.enumColumn(FooStatus).notNull().list()
+	}
+}
+
+
+describe('update enum if used in list', () => testMigrations({
+	original: {
+		model: createSchema(EnumModelOriginal).model,
+	},
+	updated: {
+		model: createSchema(EnumModelUpdated).model,
+	},
+	diff: [
+		{
+			modification: 'updateEnum',
+			enumName: 'FooStatus',
+			values: ['foo', 'bar', 'baz'],
+		},
+	],
+	sql: SQL`ALTER DOMAIN "FooStatus" RENAME TO "FooStatus__old"; 
+CREATE DOMAIN "FooStatus" AS text CONSTRAINT "foostatus_check" CHECK (VALUE IN('foo','bar','baz')); 
+ALTER TABLE "foo" ALTER "status" SET DATA TYPE "FooStatus"; 
+ALTER TABLE "foo" ALTER "status_list" SET DATA TYPE "FooStatus"[]; 
+DROP DOMAIN "FooStatus__old";`,
+}))