@friggframework/devtools 2.0.0--canary.545.ae2019f.0 → 2.0.0--canary.549.a579cca.0

package/frigg-cli/__tests__/infrastructure/repositories/FileSystemApiModuleRepository.test.js

@@ -1,583 +0,0 @@
-const {FileSystemApiModuleRepository} = require('../../../infrastructure/repositories/FileSystemApiModuleRepository');
-const {ApiModule} = require('../../../domain/entities/ApiModule');
-
-describe('FileSystemApiModuleRepository', () => {
-    let repository;
-    let mockFileSystemAdapter;
-    let mockSchemaValidator;
-    let projectRoot;
-
-    beforeEach(() => {
-        projectRoot = '/test/project';
-
-        mockFileSystemAdapter = {
-            exists: jest.fn(),
-            ensureDirectory: jest.fn(),
-            writeFile: jest.fn(),
-            readFile: jest.fn(),
-            listDirectories: jest.fn(),
-            deleteDirectory: jest.fn(),
-        };
-
-        mockSchemaValidator = {
-            validate: jest.fn(),
-        };
-
-        repository = new FileSystemApiModuleRepository(
-            mockFileSystemAdapter,
-            projectRoot,
-            mockSchemaValidator
-        );
-    });
-
-    describe('save', () => {
-        it('should save an API module with all required files', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                description: 'Salesforce API',
-                apiConfig: {
-                    authType: 'oauth2',
-                    baseUrl: 'https://api.salesforce.com',
-                },
-            });
-
-            await repository.save(apiModule);
-
-            // Verify directories created
-            expect(mockFileSystemAdapter.ensureDirectory).toHaveBeenCalledWith(
-                '/test/project/backend/src/api-modules/salesforce'
-            );
-            expect(mockFileSystemAdapter.ensureDirectory).toHaveBeenCalledWith(
-                '/test/project/backend/src/api-modules/salesforce/tests'
-            );
-
-            // Verify files written (4 files without Entity.js)
-            expect(mockFileSystemAdapter.writeFile).toHaveBeenCalledTimes(4);
-        });
-
-        it('should generate Entity.js if module has entities', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addEntity('credential', {
-                label: 'Credential',
-                type: 'credential',
-                required: true,
-                fields: ['accessToken', 'refreshToken'],
-            });
-
-            await repository.save(apiModule);
-
-            // Verify Entity.js written (5 files with Entity.js)
-            expect(mockFileSystemAdapter.writeFile).toHaveBeenCalledTimes(5);
-
-            const entityCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('Entity.js')
-            );
-            expect(entityCall).toBeDefined();
-            expect(entityCall[1]).toContain('class SalesforceEntity extends EntityBase');
-        });
-
-        it('should generate Api.js class file correctly', async () => {
-            const apiModule = ApiModule.create({
-                name: 'my-test-api',
-                version: '1.0.0',
-                displayName: 'My Test API',
-                description: 'Test API description',
-                apiConfig: {
-                    authType: 'oauth2',
-                    baseUrl: 'https://api.test.com',
-                },
-            });
-
-            await repository.save(apiModule);
-
-            const apiCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('Api.js')
-            );
-
-            expect(apiCall).toBeDefined();
-            expect(apiCall[1]).toContain('class MyTestApiApi extends ApiBase');
-            expect(apiCall[1]).toContain("this.baseUrl = 'https://api.test.com'");
-            expect(apiCall[1]).toContain("this.authType = 'oauth2'");
-            expect(apiCall[1]).toContain('module.exports = MyTestApiApi');
-        });
-
-        it('should generate definition.js file correctly', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            await repository.save(apiModule);
-
-            const definitionCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('definition.js')
-            );
-
-            expect(definitionCall).toBeDefined();
-            expect(definitionCall[1]).toContain('module.exports = {');
-            expect(definitionCall[1]).toContain('"name": "salesforce"');
-        });
-
-        it('should generate config.json file correctly', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            await repository.save(apiModule);
-
-            const configCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('config.json')
-            );
-
-            expect(configCall).toBeDefined();
-            const config = JSON.parse(configCall[1]);
-            expect(config.name).toBe('salesforce');
-            expect(config.version).toBe('1.0.0');
-            expect(config.authType).toBe('oauth2');
-        });
-
-        it('should generate README.md file correctly', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                description: 'Salesforce API client',
-                apiConfig: {
-                    authType: 'oauth2',
-                    baseUrl: 'https://api.salesforce.com',
-                },
-            });
-
-            await repository.save(apiModule);
-
-            const readmeCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('README.md')
-            );
-
-            expect(readmeCall).toBeDefined();
-            expect(readmeCall[1]).toContain('# Salesforce');
-            expect(readmeCall[1]).toContain('Salesforce API client');
-            expect(readmeCall[1]).toContain('https://api.salesforce.com');
-        });
-
-        it('should throw error if API module validation fails', async () => {
-            const apiModule = ApiModule.create({
-                name: 'test-api',
-                version: '1.0.0',
-                displayName: 'Test API',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            // Mock validate to return errors
-            jest.spyOn(apiModule, 'validate').mockReturnValue({
-                isValid: false,
-                errors: ['Invalid configuration'],
-            });
-
-            await expect(repository.save(apiModule)).rejects.toThrow(
-                'ApiModule validation failed'
-            );
-        });
-
-        it('should handle endpoints in Api.js generation', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addEndpoint('getUser', {
-                method: 'GET',
-                path: '/user',
-                description: 'Get user information',
-            });
-
-            await repository.save(apiModule);
-
-            const apiCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('Api.js')
-            );
-
-            expect(apiCall[1]).toContain('async getUser()');
-            expect(apiCall[1]).toContain('return await this.get(\'/user\')');
-        });
-
-        it('should handle OAuth scopes in README', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addScope('read:users');
-            apiModule.addScope('write:users');
-
-            await repository.save(apiModule);
-
-            const readmeCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('README.md')
-            );
-
-            expect(readmeCall[1]).toContain('read:users');
-            expect(readmeCall[1]).toContain('write:users');
-        });
-
-        it('should handle credentials in README', async () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addCredential('clientId', {
-                type: 'string',
-                description: 'OAuth Client ID',
-                required: true,
-            });
-
-            await repository.save(apiModule);
-
-            const readmeCall = mockFileSystemAdapter.writeFile.mock.calls.find(
-                call => call[0].endsWith('README.md')
-            );
-
-            expect(readmeCall[1]).toContain('clientId');
-            expect(readmeCall[1]).toContain('OAuth Client ID');
-            expect(readmeCall[1]).toContain('(Required)');
-        });
-    });
-
-    describe('findByName', () => {
-        it.skip('should find API module by name (TODO: needs full implementation)', async () => {
-            // Skip this test because findByName is a simple implementation that
-            // calls ApiModule.create({name}) which requires apiConfig.
-            // Full implementation would parse the definition.js file.
-            mockFileSystemAdapter.exists.mockResolvedValue(true);
-            mockFileSystemAdapter.readFile.mockResolvedValue('module.exports = {}');
-
-            const result = await repository.findByName('salesforce');
-
-            expect(result).toBeInstanceOf(ApiModule);
-            expect(result.name).toBe('salesforce');
-        });
-
-        it('should return null if module directory does not exist', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValueOnce(false);
-
-            const result = await repository.findByName('nonexistent');
-
-            expect(result).toBeNull();
-        });
-
-        it('should return null if definition file does not exist', async () => {
-            mockFileSystemAdapter.exists
-                .mockResolvedValueOnce(true) // Directory exists
-                .mockResolvedValueOnce(false); // Definition file doesn't exist
-
-            const result = await repository.findByName('salesforce');
-
-            expect(result).toBeNull();
-        });
-    });
-
-    describe('exists', () => {
-        it('should return true if API module exists', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValue(true);
-
-            const result = await repository.exists('salesforce');
-
-            expect(result).toBe(true);
-            expect(mockFileSystemAdapter.exists).toHaveBeenCalledWith(
-                '/test/project/backend/src/api-modules/salesforce'
-            );
-        });
-
-        it('should return false if API module does not exist', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValue(false);
-
-            const result = await repository.exists('nonexistent');
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe('list', () => {
-        it('should return empty array if api-modules directory does not exist', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValue(false);
-
-            const result = await repository.list();
-
-            expect(result).toEqual([]);
-        });
-
-        it.skip('should list all API modules (TODO: needs full findByName implementation)', async () => {
-            // Skip because list() uses findByName() which needs full implementation
-            mockFileSystemAdapter.exists.mockResolvedValue(true);
-            mockFileSystemAdapter.listDirectories.mockResolvedValue([
-                'salesforce',
-                'stripe',
-            ]);
-            mockFileSystemAdapter.readFile.mockResolvedValue('module.exports = {}');
-
-            const result = await repository.list();
-
-            expect(result).toHaveLength(2);
-            expect(result[0]).toBeInstanceOf(ApiModule);
-            expect(result[0].name).toBe('salesforce');
-            expect(result[1].name).toBe('stripe');
-        });
-
-        it('should skip invalid modules and log warning', async () => {
-            const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation();
-
-            mockFileSystemAdapter.exists
-                .mockResolvedValueOnce(true) // Directory exists
-                .mockResolvedValueOnce(true) // salesforce dir
-                .mockResolvedValueOnce(true) // salesforce definition
-                .mockResolvedValueOnce(false); // invalid dir (doesn't exist)
-
-            mockFileSystemAdapter.listDirectories.mockResolvedValue([
-                'salesforce',
-                'invalid',
-            ]);
-            mockFileSystemAdapter.readFile.mockResolvedValue('module.exports = {}');
-
-            const result = await repository.list();
-
-            // Result will be empty because findByName throws errors
-            expect(result).toEqual([]);
-            expect(consoleWarnSpy).toHaveBeenCalledWith(
-                expect.stringContaining('Failed to load API module'),
-                expect.any(String)
-            );
-
-            consoleWarnSpy.mockRestore();
-        });
-    });
-
-    describe('delete', () => {
-        it('should delete API module if it exists', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValue(true);
-
-            const result = await repository.delete('salesforce');
-
-            expect(result).toBe(true);
-            expect(mockFileSystemAdapter.deleteDirectory).toHaveBeenCalledWith(
-                '/test/project/backend/src/api-modules/salesforce'
-            );
-        });
-
-        it('should return false if API module does not exist', async () => {
-            mockFileSystemAdapter.exists.mockResolvedValue(false);
-
-            const result = await repository.delete('nonexistent');
-
-            expect(result).toBe(false);
-            expect(mockFileSystemAdapter.deleteDirectory).not.toHaveBeenCalled();
-        });
-    });
-
-    describe('_generateApiClass', () => {
-        it('should generate API class with OAuth methods', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                    baseUrl: 'https://api.salesforce.com',
-                },
-            });
-
-            const result = repository._generateApiClass(apiModule);
-
-            expect(result).toContain('class SalesforceApi extends ApiBase');
-            expect(result).toContain('async getAuthorizationUri()');
-            expect(result).toContain('async getTokenFromCode(code)');
-            expect(result).toContain('async setCredential(credential)');
-            expect(result).toContain('async testAuth()');
-        });
-
-        it('should handle kebab-case module names', () => {
-            const apiModule = ApiModule.create({
-                name: 'my-api-module',
-                version: '1.0.0',
-                displayName: 'My API Module',
-                apiConfig: {
-                    authType: 'api-key',
-                },
-            });
-
-            const result = repository._generateApiClass(apiModule);
-
-            expect(result).toContain('class MyApiModuleApi extends ApiBase');
-            expect(result).toContain('module.exports = MyApiModuleApi');
-        });
-
-        it('should include credential parameter if entity exists', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addEntity('credential', {
-                label: 'Credential',
-                type: 'credential',
-                required: true,
-            });
-
-            const result = repository._generateApiClass(apiModule);
-
-            expect(result).toContain('this.credential = params.credential');
-        });
-    });
-
-    describe('_generateEndpointMethods', () => {
-        it('should generate methods for each endpoint', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addEndpoint('getUser', {
-                method: 'GET',
-                path: '/user',
-                description: 'Get user information',
-            });
-
-            apiModule.addEndpoint('createContact', {
-                method: 'POST',
-                path: '/contacts',
-                description: 'Create a contact',
-                parameters: [{name: 'data'}],
-            });
-
-            const result = repository._generateEndpointMethods(apiModule);
-
-            expect(result).toContain('async getUser()');
-            expect(result).toContain("return await this.get('/user')");
-            expect(result).toContain('async createContact(data)');
-            expect(result).toContain("return await this.post('/contacts', {data})");
-        });
-
-        it('should return empty string if no endpoints', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            const result = repository._generateEndpointMethods(apiModule);
-
-            expect(result).toBe('');
-        });
-    });
-
-    describe('_generateEntityClass', () => {
-        it('should generate Entity class correctly', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                apiConfig: {
-                    authType: 'oauth2',
-                },
-            });
-
-            apiModule.addEntity('credential', {
-                label: 'Credential',
-                type: 'credential',
-                required: true,
-                fields: ['accessToken', 'refreshToken'],
-            });
-
-            const result = repository._generateEntityClass(apiModule);
-
-            expect(result).toContain('class SalesforceEntity extends EntityBase');
-            expect(result).toContain("return 'credential'");
-            expect(result).toContain('accessToken');
-            expect(result).toContain('refreshToken');
-            expect(result).toContain('module.exports = SalesforceEntity');
-        });
-    });
-
-    describe('_generateReadme', () => {
-        it('should generate comprehensive README', () => {
-            const apiModule = ApiModule.create({
-                name: 'salesforce',
-                version: '1.0.0',
-                displayName: 'Salesforce',
-                description: 'Salesforce API client',
-                apiConfig: {
-                    authType: 'oauth2',
-                    baseUrl: 'https://api.salesforce.com',
-                },
-            });
-
-            apiModule.addScope('read:users');
-            apiModule.addCredential('clientId', {
-                type: 'string',
-                description: 'OAuth Client ID',
-                required: true,
-            });
-            apiModule.addEntity('credential', {
-                label: 'Credential',
-                type: 'credential',
-                required: true,
-            });
-
-            const result = repository._generateReadme(apiModule);
-
-            expect(result).toContain('# Salesforce');
-            expect(result).toContain('Salesforce API client');
-            expect(result).toContain('https://api.salesforce.com');
-            expect(result).toContain('oauth2');
-            expect(result).toContain('read:users');
-            expect(result).toContain('clientId');
-            expect(result).toContain('OAuth Client ID');
-            expect(result).toContain('## Usage');
-            expect(result).toContain('## Development');
-        });
-    });
-});