g360-cli 1.10.0 → 1.11.0

package/src/lib/manifest.test.js

@@ -0,0 +1,173 @@
+/**
+ * @file manifest.test.js
+ * @description Tests para el módulo manifest
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import fs from 'fs-extra';
+import path from 'path';
+import { manifest } from '../lib/manifest.js';
+
+// Mock de fs-extra
+vi.mock('fs-extra', () => ({
+  default: {
+    writeJson: vi.fn(),
+    readJson: vi.fn(),
+    existsSync: vi.fn(),
+    remove: vi.fn()
+  }
+}));
+
+describe('manifest module', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  describe('init', () => {
+    it('should create a new manifest file', async () => {
+      const projectData = {
+        name: 'test-project',
+        template: 'web-pwa',
+        version: '1.0.0'
+      };
+
+      fs.writeJson.mockResolvedValue();
+
+      const result = await manifest.init('/test/project', projectData);
+
+      expect(fs.writeJson).toHaveBeenCalledWith(
+        path.join('/test/project', 'g360-manifest.json'),
+        expect.objectContaining({
+          name: 'test-project',
+          template: 'web-pwa',
+          version: '1.0.0',
+          createdAt: expect.any(String),
+          assets: []
+        }),
+        { spaces: 2 }
+      );
+
+      expect(result).toMatchObject({
+        name: 'test-project',
+        template: 'web-pwa',
+        version: '1.0.0'
+      });
+    });
+
+    it('should include assets array in manifest', async () => {
+      const projectData = {
+        name: 'test-project',
+        template: 'web-pwa',
+        version: '1.0.0'
+      };
+
+      fs.writeJson.mockResolvedValue();
+
+      const result = await manifest.init('/test/project', projectData);
+
+      expect(result.assets).toEqual([]);
+    });
+  });
+
+  describe('load', () => {
+    it('should load existing manifest', async () => {
+      const mockManifest = {
+        name: 'test-project',
+        template: 'web-pwa',
+        version: '1.0.0',
+        createdAt: '2024-01-01T00:00:00.000Z',
+        assets: []
+      };
+
+      fs.existsSync.mockReturnValue(true);
+      fs.readJson.mockResolvedValue(mockManifest);
+
+      const result = await manifest.load('/test/project');
+
+      expect(result).toEqual(mockManifest);
+    });
+
+    it('should return null for non-existing manifest', async () => {
+      fs.existsSync.mockReturnValue(false);
+
+      const result = await manifest.load('/test/project');
+
+      expect(result).toBeNull();
+    });
+
+    it('should return null when manifest file does not exist', async () => {
+      fs.existsSync.mockReturnValue(true);
+      fs.readJson.mockRejectedValue(new Error('File not found'));
+
+      const result = await manifest.load('/test/project');
+
+      // El código actual no maneja errores de readJson, así que debería lanzar el error
+      // Para este test, vamos a verificar que el error se maneja correctamente
+      expect(result).toBeNull();
+    });
+  });
+
+  describe('addAsset', () => {
+    it('should add asset to existing manifest', async () => {
+      const existingManifest = {
+        name: 'test-project',
+        template: 'web-pwa',
+        version: '1.0.0',
+        assets: []
+      };
+
+      const newAsset = {
+        name: 'components',
+        type: 'directory'
+      };
+
+      fs.readJson.mockResolvedValue(existingManifest);
+      fs.writeJson.mockResolvedValue();
+
+      await manifest.addAsset('/test/project', newAsset);
+
+      expect(fs.writeJson).toHaveBeenCalledWith(
+        path.join('/test/project', 'g360-manifest.json'),
+        expect.objectContaining({
+          assets: expect.arrayContaining([
+            expect.objectContaining({
+              name: 'components',
+              type: 'directory',
+              addedAt: expect.any(String)
+            })
+          ])
+        }),
+        { spaces: 2 }
+      );
+    });
+
+    it('should not add asset when manifest does not exist', async () => {
+      fs.readJson.mockResolvedValue(null);
+
+      await manifest.addAsset('/test/project', { name: 'test' });
+
+      expect(fs.writeJson).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('remove', () => {
+    it('should remove manifest file', async () => {
+      fs.existsSync.mockReturnValue(true);
+      fs.remove.mockResolvedValue();
+
+      await manifest.remove('/test/project');
+
+      expect(fs.remove).toHaveBeenCalledWith(
+        path.join('/test/project', 'g360-manifest.json')
+      );
+    });
+
+    it('should handle non-existing manifest gracefully', async () => {
+      fs.existsSync.mockReturnValue(false);
+
+      await manifest.remove('/test/project');
+
+      expect(fs.remove).not.toHaveBeenCalled();
+    });
+  });
+});

package/src/lib/validator.test.js

@@ -0,0 +1,115 @@
+/**
+ * @file validator.test.js
+ * @description Tests para el módulo validator
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import fs from 'fs-extra';
+import { validator } from '../lib/validator.js';
+
+// Mock de fs-extra
+vi.mock('fs-extra', () => ({
+  default: {
+    existsSync: vi.fn()
+  }
+}));
+
+describe('validator module', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  describe('isValidProjectName', () => {
+    it('should accept valid project names', () => {
+      expect(validator.isValidProjectName('my-project')).toBe(true);
+      expect(validator.isValidProjectName('project123')).toBe(true);
+      expect(validator.isValidProjectName('my-project-123')).toBe(true);
+      expect(validator.isValidProjectName('123project')).toBe(true); // Empieza con número es válido según la regex actual
+    });
+
+    it('should reject invalid project names', () => {
+      expect(validator.isValidProjectName('My-Project')).toBe(false); // Mayúsculas
+      expect(validator.isValidProjectName('my_project')).toBe(false); // Guiones bajos
+      expect(validator.isValidProjectName('my project')).toBe(false); // Espacios
+      expect(validator.isValidProjectName('-project')).toBe(false); // Empieza con guion
+      expect(validator.isValidProjectName('')).toBe(false); // Vacío
+    });
+  });
+
+  describe('isValidPath', () => {
+    it('should return true for existing paths', () => {
+      fs.existsSync.mockReturnValue(true);
+      expect(validator.isValidPath('/existing/path')).toBe(true);
+    });
+
+    it('should return false for non-existing paths', () => {
+      fs.existsSync.mockReturnValue(false);
+      expect(validator.isValidPath('/non/existing/path')).toBe(false);
+    });
+
+    it('should handle errors gracefully', () => {
+      fs.existsSync.mockImplementation(() => {
+        throw new Error('Permission denied');
+      });
+      expect(validator.isValidPath('/error/path')).toBe(false);
+    });
+  });
+
+  describe('validateProject', () => {
+    it('should validate a valid G360 project', () => {
+      fs.existsSync.mockImplementation((path) => {
+        // El directorio del proyecto existe
+        if (path === '/valid/project') return true;
+        // El manifest y el directorio g360 existen
+        if (path.includes('g360-manifest.json') || path.includes('g360')) return true;
+        return false;
+      });
+
+      const result = validator.validateProject('/valid/project');
+
+      expect(result.valid).toBe(true);
+      expect(result.errors).toHaveLength(0);
+      expect(result.warnings).toHaveLength(0);
+    });
+
+    it('should return errors for non-existing project', () => {
+      fs.existsSync.mockReturnValue(false);
+
+      const result = validator.validateProject('/non/existing');
+
+      expect(result.valid).toBe(false);
+      expect(result.errors).toContain('Project directory does not exist');
+    });
+
+    it('should return warnings for missing G360 files', () => {
+      fs.existsSync.mockImplementation((path) => {
+        // El directorio del proyecto existe
+        if (path === '/project/without/g360') return true;
+        // El manifest y el directorio g360 no existen
+        return false;
+      });
+
+      const result = validator.validateProject('/project/without/g360');
+
+      expect(result.valid).toBe(true);
+      expect(result.warnings).toContain('No g360-manifest.json found');
+      expect(result.warnings).toContain('No g360 directory found');
+    });
+
+    it('should handle partial G360 setup', () => {
+      fs.existsSync.mockImplementation((path) => {
+        // El directorio del proyecto existe
+        if (path === '/project/partial') return true;
+        // Solo el manifest existe
+        if (path.includes('g360-manifest.json')) return true;
+        // El directorio g360 no existe
+        return false;
+      });
+
+      const result = validator.validateProject('/project/partial');
+
+      expect(result.valid).toBe(true);
+      expect(result.warnings).toContain('No g360 directory found');
+    });
+  });
+});