@contember/schema-migrations 1.2.1 → 1.3.0-alpha.4

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

@@ -138,7 +138,7 @@ CREATE CONSTRAINT TRIGGER "log_event_trx"
 ALTER TABLE "post"
   ADD "author_id" uuid;
 ALTER TABLE "post"
-  ADD CONSTRAINT "fk_post_author_id_87ef9a" FOREIGN KEY ("author_id") REFERENCES "author"("id") ON DELETE NO ACTION DEFERRABLE INITIALLY IMMEDIATE;
+  ADD CONSTRAINT "fk_post_author_id_87ef9a" FOREIGN KEY ("author_id") REFERENCES "author"("id") ON DELETE CASCADE DEFERRABLE INITIALLY IMMEDIATE;
 CREATE INDEX "post_author_id_index" ON "post" ("author_id");
 ALTER TABLE "post" RENAME TO "article";
 CREATE TABLE "post" (
@@ -156,7 +156,7 @@ CREATE CONSTRAINT TRIGGER "log_event_trx"
 ALTER TABLE "post"
   ADD "author_id" uuid;
 ALTER TABLE "post"
-  ADD CONSTRAINT "fk_post_author_id_87ef9a" FOREIGN KEY ("author_id") REFERENCES "author"("id") ON DELETE NO ACTION DEFERRABLE INITIALLY IMMEDIATE;
+  ADD CONSTRAINT "fk_post_author_id_87ef9a" FOREIGN KEY ("author_id") REFERENCES "author"("id") ON DELETE CASCADE DEFERRABLE INITIALLY IMMEDIATE;
 CREATE INDEX "post_author_id_index1" ON "post" ("author_id");`)
 })
 

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

@@ -62,6 +62,27 @@ testMigrations('rename entity with a constraint', {
 	noDiff: true,
 })
 
+testMigrations('rename entity with one-has-one (constraint)', {
+	originalSchema: new SchemaBuilder()
+		.entity('Author', e => e.oneHasOne('content', r => r.target('Content')))
+		.entity('Content', e => e.column('foo'))
+		.buildSchema(),
+	updatedSchema: 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`ALTER TABLE "author"
+			RENAME CONSTRAINT "unique_Author_content_b64856" TO "unique_User_content_43c77f";`,
+	noDiff: true,
+})
+
 testMigrations('rename table with acl', {
 	originalSchema: new SchemaBuilder()
 		.entity('Site', entity => entity.column('name', c => c.type(Model.ColumnType.String)))

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

@@ -0,0 +1,43 @@
+import { testMigrations } from '../../src/tests'
+import { SchemaBuilder, SchemaDefinition as def } from '@contember/schema-definition'
+import { Model } from '@contember/schema'
+import { SQL } from '../../src/tags'
+
+
+namespace SchemaWithoutOrderBy {
+	export class Article {
+		title = def.stringColumn()
+	}
+}
+
+namespace SchemaWithOrderBy {
+	@def.OrderBy('title', Model.OrderDirection.asc)
+	export class Article {
+		title = def.stringColumn()
+	}
+}
+
+testMigrations('update entity add order by', {
+	originalSchema: def.createModel(SchemaWithoutOrderBy),
+	updatedSchema: def.createModel(SchemaWithOrderBy),
+	diff: [
+		{
+			modification: 'updateEntityOrderBy',
+			entityName: 'Article',
+			orderBy: [{ path: ['title'], direction: 'asc' }],
+		},
+	],
+	sql: SQL``,
+})
+
+testMigrations('update entity remove order by', {
+	originalSchema: def.createModel(SchemaWithOrderBy),
+	updatedSchema: def.createModel(SchemaWithoutOrderBy),
+	diff: [
+		{
+			modification: 'updateEntityOrderBy',
+			entityName: 'Article',
+		},
+	],
+	sql: SQL``,
+})

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

@@ -63,6 +63,28 @@ testMigrations('rename a field with a constraint', {
 	noDiff: true,
 })
 
+testMigrations('rename a field with one-has-one', {
+	originalSchema: new SchemaBuilder()
+		.entity('Author', e => e.oneHasOne('content', r => r.target('Content')))
+		.entity('Content', e => e.column('foo'))
+		.buildSchema(),
+	updatedSchema: new SchemaBuilder()
+		.entity('Author', e => e.oneHasOne('description', r => r.target('Content').joiningColumn('content_id')))
+		.entity('Content', e => e.column('foo'))
+		.buildSchema(),
+	diff: [
+		{
+			modification: 'updateFieldName',
+			entityName: 'Author',
+			fieldName: 'content',
+			newFieldName: 'description',
+		},
+	],
+	sql: SQL`ALTER TABLE "author"
+        RENAME CONSTRAINT "unique_Author_content_b64856" TO "unique_Author_description_00f1ef";`,
+	noDiff: true,
+})
+
 testMigrations('rename a relation', {
 	originalSchema: new SchemaBuilder()
 		.entity('Post', e => e.column('title').manyHasOne('user', r => r.target('Author').inversedBy('posts')))

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

@@ -26,5 +26,6 @@ testMigrations('update relation ondelete to cascade', {
 			onDelete: Model.OnDelete.cascade,
 		},
 	],
-	sql: SQL``,
+	sql: SQL`ALTER TABLE "post" DROP CONSTRAINT "fk_post_category_id_820029";
+ALTER TABLE "post" ADD CONSTRAINT "fk_post_category_id_820029" FOREIGN KEY ("category_id") REFERENCES "category"("id") ON DELETE NO ACTION DEFERRABLE INITIALLY IMMEDIATE;`,
 })

package/tests/src/tests.ts

@@ -29,8 +29,8 @@ export function testDiffSchemas(
 	updatedAcl: Acl.Schema = emptyAcl,
 ) {
 	const actualDiff = schemaDiffer.diffSchemas(
-		{ ...emptySchema, model: originalModel, acl: originalAcl, validation: {} },
-		{ ...emptySchema, model: updatedModel, acl: updatedAcl, validation: {} },
+		{ ...emptySchema, model: originalModel, acl: originalAcl },
+		{ ...emptySchema, model: updatedModel, acl: updatedAcl },
 		{ skipRecreateValidation: true },
 	)
 	try {
@@ -41,7 +41,7 @@ export function testDiffSchemas(
 		throw e
 	}
 	const { meta, ...schema } = schemaMigrator.applyModifications(
-		{ ...emptySchema, model: originalModel, acl: originalAcl, validation: {} },
+		{ ...emptySchema, model: originalModel, acl: originalAcl },
 		actualDiff,
 		VERSION_LATEST,
 	) as SchemaWithMeta
@@ -49,7 +49,6 @@ export function testDiffSchemas(
 		...emptySchema,
 		model: updatedModel,
 		acl: updatedAcl,
-		validation: {},
 	})
 }
 
@@ -61,7 +60,7 @@ export function testApplyDiff(
 	expectedAcl: Acl.Schema = emptyAcl,
 ) {
 	const { meta, ...actualSchema } = schemaMigrator.applyModifications(
-		{ ...emptySchema, model: originalModel, acl: originalAcl, validation: {} },
+		{ ...emptySchema, model: originalModel, acl: originalAcl },
 		diff,
 		VERSION_LATEST,
 	) as SchemaWithMeta
@@ -70,12 +69,11 @@ export function testApplyDiff(
 		...emptySchema,
 		model: expectedModel,
 		acl: expectedAcl,
-		validation: {},
 	})
 }
 
 export function testGenerateSql(originalSchema: Model.Schema, diff: Migration.Modification[], expectedSql: string) {
-	let schema = { ...emptySchema, model: originalSchema, acl: emptyAcl, validation: {} }
+	let schema = { ...emptySchema, model: originalSchema, acl: emptyAcl }
 	const builder = createMigrationBuilder()
 	for (let { modification, ...data } of diff) {
 		const modificationHandler = modificationFactory.create(modification, data, schema, { formatVersion: VERSION_LATEST, systemSchema: 'system' })

package/tests/tsconfig.json

@@ -5,11 +5,7 @@
 		"esModuleInterop": true
 	},
 	"references": [
-		{
-			"path": "../src"
-		},
-		{
-			"path": "../../schema-definition/src"
-		}
+		{ "path": "../src" },
+		{ "path": "../../schema-definition/src" }
 	]
 }

package/dist/src/MigrationVersionHelper.d.ts

@@ -1,9 +0,0 @@
-export declare class MigrationVersionHelper {
-    static prefixLength: number;
-    static extractVersion(filename: string): string;
-    static createVersion(name: string, increment?: number): [string, string];
-    static extractName(filename: string): string;
-    private static createTimePrefix;
-    private static normalizeMigrationLabel;
-}
-//# sourceMappingURL=MigrationVersionHelper.d.ts.map

package/dist/src/MigrationVersionHelper.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"MigrationVersionHelper.d.ts","sourceRoot":"","sources":["../../src/MigrationVersionHelper.ts"],"names":[],"mappings":"AAAA,qBAAa,sBAAsB;IAClC,OAAc,YAAY,SAA6B;WAEzC,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM;WAIxC,aAAa,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,GAAE,MAAU,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC;WAMpE,WAAW,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM;IAInD,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAyB/B,OAAO,CAAC,MAAM,CAAC,uBAAuB;CAGtC"}

package/dist/src/MigrationVersionHelper.js

@@ -1,45 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.MigrationVersionHelper = void 0;
-class MigrationVersionHelper {
-    static extractVersion(filename) {
-        return filename.substring(0, this.prefixLength);
-    }
-    static createVersion(name, increment = 0) {
-        const normalizedName = MigrationVersionHelper.normalizeMigrationLabel(name);
-        const version = MigrationVersionHelper.createTimePrefix(increment);
-        return [version, `${version}-${normalizedName}`];
-    }
-    static extractName(filename) {
-        return filename.substring(0, filename.lastIndexOf('.'));
-    }
-    static createTimePrefix(increment) {
-        const now = new Date();
-        now.setSeconds(now.getSeconds() + increment);
-        const year = now.getFullYear();
-        const month = (now.getMonth() + 1).toString().padStart(2, '0');
-        const day = now //
-            .getDate()
-            .toString()
-            .padStart(2, '0');
-        const hours = now //
-            .getHours()
-            .toString()
-            .padStart(2, '0');
-        const minutes = now //
-            .getMinutes()
-            .toString()
-            .padStart(2, '0');
-        const seconds = now //
-            .getSeconds()
-            .toString()
-            .padStart(2, '0');
-        return `${year}-${month}-${day}-${hours}${minutes}${seconds}`;
-    }
-    static normalizeMigrationLabel(name) {
-        return name.toLowerCase().replace(/[^a-z0-9]+/g, '-');
-    }
-}
-exports.MigrationVersionHelper = MigrationVersionHelper;
-MigrationVersionHelper.prefixLength = 'YYYY-MM-DD-HHIISS'.length;
-//# sourceMappingURL=MigrationVersionHelper.js.map

package/dist/src/MigrationVersionHelper.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"MigrationVersionHelper.js","sourceRoot":"","sources":["../../src/MigrationVersionHelper.ts"],"names":[],"mappings":";;;AAAA,MAAa,sBAAsB;IAG3B,MAAM,CAAC,cAAc,CAAC,QAAgB;QAC5C,OAAO,QAAQ,CAAC,SAAS,CAAC,CAAC,EAAE,IAAI,CAAC,YAAY,CAAC,CAAA;IAChD,CAAC;IAEM,MAAM,CAAC,aAAa,CAAC,IAAY,EAAE,YAAoB,CAAC;QAC9D,MAAM,cAAc,GAAG,sBAAsB,CAAC,uBAAuB,CAAC,IAAI,CAAC,CAAA;QAC3E,MAAM,OAAO,GAAG,sBAAsB,CAAC,gBAAgB,CAAC,SAAS,CAAC,CAAA;QAClE,OAAO,CAAC,OAAO,EAAE,GAAG,OAAO,IAAI,cAAc,EAAE,CAAC,CAAA;IACjD,CAAC;IAEM,MAAM,CAAC,WAAW,CAAC,QAAgB;QACzC,OAAO,QAAQ,CAAC,SAAS,CAAC,CAAC,EAAE,QAAQ,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,CAAA;IACxD,CAAC;IAEO,MAAM,CAAC,gBAAgB,CAAC,SAAiB;QAChD,MAAM,GAAG,GAAG,IAAI,IAAI,EAAE,CAAA;QACtB,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,UAAU,EAAE,GAAG,SAAS,CAAC,CAAA;QAC5C,MAAM,IAAI,GAAG,GAAG,CAAC,WAAW,EAAE,CAAA;QAC9B,MAAM,KAAK,GAAG,CAAC,GAAG,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;QAC9D,MAAM,GAAG,GAAG,GAAG,CAAC,EAAE;aAChB,OAAO,EAAE;aACT,QAAQ,EAAE;aACV,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;QAClB,MAAM,KAAK,GAAG,GAAG,CAAC,EAAE;aAClB,QAAQ,EAAE;aACV,QAAQ,EAAE;aACV,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;QAClB,MAAM,OAAO,GAAG,GAAG,CAAC,EAAE;aACpB,UAAU,EAAE;aACZ,QAAQ,EAAE;aACV,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;QAClB,MAAM,OAAO,GAAG,GAAG,CAAC,EAAE;aACpB,UAAU,EAAE;aACZ,QAAQ,EAAE;aACV,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;QAElB,OAAO,GAAG,IAAI,IAAI,KAAK,IAAI,GAAG,IAAI,KAAK,GAAG,OAAO,GAAG,OAAO,EAAE,CAAA;IAC9D,CAAC;IAEO,MAAM,CAAC,uBAAuB,CAAC,IAAY;QAClD,OAAO,IAAI,CAAC,WAAW,EAAE,CAAC,OAAO,CAAC,aAAa,EAAE,GAAG,CAAC,CAAA;IACtD,CAAC;;AA5CF,wDA6CC;AA5Cc,mCAAY,GAAG,mBAAmB,CAAC,MAAM,CAAA"}

package/src/MigrationVersionHelper.ts

@@ -1,46 +0,0 @@
-export class MigrationVersionHelper {
-	public static prefixLength = 'YYYY-MM-DD-HHIISS'.length
-
-	public static extractVersion(filename: string): string {
-		return filename.substring(0, this.prefixLength)
-	}
-
-	public static createVersion(name: string, increment: number = 0): [string, string] {
-		const normalizedName = MigrationVersionHelper.normalizeMigrationLabel(name)
-		const version = MigrationVersionHelper.createTimePrefix(increment)
-		return [version, `${version}-${normalizedName}`]
-	}
-
-	public static extractName(filename: string): string {
-		return filename.substring(0, filename.lastIndexOf('.'))
-	}
-
-	private static createTimePrefix(increment: number): string {
-		const now = new Date()
-		now.setSeconds(now.getSeconds() + increment)
-		const year = now.getFullYear()
-		const month = (now.getMonth() + 1).toString().padStart(2, '0')
-		const day = now //
-			.getDate()
-			.toString()
-			.padStart(2, '0')
-		const hours = now //
-			.getHours()
-			.toString()
-			.padStart(2, '0')
-		const minutes = now //
-			.getMinutes()
-			.toString()
-			.padStart(2, '0')
-		const seconds = now //
-			.getSeconds()
-			.toString()
-			.padStart(2, '0')
-
-		return `${year}-${month}-${day}-${hours}${minutes}${seconds}`
-	}
-
-	private static normalizeMigrationLabel(name: string) {
-		return name.toLowerCase().replace(/[^a-z0-9]+/g, '-')
-	}
-}