@contember/schema-migrations 2.1.0-alpha.9 → 2.1.0-beta.2

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

@@ -4,34 +4,34 @@ import { SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { SQL } from '../../src/tags'
 
-describe('remove enum', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Post', e =>
-				e
-					.column('title', c => c.type(Model.ColumnType.String))
-					.column('status', c => c.type(Model.ColumnType.Enum, { enumName: 'postStatus' })),
-			)
-			.enum('postStatus', ['publish', 'draft'])
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'removeField',
-			entityName: 'Post',
-			fieldName: 'status',
+describe('remove enum', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Post', e =>
+					e
+						.column('title', c => c.type(Model.ColumnType.String))
+						.column('status', c => c.type(Model.ColumnType.Enum, { enumName: 'postStatus' })))
+				.enum('postStatus', ['publish', 'draft'])
+				.buildSchema(),
 		},
-		{
-			modification: 'removeEnum',
-			enumName: 'postStatus',
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "post"
+		diff: [
+			{
+				modification: 'removeField',
+				entityName: 'Post',
+				fieldName: 'status',
+			},
+			{
+				modification: 'removeEnum',
+				enumName: 'postStatus',
+			},
+		],
+		sql: SQL`ALTER TABLE "post"
 		DROP "status";
 	DROP DOMAIN "postStatus";`,
-}))
+	}))

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

@@ -7,167 +7,159 @@ import { SchemaDefinition as def } from '@contember/schema-definition'
 import { createDatabaseMetadata } from '@contember/database'
 import { removeUniqueConstraintModification } from '../../../src'
 
-
-describe('remove relation (many has one)', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
-			.entity('Post', e =>
-				e
-					.column('title', c => c.type(Model.ColumnType.String))
-					.manyHasOne('author', r => r.target('Author').onDelete(Model.OnDelete.cascade)),
-			)
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
-			.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'removeField',
-			entityName: 'Post',
-			fieldName: 'author',
+describe('remove relation (many has one)', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
+				.entity('Post', e =>
+					e
+						.column('title', c => c.type(Model.ColumnType.String))
+						.manyHasOne('author', r => r.target('Author').onDelete(Model.OnDelete.cascade)))
+				.buildSchema(),
+		},
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
+				.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "post"
+		diff: [
+			{
+				modification: 'removeField',
+				entityName: 'Post',
+				fieldName: 'author',
+			},
+		],
+		sql: SQL`ALTER TABLE "post"
 		DROP "author_id";`,
-}))
+	}))
 
-describe('remove relation (one has many)', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Post', e => e.oneHasMany('locales', r => r.target('PostLocale').ownerNotNull().ownedBy('post')))
-			.entity('PostLocale', e =>
-				e
-					.unique(['post', 'locale'])
-					.column('title', c => c.type(Model.ColumnType.String))
-					.column('locale', c => c.type(Model.ColumnType.String)),
-			)
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Post', e => e)
-			.entity('PostLocale', e =>
-				e.column('title', c => c.type(Model.ColumnType.String)).column('locale', c => c.type(Model.ColumnType.String)),
-			)
-			.buildSchema(),
-	},
-	diff: [
-		removeUniqueConstraintModification.createModification({
-			entityName: 'PostLocale',
-			unique: {
-				fields: ['post', 'locale'],
-			},
-		}),
-		{
-			modification: 'removeField',
-			entityName: 'PostLocale',
-			fieldName: 'post',
+describe('remove relation (one has many)', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Post', e => e.oneHasMany('locales', r => r.target('PostLocale').ownerNotNull().ownedBy('post')))
+				.entity('PostLocale', e =>
+					e
+						.unique(['post', 'locale'])
+						.column('title', c => c.type(Model.ColumnType.String))
+						.column('locale', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+		},
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Post', e => e)
+				.entity('PostLocale', e => e.column('title', c => c.type(Model.ColumnType.String)).column('locale', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "post_locale" DROP CONSTRAINT "uniq_post_locale_post_id_locale";
+		diff: [
+			removeUniqueConstraintModification.createModification({
+				entityName: 'PostLocale',
+				unique: {
+					fields: ['post', 'locale'],
+				},
+			}),
+			{
+				modification: 'removeField',
+				entityName: 'PostLocale',
+				fieldName: 'post',
+			},
+		],
+		sql: SQL`ALTER TABLE "post_locale" DROP CONSTRAINT "uniq_post_locale_post_id_locale";
 						ALTER TABLE "post_locale" DROP "post_id";`,
-	databaseMetadata: createDatabaseMetadata({
-		foreignKeys: [],
-		indexes: [],
-		uniqueConstraints: [{
-			constraintName: 'uniq_post_locale_post_id_locale',
-			columnNames: ['post_id', 'locale'],
-			tableName: 'post_locale',
-			deferrable: false,
-			deferred: false,
-		}],
-	}),
-}))
+		databaseMetadata: createDatabaseMetadata({
+			foreignKeys: [],
+			indexes: [],
+			uniqueConstraints: [{
+				constraintName: 'uniq_post_locale_post_id_locale',
+				columnNames: ['post_id', 'locale'],
+				tableName: 'post_locale',
+				deferrable: false,
+				deferred: false,
+			}],
+		}),
+	}))
 
-describe('remove relation (many has many)', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Post', e =>
-				e.column('title', c => c.type(Model.ColumnType.String)).manyHasMany('categories', r => r.target('Category')),
-			)
-			.entity('Category', e => e.column('title', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
-			.entity('Category', e => e.column('title', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'removeField',
-			entityName: 'Post',
-			fieldName: 'categories',
+describe('remove relation (many has many)', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)).manyHasMany('categories', r => r.target('Category')))
+				.entity('Category', e => e.column('title', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`DROP TABLE "post_categories";`,
-}))
-
-describe('remove relation (one has one)', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Site', entity =>
-				entity
-					.column('name', c => c.type(Model.ColumnType.String))
-					.oneHasOne('setting', r => r.target('SiteSetting').inversedBy('site')),
-			)
-			.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)))
-			.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'removeField',
-			entityName: 'Site',
-			fieldName: 'setting',
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Post', e => e.column('title', c => c.type(Model.ColumnType.String)))
+				.entity('Category', e => e.column('title', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "site" DROP "setting_id";`,
-}))
+		diff: [
+			{
+				modification: 'removeField',
+				entityName: 'Post',
+				fieldName: 'categories',
+			},
+		],
+		sql: SQL`DROP TABLE "post_categories";`,
+	}))
 
-describe('remove relation inverse side', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Site', entity =>
-				entity
-					.column('name', c => c.type(Model.ColumnType.String))
-					.oneHasOne('setting', r => r.target('SiteSetting').inversedBy('site')),
-			)
-			.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Site', entity =>
-				entity.column('name', c => c.type(Model.ColumnType.String)).oneHasOne('setting', r => r.target('SiteSetting')),
-			)
-			.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'removeField',
-			entityName: 'SiteSetting',
-			fieldName: 'site',
+describe('remove relation (one has one)', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Site', entity =>
+					entity
+						.column('name', c => c.type(Model.ColumnType.String))
+						.oneHasOne('setting', r => r.target('SiteSetting').inversedBy('site')))
+				.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+		},
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)))
+				.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL``,
-}))
+		diff: [
+			{
+				modification: 'removeField',
+				entityName: 'Site',
+				fieldName: 'setting',
+			},
+		],
+		sql: SQL`ALTER TABLE "site" DROP "setting_id";`,
+	}))
 
+describe('remove relation inverse side', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Site', entity =>
+					entity
+						.column('name', c => c.type(Model.ColumnType.String))
+						.oneHasOne('setting', r => r.target('SiteSetting').inversedBy('site')))
+				.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+		},
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)).oneHasOne('setting', r => r.target('SiteSetting')))
+				.entity('SiteSetting', e => e.column('url', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+		},
+		diff: [
+			{
+				modification: 'removeField',
+				entityName: 'SiteSetting',
+				fieldName: 'site',
+			},
+		],
+		sql: SQL``,
+	}))
 
 namespace DropIndexOrigSchema {
-
 	@def.Unique('title', 'author')
 	@def.Index('title', 'author')
 	export class Article {
@@ -193,30 +185,29 @@ namespace DropIndexUpSchema {
 	}
 }
 
-
-describe('test drop index / unique when removing a field', () => testMigrations({
-	original: createSchema(DropIndexOrigSchema),
-	updated: createSchema(DropIndexUpSchema),
-	diff: [{
-		modification: 'removeField',
-		entityName: 'Article',
-		fieldName: 'author',
-	}, {
-		modification: 'createColumn',
-		entityName: 'Article',
-		field: { name: 'author', columnName: 'author', nullable: true, type: 'String', columnType: 'text' },
-	}, {
-		modification: 'createUniqueConstraint',
-		entityName: 'Article',
-		unique: { fields: ['title', 'author'] },
-	}, {
-		modification: 'createIndex',
-		entityName: 'Article',
-		index: { fields: ['title', 'author'] },
-	}],
-	sql: SQL`ALTER TABLE "article" DROP "author_id";
+describe('test drop index / unique when removing a field', () =>
+	testMigrations({
+		original: createSchema(DropIndexOrigSchema),
+		updated: createSchema(DropIndexUpSchema),
+		diff: [{
+			modification: 'removeField',
+			entityName: 'Article',
+			fieldName: 'author',
+		}, {
+			modification: 'createColumn',
+			entityName: 'Article',
+			field: { name: 'author', columnName: 'author', nullable: true, type: 'String', columnType: 'text' },
+		}, {
+			modification: 'createUniqueConstraint',
+			entityName: 'Article',
+			unique: { fields: ['title', 'author'] },
+		}, {
+			modification: 'createIndex',
+			entityName: 'Article',
+			index: { fields: ['title', 'author'] },
+		}],
+		sql: SQL`ALTER TABLE "article" DROP "author_id";
 ALTER TABLE "article" ADD "author" text;
 ALTER TABLE "article" ADD UNIQUE ("title", "author");
 CREATE INDEX ON "article" ("title", "author");`,
-}))
-
+	}))

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

@@ -16,32 +16,33 @@ namespace EventLogDisabled {
 	}
 }
 
-
-describe('event log - disable', () => testMigrations({
-	original: createSchema(EventLogNoConfig),
-	updated: createSchema(EventLogDisabled),
-	diff: [
-		{
-			modification: 'toggleEventLog',
-			entityName: 'Author',
-			enabled: false,
-		},
-	],
-	sql: SQL `DROP TRIGGER "log_event" ON "author";
+describe('event log - disable', () =>
+	testMigrations({
+		original: createSchema(EventLogNoConfig),
+		updated: createSchema(EventLogDisabled),
+		diff: [
+			{
+				modification: 'toggleEventLog',
+				entityName: 'Author',
+				enabled: false,
+			},
+		],
+		sql: SQL`DROP TRIGGER "log_event" ON "author";
 	DROP TRIGGER "log_event_trx" ON "author";`,
-}))
+	}))
 
-describe('event log - enable', () => testMigrations({
-	original: createSchema(EventLogDisabled),
-	updated: createSchema(EventLogNoConfig),
-	diff: [
-		{
-			modification: 'toggleEventLog',
-			entityName: 'Author',
-			enabled: true,
-		},
-	],
-	sql: SQL`CREATE TRIGGER "log_event"
+describe('event log - enable', () =>
+	testMigrations({
+		original: createSchema(EventLogDisabled),
+		updated: createSchema(EventLogNoConfig),
+		diff: [
+			{
+				modification: 'toggleEventLog',
+				entityName: 'Author',
+				enabled: true,
+			},
+		],
+		sql: SQL`CREATE TRIGGER "log_event"
 		AFTER INSERT OR UPDATE OR DELETE
 		ON "author"
 		FOR EACH ROW
@@ -52,4 +53,4 @@ describe('event log - enable', () => testMigrations({
 		DEFERRABLE INITIALLY DEFERRED
 		FOR EACH ROW
 	EXECUTE PROCEDURE "system"."trigger_event_commit"();`,
-}))
+	}))

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

@@ -24,33 +24,35 @@ namespace EventLogDisabled {
 	}
 }
 
-describe('event log junction - disable', () => testMigrations({
-	original: createSchema(EventLogNoConfig),
-	updated: createSchema(EventLogDisabled),
-	diff: [
-		{
-			modification: 'toggleJunctionEventLog',
-			entityName: 'Author',
-			fieldName: 'tags',
-			enabled: false,
-		},
-	],
-	sql: SQL `DROP TRIGGER "log_event" ON "author_tags";
+describe('event log junction - disable', () =>
+	testMigrations({
+		original: createSchema(EventLogNoConfig),
+		updated: createSchema(EventLogDisabled),
+		diff: [
+			{
+				modification: 'toggleJunctionEventLog',
+				entityName: 'Author',
+				fieldName: 'tags',
+				enabled: false,
+			},
+		],
+		sql: SQL`DROP TRIGGER "log_event" ON "author_tags";
 	DROP TRIGGER "log_event_trx" ON "author_tags";`,
-}))
+	}))
 
-describe('event log junction - enable', () => testMigrations({
-	original: createSchema(EventLogDisabled),
-	updated: createSchema(EventLogNoConfig),
-	diff: [
-		{
-			modification: 'toggleJunctionEventLog',
-			entityName: 'Author',
-			fieldName: 'tags',
-			enabled: true,
-		},
-	],
-	sql: SQL`CREATE TRIGGER "log_event"
+describe('event log junction - enable', () =>
+	testMigrations({
+		original: createSchema(EventLogDisabled),
+		updated: createSchema(EventLogNoConfig),
+		diff: [
+			{
+				modification: 'toggleJunctionEventLog',
+				entityName: 'Author',
+				fieldName: 'tags',
+				enabled: true,
+			},
+		],
+		sql: SQL`CREATE TRIGGER "log_event"
 		AFTER INSERT OR UPDATE OR DELETE
 		ON "author_tags"
 		FOR EACH ROW
@@ -61,4 +63,4 @@ describe('event log junction - enable', () => testMigrations({
 		DEFERRABLE INITIALLY DEFERRED
 		FOR EACH ROW
 	EXECUTE PROCEDURE "system"."trigger_event_commit"();`,
-}))
+	}))