Mirrors/super-productivity
1
0
Fork 0
mirror of https://github.com/johannesjo/super-productivity.git synced 2026-01-23 02:36:05 +00:00
Code Issues Projects Releases Wiki Activity

fix: update tests to reflect current schema version 2 after migration

Browse source
This commit is contained in:
Ivan Kalashnikov 2026-01-19 14:18:25 +07:00
parent 263495b8cd
commit aad5cfd892
1 changed files with 11 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

23
src/app/op-log/persistence/schema-migration.service.spec.ts
View file

@ -54,9 +54,9 @@ describe('SchemaMigrationService', () => {
expect(service.getCurrentVersion()).toBe(CURRENT_SCHEMA_VERSION);
});
it('should return 1 as the initial version', () => {
// Current implementation starts at version 1
expect(service.getCurrentVersion()).toBe(1);
it('should return 2 as the current version', () => {
// Current implementation is at version 2 after migration from MiscConfig to TasksConfig
expect(service.getCurrentVersion()).toBe(2);
});
});
@ -79,17 +79,15 @@ describe('SchemaMigrationService', () => {
expect(service.needsMigration(cache)).toBeFalse();
});
it('should return false for cache with undefined schemaVersion (defaults to 1)', () => {
it('should return true for cache with undefined schemaVersion (defaults to 1)', () => {
const cache = createMockCache(undefined);
// When schemaVersion is undefined, it defaults to 1
// Since CURRENT_SCHEMA_VERSION is 1, no migration needed
expect(service.needsMigration(cache)).toBeFalse();
// Since CURRENT_SCHEMA_VERSION is 2, migration is needed
expect(service.needsMigration(cache)).toBeTrue();
});
it('should return true for cache with older version', () => {
// This test will only make sense when we have migrations
// For now, since CURRENT_SCHEMA_VERSION is 1, there's no "older" version
const cache = createMockCache(0); // Version 0 is older
const cache = createMockCache(1); // Version 1 is older than current version 2
expect(service.needsMigration(cache)).toBeTrue();
});
});
@ -100,14 +98,15 @@ describe('SchemaMigrationService', () => {
expect(service.operationNeedsMigration(op)).toBeFalse();
});
it('should return false for operation with undefined schemaVersion (defaults to 1)', () => {
it('should return true for operation with undefined schemaVersion (defaults to 1)', () => {
const op = createMockOperation('op-1');
op.schemaVersion = undefined as any;
expect(service.operationNeedsMigration(op)).toBeFalse();
// Since CURRENT_SCHEMA_VERSION is 2, migration is needed
expect(service.operationNeedsMigration(op)).toBeTrue();
});
it('should return true for operation with older version', () => {
const op = createMockOperation('op-1', 0);
const op = createMockOperation('op-1', 1);
expect(service.operationNeedsMigration(op)).toBeTrue();
});
});