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

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

@@ -1,42 +1,40 @@
 import { describe } from 'bun:test'
 import { testMigrations } from '../../src/tests'
-import { SchemaBuilder, SchemaDefinition as def, createSchema } from '@contember/schema-definition'
+import { createSchema, SchemaBuilder, SchemaDefinition as def } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { SQL } from '../../src/tags'
 import { updateColumnDefinitionModification } from '../../../src'
 
-describe('update column definition', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Author', e =>
-				e.column('name', c => c.type(Model.ColumnType.String)).column('registeredAt', c => c.type(Model.ColumnType.Date)),
-			)
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Author', e =>
-				e
-					.column('name', c => c.type(Model.ColumnType.String))
-					.column('registeredAt', c => c.type(Model.ColumnType.DateTime)),
-			)
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'updateColumnDefinition',
-			entityName: 'Author',
-			fieldName: 'registeredAt',
-			definition: {
-				type: Model.ColumnType.DateTime,
-				columnType: 'timestamptz',
-				nullable: true,
-			},
+describe('update column definition', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)).column('registeredAt', c => c.type(Model.ColumnType.Date)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "author"
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Author', e =>
+					e
+						.column('name', c => c.type(Model.ColumnType.String))
+						.column('registeredAt', c => c.type(Model.ColumnType.DateTime)))
+				.buildSchema(),
+		},
+		diff: [
+			{
+				modification: 'updateColumnDefinition',
+				entityName: 'Author',
+				fieldName: 'registeredAt',
+				definition: {
+					type: Model.ColumnType.DateTime,
+					columnType: 'timestamptz',
+					nullable: true,
+				},
+			},
+		],
+		sql: SQL`ALTER TABLE "author"
 		ALTER "registered_at" SET DATA TYPE timestamptz USING "registered_at"::timestamptz;`,
-}))
+	}))
 
 namespace SeqOrig {
 	export class Author {
@@ -48,31 +46,31 @@ namespace SeqUpdated {
 		idSeq = def.intColumn().notNull().sequence({ start: 10 })
 	}
 }
-describe('add sequence to existing column', () => testMigrations({
-	original: createSchema(SeqOrig),
-	updated: createSchema(SeqUpdated),
-	diff: [
-		updateColumnDefinitionModification.createModification({
-			entityName: 'Author',
-			fieldName: 'idSeq',
-			definition: {
-				type: Model.ColumnType.Int,
-				columnType: 'integer',
-				nullable: false,
-				sequence: {
-					precedence: 'BY DEFAULT',
-					start: 10,
+describe('add sequence to existing column', () =>
+	testMigrations({
+		original: createSchema(SeqOrig),
+		updated: createSchema(SeqUpdated),
+		diff: [
+			updateColumnDefinitionModification.createModification({
+				entityName: 'Author',
+				fieldName: 'idSeq',
+				definition: {
+					type: Model.ColumnType.Int,
+					columnType: 'integer',
+					nullable: false,
+					sequence: {
+						precedence: 'BY DEFAULT',
+						start: 10,
+					},
 				},
-			},
-		}),
-	],
-	sql: SQL`
+			}),
+		],
+		sql: SQL`
 ALTER TABLE "author"
 		ALTER "id_seq" SET DATA TYPE integer USING COALESCE("id_seq"::integer, nextval(PG_GET_SERIAL_SEQUENCE($pga$author$pga$, $pga$id_seq$pga$))),
 		ALTER "id_seq" SET NOT NULL,
 		ALTER "id_seq" ADD GENERATED BY DEFAULT AS IDENTITY (START WITH 10);`,
-}))
-
+	}))
 
 namespace ViewColDepOrig {
 	export class Author {
@@ -83,7 +81,6 @@ namespace ViewColDepOrig {
 		dependencies: [Author],
 	})
 	export class AuthorMetaX {
-
 	}
 
 	@def.View('SELECT * FROM author_meta_x', {
@@ -102,7 +99,6 @@ namespace ViewColDepUpdated {
 		dependencies: [Author],
 	})
 	export class AuthorMetaX {
-
 	}
 
 	@def.View('SELECT * FROM author_meta_x', {
@@ -112,68 +108,68 @@ namespace ViewColDepUpdated {
 	}
 }
 
-describe('recreate view dependent on entity after changing column type', () => testMigrations({
-	original: createSchema(ViewColDepOrig),
-	updated: createSchema(ViewColDepUpdated),
-	diff: [{ modification: 'removeEntity', entityName: 'AuthorMetaY' }, {
-		modification: 'removeEntity',
-		entityName: 'AuthorMetaX',
-	}, {
-		modification: 'updateColumnDefinition',
-		entityName: 'Author',
-		fieldName: 'val',
-		definition: { nullable: true, type: 'String', columnType: 'text' },
-	}, {
-		modification: 'createView',
-		entity: {
-			name: 'AuthorMetaX',
-			primary: 'id',
-			primaryColumn: 'id',
-			unique: [],
-			indexes: [],
-			fields: {
-				id: {
-					name: 'id',
-					columnName: 'id',
-					nullable: false,
-					type: 'Uuid',
-					columnType: 'uuid',
+describe('recreate view dependent on entity after changing column type', () =>
+	testMigrations({
+		original: createSchema(ViewColDepOrig),
+		updated: createSchema(ViewColDepUpdated),
+		diff: [{ modification: 'removeEntity', entityName: 'AuthorMetaY' }, {
+			modification: 'removeEntity',
+			entityName: 'AuthorMetaX',
+		}, {
+			modification: 'updateColumnDefinition',
+			entityName: 'Author',
+			fieldName: 'val',
+			definition: { nullable: true, type: 'String', columnType: 'text' },
+		}, {
+			modification: 'createView',
+			entity: {
+				name: 'AuthorMetaX',
+				primary: 'id',
+				primaryColumn: 'id',
+				unique: [],
+				indexes: [],
+				fields: {
+					id: {
+						name: 'id',
+						columnName: 'id',
+						nullable: false,
+						type: 'Uuid',
+						columnType: 'uuid',
+					},
 				},
+				tableName: 'author_meta_x',
+				eventLog: { enabled: true },
+				view: { sql: 'SELECT * FROM author', dependencies: ['Author'] },
 			},
-			tableName: 'author_meta_x',
-			eventLog: { enabled: true },
-			view: { sql: 'SELECT * FROM author', dependencies: ['Author'] },
-		},
-	}, {
-		modification: 'createView',
-		entity: {
-			name: 'AuthorMetaY',
-			primary: 'id',
-			primaryColumn: 'id',
-			unique: [],
-			indexes: [],
-			fields: {
-				id: {
-					name: 'id',
-					columnName: 'id',
-					nullable: false,
-					type: 'Uuid',
-					columnType: 'uuid',
+		}, {
+			modification: 'createView',
+			entity: {
+				name: 'AuthorMetaY',
+				primary: 'id',
+				primaryColumn: 'id',
+				unique: [],
+				indexes: [],
+				fields: {
+					id: {
+						name: 'id',
+						columnName: 'id',
+						nullable: false,
+						type: 'Uuid',
+						columnType: 'uuid',
+					},
 				},
+				tableName: 'author_meta_y',
+				eventLog: { enabled: true },
+				view: { sql: 'SELECT * FROM author_meta_x', dependencies: ['AuthorMetaX'] },
 			},
-			tableName: 'author_meta_y',
-			eventLog: { enabled: true },
-			view: { sql: 'SELECT * FROM author_meta_x', dependencies: ['AuthorMetaX'] },
-		},
-	}],
-	sql: SQL`DROP VIEW "author_meta_y";
+		}],
+		sql: SQL`DROP VIEW "author_meta_y";
 DROP VIEW "author_meta_x";
 ALTER TABLE "author"
   ALTER "val" SET DATA TYPE text USING "val"::text;
 CREATE VIEW "author_meta_x" AS SELECT * FROM author;
 CREATE VIEW "author_meta_y" AS SELECT * FROM author_meta_x;`,
-}))
-
+	}))
 
 namespace NotNullOrig {
 	export class Author {
@@ -185,71 +181,101 @@ namespace NotNullUpdated {
 		name = def.stringColumn().notNull()
 	}
 }
-describe('set not null and fill', () => testMigrations({
-	original: createSchema(NotNullOrig),
-	updated: createSchema(NotNullUpdated),
-	noDiff: true,
-	diff: [
-		updateColumnDefinitionModification.createModification({
-			entityName: 'Author',
-			fieldName: 'name',
-			definition: {
-				type: Model.ColumnType.String,
-				columnType: 'text',
-				nullable: false,
-			},
-			fillValue: 'unnamed',
-		}),
-	],
-	sql: SQL`
+describe('set not null and fill', () =>
+	testMigrations({
+		original: createSchema(NotNullOrig),
+		updated: createSchema(NotNullUpdated),
+		noDiff: true,
+		diff: [
+			updateColumnDefinitionModification.createModification({
+				entityName: 'Author',
+				fieldName: 'name',
+				definition: {
+					type: Model.ColumnType.String,
+					columnType: 'text',
+					nullable: false,
+				},
+				fillValue: 'unnamed',
+			}),
+		],
+		sql: SQL`
 UPDATE "author" SET "name" = $pga$unnamed$pga$ WHERE "name" IS NULL;
 SET CONSTRAINTS ALL IMMEDIATE; SET CONSTRAINTS ALL DEFERRED;
 ALTER TABLE "author" ALTER "name" SET NOT NULL;`,
-}))
+	}))
 
-describe('set not null and fill with "using"', () => testMigrations({
-	original: createSchema(NotNullOrig),
-	updated: createSchema(NotNullUpdated),
-	noDiff: true,
-	diff: [
-		updateColumnDefinitionModification.createModification({
-			entityName: 'Author',
-			fieldName: 'name',
-			definition: {
-				type: Model.ColumnType.String,
-				columnType: 'text',
-				nullable: false,
+describe('set not null and fill with "using"', () =>
+	testMigrations({
+		original: createSchema(NotNullOrig),
+		updated: createSchema(NotNullUpdated),
+		noDiff: true,
+		diff: [
+			updateColumnDefinitionModification.createModification({
+				entityName: 'Author',
+				fieldName: 'name',
+				definition: {
+					type: Model.ColumnType.String,
+					columnType: 'text',
+					nullable: false,
+				},
+				fillValue: 'unnamed',
+				valueMigrationStrategy: 'using',
+			}),
+		],
+		sql: SQL` ALTER TABLE "author" ALTER "name" SET DATA TYPE text USING COALESCE("name"::text, $pga$unnamed$pga$), ALTER "name" SET NOT NULL;`,
+	}))
+
+describe('change string to array string', () =>
+	testMigrations({
+		original: createSchema({
+			Author: class Author {
+				email = def.stringColumn()
 			},
-			fillValue: 'unnamed',
-			valueMigrationStrategy: 'using',
 		}),
-	],
-	sql: SQL` ALTER TABLE "author" ALTER "name" SET DATA TYPE text USING COALESCE("name"::text, $pga$unnamed$pga$), ALTER "name" SET NOT NULL;`,
-}))
-
+		updated: createSchema({
+			Author: class Author {
+				email = def.stringColumn().list()
+			},
+		}),
+		diff: [
+			updateColumnDefinitionModification.createModification({
+				entityName: 'Author',
+				fieldName: 'email',
+				definition: {
+					type: Model.ColumnType.String,
+					columnType: 'text',
+					nullable: true,
+					list: true,
+				},
+			}),
+		],
+		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', () => testMigrations({
-	original: createSchema({
-		Author: class Author {
-			email = def.stringColumn()
-		},
-	}),
-	updated: createSchema({
-		Author: class Author {
-			email = def.stringColumn().list()
-		},
-	}),
-	diff: [
-		updateColumnDefinitionModification.createModification({
-			entityName: 'Author',
-			fieldName: 'email',
-			definition: {
-				type: Model.ColumnType.String,
-				columnType: 'text',
-				nullable: true,
-				list: true,
+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()
 			},
 		}),
-	],
-	sql: SQL`ALTER TABLE "author" ALTER "email" SET DATA TYPE text[] USING ARRAY["email"]::text[];`,
-}))
+		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/updateEntityName.test.ts

@@ -1,98 +1,102 @@
 import { describe } from 'bun:test'
 import { testMigrations } from '../../src/tests'
-import { SchemaBuilder, c, createSchema } from '@contember/schema-definition'
+import { c, createSchema, SchemaBuilder } from '@contember/schema-definition'
 import { Model } from '@contember/schema'
 import { SQL } from '../../src/tags'
 
-describe('rename entity without renaming a table', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Author', e => e.tableName('user').column('name', c => c.type(Model.ColumnType.String)))
-			.entity('Post', e => e.column('title').manyHasOne('author', r => r.target('Author')))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('User', e => e.column('name', c => c.type(Model.ColumnType.String)))
-			.entity('Post', e => e.column('title').manyHasOne('author', r => r.target('User')))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'updateEntityName',
-			entityName: 'Author',
-			newEntityName: 'User',
+describe('rename entity without renaming a table', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.tableName('user').column('name', c => c.type(Model.ColumnType.String)))
+				.entity('Post', e => e.column('title').manyHasOne('author', r => r.target('Author')))
+				.buildSchema(),
 		},
-	],
-	sql: SQL``,
-	noDiff: true,
-}))
+		updated: {
+			model: new SchemaBuilder()
+				.entity('User', e => e.column('name', c => c.type(Model.ColumnType.String)))
+				.entity('Post', e => e.column('title').manyHasOne('author', r => r.target('User')))
+				.buildSchema(),
+		},
+		diff: [
+			{
+				modification: 'updateEntityName',
+				entityName: 'Author',
+				newEntityName: 'User',
+			},
+		],
+		sql: SQL``,
+		noDiff: true,
+	}))
 
-describe('rename entity and table', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('User', e => e.column('name', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'updateEntityName',
-			entityName: 'Author',
-			newEntityName: 'User',
-			tableName: 'user',
+describe('rename entity and table', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.column('name', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+		},
+		updated: {
+			model: new SchemaBuilder()
+				.entity('User', e => e.column('name', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
 		},
-	],
-	sql: SQL`ALTER TABLE "author"
+		diff: [
+			{
+				modification: 'updateEntityName',
+				entityName: 'Author',
+				newEntityName: 'User',
+				tableName: 'user',
+			},
+		],
+		sql: SQL`ALTER TABLE "author"
 		RENAME TO "user";`,
-	noDiff: true,
-}))
-
+		noDiff: true,
+	}))
 
-describe('rename entity with one-has-one (constraint)', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Author', e => e.oneHasOne('content', r => r.target('Content')))
-			.entity('Content', e => e.column('foo'))
-			.buildSchema(),
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('User', e => e.tableName('author').oneHasOne('content', r => r.target('Content')))
-			.entity('Content', e => e.column('foo'))
-			.buildSchema(),
-	},
-	diff: [
-		{
-			modification: 'updateEntityName',
-			entityName: 'Author',
-			newEntityName: 'User',
+describe('rename entity with one-has-one (constraint)', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Author', e => e.oneHasOne('content', r => r.target('Content')))
+				.entity('Content', e => e.column('foo'))
+				.buildSchema(),
 		},
-	],
-	sql: SQL``,
-	noDiff: true,
-}))
+		updated: {
+			model: new SchemaBuilder()
+				.entity('User', e => e.tableName('author').oneHasOne('content', r => r.target('Content')))
+				.entity('Content', e => e.column('foo'))
+				.buildSchema(),
+		},
+		diff: [
+			{
+				modification: 'updateEntityName',
+				entityName: 'Author',
+				newEntityName: 'User',
+			},
+		],
+		sql: SQL``,
+		noDiff: true,
+	}))
 
-describe('rename table with acl', () => testMigrations({
-	original: {
-		model: new SchemaBuilder()
-			.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)))
-			.buildSchema(),
-		acl: {
-			roles: {
-				admin: {
-					variables: {},
-					stages: '*',
-					entities: {
-						Site: {
-							predicates: {},
-							operations: {
-								read: {
-									id: true,
+describe('rename table with acl', () =>
+	testMigrations({
+		original: {
+			model: new SchemaBuilder()
+				.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)))
+				.buildSchema(),
+			acl: {
+				roles: {
+					admin: {
+						variables: {},
+						stages: '*',
+						entities: {
+							Site: {
+								predicates: {},
+								operations: {
+									read: {
+										id: true,
+									},
 								},
 							},
 						},
@@ -100,22 +104,22 @@ describe('rename table with acl', () => testMigrations({
 				},
 			},
 		},
-	},
-	updated: {
-		model: new SchemaBuilder()
-			.entity('Website', entity => entity.column('name', c => c.type(Model.ColumnType.String)).tableName('site'))
-			.buildSchema(),
-		acl: {
-			roles: {
-				admin: {
-					variables: {},
-					stages: '*',
-					entities: {
-						Website: {
-							predicates: {},
-							operations: {
-								read: {
-									id: true,
+		updated: {
+			model: new SchemaBuilder()
+				.entity('Website', entity => entity.column('name', c => c.type(Model.ColumnType.String)).tableName('site'))
+				.buildSchema(),
+			acl: {
+				roles: {
+					admin: {
+						variables: {},
+						stages: '*',
+						entities: {
+							Website: {
+								predicates: {},
+								operations: {
+									read: {
+										id: true,
+									},
 								},
 							},
 						},
@@ -123,14 +127,13 @@ describe('rename table with acl', () => testMigrations({
 				},
 			},
 		},
-	},
-	diff: [
-		{
-			modification: 'updateEntityName',
-			entityName: 'Site',
-			newEntityName: 'Website',
-		},
-	],
-	sql: SQL``,
-	noDiff: true,
-}))
+		diff: [
+			{
+				modification: 'updateEntityName',
+				entityName: 'Site',
+				newEntityName: 'Website',
+			},
+		],
+		sql: SQL``,
+		noDiff: true,
+	}))