@into-cps-association/libms 0.4.4 → 0.4.7

package/eslint.config.js

@@ -0,0 +1,60 @@
+import { fixupConfigRules } from '@eslint/compat';
+import { FlatCompat } from '@eslint/eslintrc';
+import globals from 'globals';
+import jest from 'eslint-plugin-jest';
+import js from '@eslint/js';
+import prettier from 'eslint-config-prettier';
+import ts from '@typescript-eslint/eslint-plugin';
+import tsParser from '@typescript-eslint/parser';
+import imprt from 'eslint-plugin-import'; // 'import' is ambiguous & prettier has trouble
+
+const flatCompat = new FlatCompat();
+
+export default [
+  {
+    ...js.configs.recommended,
+    files: ['src/**', 'test/**'],
+  },
+  {
+    ...fixupConfigRules(flatCompat.extends('airbnb-base')),
+    files: ['src/**', 'test/**'],
+  },
+  prettier,
+  {
+    languageOptions: {
+      globals: {
+        ...globals.jest,
+        ...globals.node,
+        Atomics: 'readonly',
+        SharedArrayBuffer: 'readonly',
+      },
+      parser: tsParser,
+      parserOptions: {
+        project: './tsconfig.json',
+        requireConfigFile: false,
+        ecmaVersion: 2022,
+        ecmaFeatures: { modules: true },
+      },
+    },
+    files: ['src/**', 'test/**'],
+    plugins: { jest, '@typescript-eslint': ts, import: imprt, ts },
+    rules: {
+      'import/no-extraneous-dependencies': ['error', { devDependencies: true }],
+      'no-console': 'error',
+      'import/first': 'error',
+      'linebreak-style': 0, // disable linter linebreak rule, to allow for both unix and windows developement
+      'import/no-unresolved': 'off', // Whatever IDE will pass an error if if the module is not found, so no reason for this..
+      'import/extensions': 'off', // That includes the production build.. We use linter for code checking / clean code optimization..
+      'no-use-before-define': 'off',
+    },
+    ignores: [
+      'api/*',
+      'build/*',
+      'coverage/*',
+      'dist/*',
+      'node_modules/*',
+      'script/*',
+      'src/types.ts',
+    ],
+  },
+];

package/jest.config.ts

@@ -0,0 +1,47 @@
+import { createDefaultEsmPreset, type JestConfigWithTsJest } from 'ts-jest';
+
+const jestConfig: JestConfigWithTsJest = {
+  testEnvironment: 'node',
+  transform: {
+    ...createDefaultEsmPreset().transform,
+    '\\.[jt]sx?$": "ts-jest': [
+      'ts-jest',
+      {
+        useESM: true,
+      },
+    ],
+  },
+  collectCoverage: true,
+  coverageReporters: ['text', 'cobertura', 'clover', 'lcov', 'json'],
+  collectCoverageFrom: ['src/**/*.{ts,js}'],
+  coveragePathIgnorePatterns: [
+    'node_modules',
+    './dist',
+    './src/app.module.ts',
+    './src/main.ts',
+    './src/bootstrap.ts',
+  ],
+  extensionsToTreatAsEsm: ['.ts'],
+  moduleFileExtensions: ['js', 'json', 'ts'],
+  modulePathIgnorePatterns: [],
+  coverageDirectory: '<rootDir>/coverage/',
+  coverageThreshold: {
+    global: {
+      branches: 20,
+      functions: 30,
+      lines: 50,
+      statements: 50,
+    },
+  },
+  verbose: true,
+  testRegex: './test/.*\\.spec.tsx?$',
+  modulePaths: ['<rootDir>', '<rootDir>/src/', '<rootDir>/test/'],
+  moduleDirectories: ['node_modules', 'src', 'test'],
+  rootDir: './',
+  roots: ['<rootDir>', 'src/', 'test/'],
+  moduleNameMapper: {
+    '^(\\.\\.?\\/.+)\\.jsx?$': '$1',
+  },
+};
+
+export default jestConfig;

package/package.json

@@ -1,81 +1,87 @@
 {
   "name": "@into-cps-association/libms",
-  "version": "0.4.4",
+  "version": "0.4.7",
   "description": "microservices that handles request by fetching and returning the file-names and folders of given directory",
   "author": "phillip.boe.jensen@gmail.com",
   "contributors": [
     "Prasad Talasila",
     "Mads Kelberg",
-    "Linda Nguyen"
+    "Linda Nguyen",
+    "Nichlaes H. Sørensen"
   ],
   "private": false,
   "license": "SEE LICENSE IN <LICENSE.md>",
+  "type": "module",
+  "main": "dist/src/main.js",
   "scripts": {
-    "build": "npx tsc",
+    "build": "tsc",
     "clean": "npx rimraf build node_modules coverage dist src.svg test.svg",
     "format": "prettier --ignore-path ../.gitignore --write \"**/*.{ts,tsx,css,scss}\"",
     "graph": "npx madge --image src.svg src && npx madge --image test.svg test",
-    "start": "node dist/src/main.js",
-    "start:pm2": "pm2 start pm2.config.js",
+    "start": "tsc & node dist/src/main.js",
+    "start:pm2": "pm2 start pm2.config.cjs",
     "stop:pm2": "pm2 delete libms",
-    "syntax": "npx eslint . --fix",
+    "syntax": "eslint . --fix",
     "pretest": "npx shx cp test/data/user2/tools/README.md ../../files/user2/tools/README.md",
     "posttest": "npx rimraf ../../files/user2/tools/README.md",
-    "test:all": "npx cross-env LOCAL_PATH=test/data jest --testPathIgnorePatterns=cloudcmd --coverage",
-    "test:e2e": "npx cross-env LOCAL_PATH=test/data jest --config ./test/jest-e2e.json --coverage",
+    "test:all": "npx cross-env NODE_OPTIONS=--experimental-vm-modules LOCAL_PATH=test/data jest --testPathIgnorePatterns=cloudcmd --coverage",
+    "test:e2e": "npx cross-env NODE_OPTIONS=--experimental-vm-modules LOCAL_PATH=test/data jest --config ./test/jest-e2e.json --coverage",
     "test:http": "yarn build && pm2 start -f --name libms-test dist/src/main.js -- -c .env -H ./config/http.json && jest test/cloudcmd --coverage --coverageThreshold=\"{}\" && pm2 delete libms-test",
     "test:http-nocov": "yarn build && pm2 start -f --name libms-test dist/src/main.js -- -c .env -H ./config/http.json && jest test/cloudcmd  --coverage=false && pm2 delete libms-test",
-    "test:http-github": "yarn build && yarn start:pm2 && jest test/cloudcmd --coverage --coverageThreshold=\"{}\" && pm2 delete libms",
-    "test:int": "npx cross-env LOCAL_PATH=test/data jest ../test/integration --coverage",
+    "test:http-github": "jest test/cloudcmd --coverage --coverageThreshold=\"{}\"",
+    "test:int": "npx cross-env NODE_OPTIONS=--experimental-vm-modules LOCAL_PATH=test/data jest ../test/integration --coverage",
     "test:nocov": "yarn test:http-nocov && npx cross-env LOCAL_PATH=test/data jest --testPathIgnorePatterns=cloudcmd --coverage=false",
-    "test:unit": "npx cross-env LOCAL_PATH=test/data jest ../test/unit --coverage"
+    "test:unit": "npx cross-env NODE_OPTIONS=--experimental-vm-modules LOCAL_PATH=test/data jest ../test/unit --coverage"
   },
   "bin": "./dist/src/main.js",
   "dependencies": {
-    "@apollo/client": "^3.8.9",
-    "@apollo/server": "^4.10.0",
-    "@nestjs/apollo": "^12.0.11",
-    "@nestjs/common": "^10.3.7",
-    "@nestjs/config": "^3.2.0",
-    "@nestjs/core": "^10.3.3",
-    "@nestjs/graphql": "^12.0.11",
-    "@nestjs/platform-express": "^10.3.7",
-    "axios": "^1.5.1",
-    "cloudcmd": "^16.17.7",
-    "commander": "^11.1.0",
-    "dotenv": "^16.3.1",
-    "graphql": "^16.8.1",
-    "mock-fs": "^5.2.0",
-    "reflect-metadata": "^0.2.1",
+    "@apollo/client": "^3.11.8",
+    "@apollo/server": "^4.11.0",
+    "@nestjs/apollo": "^12.2.0",
+    "@nestjs/common": "^10.4.4",
+    "@nestjs/config": "^3.2.3",
+    "@nestjs/core": "^10.4.4",
+    "@nestjs/graphql": "^12.2.0",
+    "@nestjs/platform-express": "^10.4.4",
+    "axios": "^1.7.7",
+    "cloudcmd": "^18.1.0",
+    "commander": "^12.1.0",
+    "dotenv": "^16.4.5",
+    "globals": "^15.9.0",
+    "graphql": "^16.9.0",
+    "mock-fs": "^5.3.0",
+    "reflect-metadata": "^0.2.2",
     "rxjs": "^7.8.1",
-    "socket.io": "^4.7.2",
-    "type-graphql": "^2.0.0-beta.3"
+    "socket.io": "^4.8.0",
+    "type-graphql": "^2.0.0-rc.2"
   },
   "devDependencies": {
-    "@nestjs/cli": "^10.3.2",
-    "@nestjs/schematics": "^10.1.1",
-    "@nestjs/testing": "^10.3.3",
-    "@types/express": "^4.17.21",
-    "@types/jest": "^29.5.12",
-    "@types/node": "20.12.5",
+    "@eslint/compat": "^1.1.1",
+    "@eslint/eslintrc": "^3.1.0",
+    "@nestjs/cli": "^10.4.5",
+    "@nestjs/schematics": "^10.1.4",
+    "@nestjs/testing": "^10.4.4",
+    "@types/express": "^5.0.0",
+    "@types/jest": "^29.5.13",
+    "@types/node": "22.7.0",
     "@types/supertest": "^6.0.2",
-    "@typescript-eslint/eslint-plugin": "^6.19.0",
-    "@typescript-eslint/parser": "^6.19.0",
+    "@typescript-eslint/eslint-plugin": "^8.8.0",
+    "@typescript-eslint/parser": "^8.8.0",
     "cross-fetch": "^4.0.0",
-    "eslint": "^8.56.0",
+    "eslint": "^9.11.1",
     "eslint-config-airbnb-base": "^15.0.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-jest": "^28.2.0",
-    "eslint-plugin-prettier": "^5.1.3",
-    "graphql-scalars": "^1.22.2",
-    "jest": "29.7.0",
-    "prettier": "^3.2.2",
-    "react": "^18.2.0",
-    "supertest": "^6.3.4",
-    "ts-jest": "^29.1.1",
-    "ts-node": "^10.9.1",
+    "eslint-plugin-import": "^2.31.0",
+    "eslint-plugin-jest": "^28.8.3",
+    "eslint-plugin-prettier": "^5.2.1",
+    "graphql-scalars": "^1.23.0",
+    "jest": "^29.7.0",
+    "prettier": "^3.3.3",
+    "react": "^18.3.1",
+    "supertest": "^7.0.0",
+    "ts-jest": "^29.2.5",
+    "ts-node": "^10.9.2",
     "tsconfig-paths": "4.2.0",
-    "typescript": "^5.2.2"
+    "typescript": "^5.6.2"
   }
 }

package/src/app.module.ts

@@ -3,7 +3,7 @@ import { Module } from '@nestjs/common';
 import { GraphQLModule } from '@nestjs/graphql';
 import { ApolloDriver } from '@nestjs/apollo';
 import { join } from 'path';
-import FilesModule from './files/files.module';
+import FilesModule from './files/files.module.js';
 
 @Module({
   imports: [

package/src/bootstrap.ts

@@ -1,8 +1,8 @@
 import { NestFactory } from '@nestjs/core';
 import { ConfigService } from '@nestjs/config';
 import * as dotenv from 'dotenv';
-import AppModule from './app.module';
-import cloudCMD from './cloudcmd/cloudcmd';
+import AppModule from './app.module.js';
+import cloudCMD from './cloudcmd/cloudcmd.js';
 
 type BootstrapOptions = {
   config?: string;
@@ -15,7 +15,7 @@ export default async function bootstrap(options?: BootstrapOptions) {
     path: options?.config ?? '.env',
     override: true,
   });
-  if (configFile.error) {
+  if (configFile.error && process.env.LOCAL_PATH === undefined) {
     // eslint-disable-next-line no-console
     console.error(configFile.error);
     if (options.runHelp) {

package/src/cloudcmd/cloudcmd.ts

@@ -1,7 +1,9 @@
 import { INestApplication } from '@nestjs/common';
 import { Server } from 'socket.io';
-import * as cloudcmd from 'cloudcmd';
-import { join } from 'path';
+import cloudcmd from 'cloudcmd';
+import { join, relative } from 'path';
+
+const isWindowsAbsolutePath = (filesPath: string) => filesPath.includes(':');
 
 const runCloudCMD = (
   app: INestApplication,
@@ -13,7 +15,13 @@ const runCloudCMD = (
     configPath: join(process.cwd(), optionsPath),
   });
 
-  configManager('root', filesPath);
+  if (isWindowsAbsolutePath(filesPath)) {
+    const workDir = process.cwd();
+    const relativePath = relative(workDir, filesPath);
+    configManager('root', relativePath);
+  } else {
+    configManager('root', filesPath);
+  }
 
   const server = app.getHttpServer();
 

package/src/enums/config-mode.enum.ts

@@ -0,0 +1,4 @@
+export enum CONFIG_MODE {
+  GIT = 'git',
+  LOCAL = 'local',
+}

package/src/files/files-service.factory.ts

@@ -0,0 +1,19 @@
+import { Injectable } from '@nestjs/common';
+import { ConfigService } from '@nestjs/config';
+import { IFilesService } from './interfaces/files.service.interface.js';
+
+@Injectable()
+export default class FilesServiceFactory {
+  static create(
+    configService: ConfigService,
+    fileServices: IFilesService[],
+  ): IFilesService {
+    const mode = configService.get<string>('MODE');
+    const service = fileServices.find((s) => s.getMode() == mode);
+
+    if (service == undefined) {
+      throw new Error(`Invalid MODE: ${mode}`);
+    }
+    return service;
+  }
+}

package/src/files/files.module.ts

@@ -1,9 +1,29 @@
 import { Module } from '@nestjs/common';
-import FilesResolver from './resolvers/files.resolver';
-import FilesServiceFactory from './services/files-service.factory';
-import LocalFilesService from './services/local-files.service';
+import FilesResolver from './files.resolver.js';
+import { GitFilesModule } from './git/git-files.module.js';
+import { LocalFilesModule } from './local/local-files.module.js';
+import LocalFilesService from './local/local-files.service.js';
+import GitFilesService from './git/git-files.service.js';
+import { FILE_SERVICE } from './interfaces/files.service.interface.js';
+import FilesServiceFactory from './files-service.factory.js';
+import { ConfigService } from '@nestjs/config';
 
 @Module({
-  providers: [FilesResolver, LocalFilesService, FilesServiceFactory],
+  imports: [LocalFilesModule, GitFilesModule],
+  providers: [
+    FilesResolver,
+    {
+      provide: FILE_SERVICE,
+      useFactory: (
+        configService: ConfigService,
+        localFilesService: LocalFilesService,
+        gitFilesService: GitFilesService,
+      ) => {
+        const fileServices = [localFilesService, gitFilesService];
+        return FilesServiceFactory.create(configService, fileServices);
+      },
+      inject: [ConfigService, LocalFilesService, GitFilesService],
+    },
+  ],
 })
 export default class FilesModule {}

package/src/files/{resolvers/files.resolver.ts → files.resolver.ts}

@@ -1,15 +1,16 @@
 import { Resolver, Query, Args } from '@nestjs/graphql';
-import { IFilesService } from '../interfaces/files.service.interface';
-import FilesServiceFactory from '../services/files-service.factory';
-import { Project } from '../../types';
+import {
+  FILE_SERVICE,
+  IFilesService,
+} from './interfaces/files.service.interface.js';
+import { Project } from '../types.js';
+import { Inject } from '@nestjs/common';
 
 @Resolver()
 export default class FilesResolver {
-  private readonly filesService: IFilesService;
-
-  constructor(filesServiceFactory: FilesServiceFactory) {
-    this.filesService = filesServiceFactory.create();
-  }
+  constructor(
+    @Inject(FILE_SERVICE) private readonly filesService: IFilesService,
+  ) {}
 
   @Query(() => Project)
   async listDirectory(@Args('path') path: string): Promise<Project> {

package/src/files/git/git-files.module.ts

@@ -0,0 +1,8 @@
+import { Module } from '@nestjs/common';
+import GitFilesService from './git-files.service.js';
+
+@Module({
+  providers: [GitFilesService],
+  exports: [GitFilesService],
+})
+export class GitFilesModule {}

package/src/files/git/git-files.service.ts

@@ -0,0 +1,20 @@
+import { Injectable } from '@nestjs/common';
+import { Project } from 'src/types.js';
+import { IFilesService } from '../interfaces/files.service.interface.js';
+import { CONFIG_MODE } from '../../enums/config-mode.enum.js';
+
+@Injectable()
+export default class GitFilesService implements IFilesService {
+// eslint-disable-next-line no-useless-constructor, no-empty-function
+  constructor() {}
+  getMode(): CONFIG_MODE {
+    return CONFIG_MODE.GIT;
+  }
+
+  listDirectory(): Promise<Project> {
+    throw new Error('Method not implemented.');
+  }
+  readFile(): Promise<Project> {
+    throw new Error('Method not implemented.');
+  }
+}

package/src/files/interfaces/files.service.interface.ts

@@ -1,7 +1,10 @@
-import { Project } from 'src/types';
+import { CONFIG_MODE } from '../../enums/config-mode.enum.js';
+import { Project } from 'src/types.js';
 
+export const FILE_SERVICE = 'FILE_SERVICE';
 // FileService interface
 export interface IFilesService {
   listDirectory(path: string): Promise<Project>;
   readFile(path: string): Promise<Project>;
+  getMode(): CONFIG_MODE;
 }

package/src/files/local/local-files.module.ts

@@ -0,0 +1,8 @@
+import { Module } from '@nestjs/common';
+import LocalFilesService from './local-files.service.js';
+
+@Module({
+  providers: [LocalFilesService],
+  exports: [LocalFilesService],
+})
+export class LocalFilesModule {}

package/src/files/{services → local}/local-files.service.ts

@@ -2,18 +2,24 @@ import { Injectable, InternalServerErrorException } from '@nestjs/common';
 import * as fs from 'fs';
 import { join } from 'path';
 import { ConfigService } from '@nestjs/config';
-import { Project } from 'src/types';
-import { IFilesService } from '../interfaces/files.service.interface';
+import { Project } from 'src/types.js';
+import { IFilesService } from '../interfaces/files.service.interface.js';
+import { CONFIG_MODE } from '../../enums/config-mode.enum.js';
 
 @Injectable()
 export default class LocalFilesService implements IFilesService {
-  // eslint-disable-next-line no-useless-constructor, no-empty-function
-  constructor(private configService: ConfigService) {}
+  private readonly dataPath: string;
 
-  async listDirectory(path: string): Promise<Project> {
-    const dataPath = this.configService.get('LOCAL_PATH');
-    const fullPath = join(dataPath, path);
+  constructor(private configService: ConfigService) {
+    this.dataPath = this.configService.get('LOCAL_PATH');
 
+  }
+  getMode(): CONFIG_MODE {
+    return CONFIG_MODE.LOCAL;
+  }
+
+  async listDirectory(path: string): Promise<Project> {
+    const fullPath = join(this.dataPath, path);
     const files = await fs.promises.readdir(fullPath);
 
     const edges = await Promise.all(
@@ -33,8 +39,7 @@ export default class LocalFilesService implements IFilesService {
   }
 
   async readFile(path: string): Promise<Project> {
-    const dataPath = this.configService.get('LOCAL_PATH');
-    const fullPath = join(dataPath, path);
+    const fullPath = join(this.dataPath, path);
 
     try {
       const content = await (
@@ -45,7 +50,7 @@ export default class LocalFilesService implements IFilesService {
 
       return LocalFilesService.formatResponse(name, content);
     } catch (error) {
-      throw new InternalServerErrorException('Error reading file');
+      throw new InternalServerErrorException('Error reading file', error);
     }
   }
 

package/src/main.ts

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { Command } from 'commander';
-import bootstrap from './bootstrap';
+import bootstrap from './bootstrap.js';
 
 type ProgramOptions = {
   config?: string;

package/test/cloudcmd/cloudcmd.spec.ts

@@ -23,13 +23,11 @@ describe('cloudcmd test for the application', () => {
         responseType: 'json',
       },
     );
-    /* eslint-disable no-console */
-    console.log(response.data);
-    /* eslint-enable no-console */
     expect(response.data.path).toEqual('/');
-    expect(response.data.files[0].name).toEqual('common');
-    expect(response.data.files[1].name).toEqual('user1');
-    expect(response.data.files[2].name).toEqual('user2');
+    const fileNames = response.data.files.map((file) => file.name);
+    expect(fileNames).toContain('common');
+    expect(fileNames).toContain('user1');
+    expect(fileNames).toContain('user2');
   }, 10000);
 
   it('should return the content of a file that is uplaoded to cloudcmd ', async () => {

package/test/e2e/app.e2e.spec.ts

@@ -1,14 +1,15 @@
+import { describe, it, expect } from '@jest/globals';
 import { Test, TestingModule } from '@nestjs/testing';
 import { INestApplication } from '@nestjs/common';
-import * as request from 'supertest';
+import request from 'supertest';
 import fetch from 'cross-fetch';
 import {
   ApolloClient,
   DocumentNode,
-  HttpLink,
   InMemoryCache,
   gql,
-} from '@apollo/client';
+} from '@apollo/client/core/core.cjs';
+import { HttpLink } from '@apollo/client/link/http/http.cjs';
 import AppModule from '../../src/app.module';
 import {
   e2eReadFile,

package/test/integration/files.service.integration.spec.ts

@@ -1,8 +1,8 @@
+import { describe, it, expect, jest } from '@jest/globals';
 import { Test, TestingModule } from '@nestjs/testing';
 import { ConfigService } from '@nestjs/config';
-import FilesResolver from '../../src/files/resolvers/files.resolver';
-import FilesServiceFactory from '../../src/files/services/files-service.factory';
-import LocalFilesService from '../../src/files/services/local-files.service';
+import FilesResolver from '../../src/files/files.resolver';
+import LocalFilesService from '../../src/files/local/local-files.service';
 import {
   pathToTestDirectory,
   pathToTestFileContent,
@@ -10,6 +10,9 @@ import {
   testFileContent,
   MockConfigService,
 } from '../testUtil';
+import GitFilesService from '../../src/files/git/git-files.service';
+import { FILE_SERVICE } from '../../src/files/interfaces/files.service.interface';
+import FilesServiceFactory from '../../src/files/files-service.factory';
 
 describe('Integration tests for FilesResolver', () => {
   let filesResolver: FilesResolver;
@@ -20,8 +23,20 @@ describe('Integration tests for FilesResolver', () => {
     const module: TestingModule = await Test.createTestingModule({
       providers: [
         FilesResolver,
-        FilesServiceFactory,
+        {
+          provide: FILE_SERVICE,
+          useFactory: (
+            configService: ConfigService,
+            localFilesService: LocalFilesService,
+            gitFilesService: GitFilesService,
+          ) => {
+            const fileServices = [localFilesService, gitFilesService];
+            return FilesServiceFactory.create(configService, fileServices);
+          },
+          inject: [ConfigService, LocalFilesService, GitFilesService],
+        },
         LocalFilesService,
+        GitFilesService,
         { provide: ConfigService, useClass: MockConfigService },
       ],
     }).compile();

package/test/unit/files-service.factory.unit.spec.ts

@@ -1,37 +1,44 @@
 // files-service.factory.spec.ts
+import { describe, it, expect, jest } from '@jest/globals';
 import { Test, TestingModule } from '@nestjs/testing';
 import { ConfigService } from '@nestjs/config';
-import FilesServiceFactory from '../../src/files/services/files-service.factory';
-import LocalFilesService from '../../src/files/services/local-files.service';
+import FilesServiceFactory from '../../src/files/files-service.factory';
+import LocalFilesService from '../../src/files/local/local-files.service';
 import { IFilesService } from '../../src/files/interfaces/files.service.interface';
+import GitFilesService from '../../src/files/git/git-files.service';
 
 describe('FilesServiceFactory', () => {
-  let serviceFactory: FilesServiceFactory;
   let configService: ConfigService;
   let localFilesService: IFilesService;
+  let gitFilesService: IFilesService;
+  let fileServices: IFilesService[];
 
   beforeEach(async () => {
-    localFilesService = new LocalFilesService(configService as ConfigService);
-
+    
     const module: TestingModule = await Test.createTestingModule({
       providers: [
-        FilesServiceFactory,
+        GitFilesService,
         { provide: ConfigService, useValue: { get: jest.fn() } },
-        { provide: LocalFilesService, useValue: localFilesService },
       ],
     }).compile();
-
-    serviceFactory = module.get<FilesServiceFactory>(FilesServiceFactory);
+    
     configService = module.get<ConfigService>(ConfigService);
+    localFilesService = new LocalFilesService(configService);
+    gitFilesService = module.get<GitFilesService>(GitFilesService);
+    fileServices = [gitFilesService, localFilesService];
   });
 
   it('should create a local files service when MODE is local', () => {
     jest.spyOn(configService, 'get').mockReturnValue('local');
-    expect(serviceFactory.create()).toBe(localFilesService);
+    expect(FilesServiceFactory.create(configService, fileServices)).toBe(
+      localFilesService,
+    );
   });
 
   it('should throw an error when MODE is invalid', () => {
     jest.spyOn(configService, 'get').mockReturnValue('invalid');
-    expect(() => serviceFactory.create()).toThrowError(`Invalid MODE: invalid`);
+    expect(() =>
+      FilesServiceFactory.create(configService, fileServices),
+    ).toThrow(`Invalid MODE: invalid`);
   });
 });