UTs

Author: brendan-kellam

packages/backend/src/db.test.ts

@@ -0,0 +1,32 @@
+import { expect, test } from 'vitest';
+import { migration_addFileLimitSize, migration_addSettings, Schema } from './db';
+import { DEFAULT_SETTINGS } from './constants';
+import { DeepPartial } from './types';
+
+
+test('migration_addSettings adds the `settings` field with defaults if it does not exist', () => {
+    const schema: DeepPartial<Schema> = {};
+
+    const migratedSchema = migration_addSettings(schema as Schema);
+    expect(migratedSchema).toStrictEqual({
+        settings: DEFAULT_SETTINGS,
+    });
+});
+
+test('migration_addFileLimitSize adds the `fileLimitSize` field with the default value if it does not exist', () => {
+    const schema: DeepPartial<Schema> = {
+        settings: {},
+    }
+
+    const migratedSchema = migration_addFileLimitSize(schema as Schema);
+    expect(migratedSchema).toStrictEqual({
+        settings: {
+            fileLimitSize: DEFAULT_SETTINGS.fileLimitSize,
+        }
+    });
+});
+
+test('migration_addFileLimitSize will throw if `settings` is not defined', () => {
+    const schema: DeepPartial<Schema> = {};
+    expect(() => migration_addFileLimitSize(schema as Schema)).toThrow();
+});

packages/backend/src/main.test.ts

@@ -1,6 +1,6 @@
 import { expect, test } from 'vitest';
-import { isRepoReindexingRequired } from './main';
-import { Repository } from './types';
+import { isAllRepoReindexingRequired, isRepoReindexingRequired } from './main';
+import { Repository, Settings } from './types';
 
 test('isRepoReindexingRequired should return false when no changes are made', () => {
     const previous: Repository = {
@@ -77,6 +77,23 @@ test('isRepoReindexingRequired should return true when local excludedPaths chang
     expect(isRepoReindexingRequired(previous, current)).toBe(true);
 });
 
-test('isAllRepoReindexingRequired ...', () => {
-    // todo!
+test('isAllRepoReindexingRequired should return false when fileLimitSize has not changed', () => {
+    const previous: Settings = {
+        fileLimitSize: 1000,
+    }
+    const current: Settings = {
+        ...previous,
+    }
+    expect(isAllRepoReindexingRequired(previous, current)).toBe(false);
+});
+
+test('isAllRepoReindexingRequired should return true when fileLimitSize has changed', () => {
+    const previous: Settings = {
+        fileLimitSize: 1000,
+    }
+    const current: Settings = {
+        ...previous,
+        fileLimitSize: 2000,
+    }
+    expect(isAllRepoReindexingRequired(previous, current)).toBe(true);
 });

packages/backend/src/types.ts

@@ -44,4 +44,9 @@ export type AppContext = {
 
 export type Settings = {
     fileLimitSize: number;
-}
+}
+
+// @see : https://stackoverflow.com/a/61132308
+export type DeepPartial<T> = T extends object ? {
+    [P in keyof T]?: DeepPartial<T[P]>;
+} : T;