create-fluxstack 1.0.13 → 1.0.15

package/core/plugins/__tests__/registry.test.ts

@@ -1,335 +0,0 @@
-/**
- * Tests for Plugin Registry
- */
-
-import { describe, it, expect, beforeEach, vi } from 'vitest'
-import { PluginRegistry } from '../registry'
-import type { Plugin, PluginManifest } from '../types'
-import type { Logger } from '../../utils/logger/index'
-
-// Mock logger
-const mockLogger: Logger = {
-  debug: vi.fn(),
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-  child: vi.fn(() => mockLogger),
-  time: vi.fn(),
-  timeEnd: vi.fn(),
-  request: vi.fn()
-}
-
-describe('PluginRegistry', () => {
-  let registry: PluginRegistry
-
-  beforeEach(() => {
-    registry = new PluginRegistry({ logger: mockLogger })
-    vi.clearAllMocks()
-  })
-
-  describe('Plugin Registration', () => {
-    it('should register a plugin successfully', async () => {
-      const plugin: Plugin = {
-        name: 'test-plugin',
-        version: '1.0.0'
-      }
-
-      await registry.register(plugin)
-      expect(registry.get('test-plugin')).toBe(plugin)
-      expect(registry.has('test-plugin')).toBe(true)
-    })
-
-    it('should register a plugin with manifest', async () => {
-      const plugin: Plugin = {
-        name: 'test-plugin',
-        version: '1.0.0'
-      }
-
-      const manifest: PluginManifest = {
-        name: 'test-plugin',
-        version: '1.0.0',
-        description: 'Test plugin',
-        author: 'Test Author',
-        license: 'MIT',
-        keywords: ['test'],
-        dependencies: {},
-        fluxstack: {
-          version: '1.0.0',
-          hooks: ['setup']
-        }
-      }
-
-      await registry.register(plugin, manifest)
-      expect(registry.getManifest('test-plugin')).toBe(manifest)
-    })
-
-    it('should throw error when registering duplicate plugin', async () => {
-      const plugin: Plugin = {
-        name: 'duplicate-plugin'
-      }
-
-      await registry.register(plugin)
-      await expect(registry.register(plugin)).rejects.toThrow("Plugin 'duplicate-plugin' is already registered")
-    })
-
-    it('should validate plugin structure', async () => {
-      const invalidPlugin = {
-        // Missing name property
-        version: '1.0.0'
-      } as Plugin
-
-      await expect(registry.register(invalidPlugin)).rejects.toThrow('Plugin must have a valid name property')
-    })
-
-    it('should unregister a plugin successfully', async () => {
-      const plugin: Plugin = {
-        name: 'removable-plugin'
-      }
-
-      await registry.register(plugin)
-      expect(registry.get('removable-plugin')).toBe(plugin)
-
-      registry.unregister('removable-plugin')
-      expect(registry.get('removable-plugin')).toBeUndefined()
-      expect(registry.has('removable-plugin')).toBe(false)
-    })
-
-    it('should throw error when unregistering non-existent plugin', () => {
-      expect(() => registry.unregister('non-existent')).toThrow("Plugin 'non-existent' is not registered")
-    })
-
-    it('should prevent unregistering plugin with dependents', async () => {
-      const pluginA: Plugin = { name: 'plugin-a' }
-      const pluginB: Plugin = { name: 'plugin-b', dependencies: ['plugin-a'] }
-
-      await registry.register(pluginA)
-      await registry.register(pluginB)
-
-      expect(() => registry.unregister('plugin-a')).toThrow(
-        "Cannot unregister plugin 'plugin-a' because it is required by: plugin-b"
-      )
-    })
-  })
-
-  describe('Plugin Retrieval', () => {
-    it('should get all registered plugins', async () => {
-      const plugin1: Plugin = { name: 'plugin-1' }
-      const plugin2: Plugin = { name: 'plugin-2' }
-
-      await registry.register(plugin1)
-      await registry.register(plugin2)
-
-      const allPlugins = registry.getAll()
-      expect(allPlugins).toHaveLength(2)
-      expect(allPlugins).toContain(plugin1)
-      expect(allPlugins).toContain(plugin2)
-    })
-
-    it('should return undefined for non-existent plugin', () => {
-      expect(registry.get('non-existent')).toBeUndefined()
-    })
-
-    it('should get plugin dependencies', async () => {
-      const plugin: Plugin = {
-        name: 'plugin-with-deps',
-        dependencies: ['dep1', 'dep2']
-      }
-
-      await registry.register(plugin)
-      expect(registry.getDependencies('plugin-with-deps')).toEqual(['dep1', 'dep2'])
-    })
-
-    it('should get plugin dependents', async () => {
-      const pluginA: Plugin = { name: 'plugin-a' }
-      const pluginB: Plugin = { name: 'plugin-b', dependencies: ['plugin-a'] }
-      const pluginC: Plugin = { name: 'plugin-c', dependencies: ['plugin-a'] }
-
-      await registry.register(pluginA)
-      await registry.register(pluginB)
-      await registry.register(pluginC)
-
-      const dependents = registry.getDependents('plugin-a')
-      expect(dependents).toContain('plugin-b')
-      expect(dependents).toContain('plugin-c')
-    })
-
-    it('should get registry statistics', async () => {
-      const plugin1: Plugin = { name: 'plugin-1' }
-      const plugin2: Plugin = { name: 'plugin-2' }
-
-      await registry.register(plugin1)
-      await registry.register(plugin2)
-
-      const stats = registry.getStats()
-      expect(stats.totalPlugins).toBe(2)
-      expect(stats.loadOrder).toBe(2)
-    })
-  })
-
-  describe('Dependency Management', () => {
-    it('should validate dependencies successfully', async () => {
-      const pluginA: Plugin = {
-        name: 'plugin-a'
-      }
-
-      const pluginB: Plugin = {
-        name: 'plugin-b',
-        dependencies: ['plugin-a']
-      }
-
-      await registry.register(pluginA)
-      await registry.register(pluginB)
-
-      expect(() => registry.validateDependencies()).not.toThrow()
-    })
-
-    it('should throw error for missing dependencies', async () => {
-      const pluginWithMissingDep: Plugin = {
-        name: 'plugin-with-missing-dep',
-        dependencies: ['non-existent-plugin']
-      }
-
-      await registry.register(pluginWithMissingDep)
-      expect(() => registry.validateDependencies()).toThrow(
-        "Plugin dependency validation failed"
-      )
-    })
-
-    it('should detect circular dependencies', async () => {
-      const pluginA: Plugin = {
-        name: 'plugin-a',
-        dependencies: ['plugin-b']
-      }
-
-      const pluginB: Plugin = {
-        name: 'plugin-b',
-        dependencies: ['plugin-a']
-      }
-
-      await registry.register(pluginA)
-      
-      await expect(registry.register(pluginB)).rejects.toThrow('Circular dependency detected')
-    })
-  })
-
-  describe('Load Order', () => {
-    it('should determine correct load order based on dependencies', async () => {
-      const pluginA: Plugin = {
-        name: 'plugin-a'
-      }
-
-      const pluginB: Plugin = {
-        name: 'plugin-b',
-        dependencies: ['plugin-a']
-      }
-
-      const pluginC: Plugin = {
-        name: 'plugin-c',
-        dependencies: ['plugin-b']
-      }
-
-      await registry.register(pluginA)
-      await registry.register(pluginB)
-      await registry.register(pluginC)
-
-      const loadOrder = registry.getLoadOrder()
-      
-      expect(loadOrder.indexOf('plugin-a')).toBeLessThan(loadOrder.indexOf('plugin-b'))
-      expect(loadOrder.indexOf('plugin-b')).toBeLessThan(loadOrder.indexOf('plugin-c'))
-    })
-
-    it('should respect plugin priorities', async () => {
-      const lowPriorityPlugin: Plugin = {
-        name: 'low-priority',
-        priority: 1
-      }
-
-      const highPriorityPlugin: Plugin = {
-        name: 'high-priority',
-        priority: 10
-      }
-
-      await registry.register(lowPriorityPlugin)
-      await registry.register(highPriorityPlugin)
-
-      const loadOrder = registry.getLoadOrder()
-      
-      expect(loadOrder.indexOf('high-priority')).toBeLessThan(loadOrder.indexOf('low-priority'))
-    })
-
-    it('should handle plugins without priorities', async () => {
-      const pluginWithoutPriority: Plugin = {
-        name: 'no-priority'
-      }
-
-      const pluginWithPriority: Plugin = {
-        name: 'with-priority',
-        priority: 5
-      }
-
-      await registry.register(pluginWithoutPriority)
-      await registry.register(pluginWithPriority)
-
-      const loadOrder = registry.getLoadOrder()
-      
-      expect(loadOrder.indexOf('with-priority')).toBeLessThan(loadOrder.indexOf('no-priority'))
-    })
-  })
-
-  describe('Plugin Discovery', () => {
-    it('should discover plugins from directories', async () => {
-      // This would require mocking the filesystem
-      // For now, just test that the method exists and returns an array
-      const results = await registry.discoverPlugins({
-        directories: ['non-existent-dir']
-      })
-      
-      expect(Array.isArray(results)).toBe(true)
-    })
-
-    it('should load plugin from path', async () => {
-      // This would require mocking the filesystem and import
-      // For now, just test that the method exists
-      const result = await registry.loadPlugin('non-existent-path')
-      
-      expect(result).toHaveProperty('success')
-      expect(result.success).toBe(false)
-      expect(result).toHaveProperty('error')
-    })
-  })
-
-  describe('Plugin Configuration', () => {
-    it('should validate plugin configuration', async () => {
-      const plugin: Plugin = {
-        name: 'config-plugin',
-        configSchema: {
-          type: 'object',
-          properties: {
-            apiKey: { type: 'string' }
-          },
-          required: ['apiKey']
-        }
-      }
-
-      const config = {
-        plugins: {
-          enabled: ['config-plugin'],
-          disabled: [],
-          config: {
-            'config-plugin': {
-              apiKey: 'test-key'
-            }
-          }
-        }
-      }
-
-      const registryWithConfig = new PluginRegistry({ 
-        logger: mockLogger,
-        config: config as any
-      })
-
-      await registryWithConfig.register(plugin)
-      expect(registryWithConfig.get('config-plugin')).toBe(plugin)
-    })
-  })
-})

package/core/utils/__tests__/errors.test.ts

@@ -1,139 +0,0 @@
-/**
- * Tests for Error Handling System
- */
-
-import { describe, it, expect } from 'vitest'
-import {
-  FluxStackError,
-  ValidationError,
-  NotFoundError,
-  UnauthorizedError,
-  ForbiddenError,
-  ConflictError,
-  InternalServerError,
-  ServiceUnavailableError
-} from '../errors'
-
-describe('Error Classes', () => {
-  describe('FluxStackError', () => {
-    it('should create error with all properties', () => {
-      const context = { field: 'email', value: 'invalid' }
-      const error = new FluxStackError('Test error', 'TEST_ERROR', 400, context)
-
-      expect(error.message).toBe('Test error')
-      expect(error.code).toBe('TEST_ERROR')
-      expect(error.statusCode).toBe(400)
-      expect(error.context).toBe(context)
-      expect(error.timestamp).toBeInstanceOf(Date)
-      expect(error.name).toBe('FluxStackError')
-    })
-
-    it('should default to status code 500', () => {
-      const error = new FluxStackError('Test error', 'TEST_ERROR')
-      expect(error.statusCode).toBe(500)
-    })
-
-    it('should serialize to JSON correctly', () => {
-      const error = new FluxStackError('Test error', 'TEST_ERROR', 400, { test: true })
-      const json = error.toJSON()
-
-      expect(json.name).toBe('FluxStackError')
-      expect(json.message).toBe('Test error')
-      expect(json.code).toBe('TEST_ERROR')
-      expect(json.statusCode).toBe(400)
-      expect(json.context).toEqual({ test: true })
-      expect(json.timestamp).toBeInstanceOf(Date)
-      expect(json.stack).toBeDefined()
-    })
-  })
-
-  describe('ValidationError', () => {
-    it('should create validation error with correct defaults', () => {
-      const error = new ValidationError('Invalid input')
-
-      expect(error.message).toBe('Invalid input')
-      expect(error.code).toBe('VALIDATION_ERROR')
-      expect(error.statusCode).toBe(400)
-      expect(error.name).toBe('ValidationError')
-    })
-
-    it('should include context', () => {
-      const context = { field: 'email', rule: 'required' }
-      const error = new ValidationError('Email is required', context)
-
-      expect(error.context).toBe(context)
-    })
-  })
-
-  describe('NotFoundError', () => {
-    it('should create not found error', () => {
-      const error = new NotFoundError('User')
-
-      expect(error.message).toBe('User not found')
-      expect(error.code).toBe('NOT_FOUND')
-      expect(error.statusCode).toBe(404)
-      expect(error.name).toBe('NotFoundError')
-    })
-  })
-
-  describe('UnauthorizedError', () => {
-    it('should create unauthorized error with default message', () => {
-      const error = new UnauthorizedError()
-
-      expect(error.message).toBe('Unauthorized')
-      expect(error.code).toBe('UNAUTHORIZED')
-      expect(error.statusCode).toBe(401)
-      expect(error.name).toBe('UnauthorizedError')
-    })
-
-    it('should create unauthorized error with custom message', () => {
-      const error = new UnauthorizedError('Invalid token')
-
-      expect(error.message).toBe('Invalid token')
-    })
-  })
-
-  describe('ForbiddenError', () => {
-    it('should create forbidden error', () => {
-      const error = new ForbiddenError('Access denied')
-
-      expect(error.message).toBe('Access denied')
-      expect(error.code).toBe('FORBIDDEN')
-      expect(error.statusCode).toBe(403)
-      expect(error.name).toBe('ForbiddenError')
-    })
-  })
-
-  describe('ConflictError', () => {
-    it('should create conflict error', () => {
-      const error = new ConflictError('Resource already exists')
-
-      expect(error.message).toBe('Resource already exists')
-      expect(error.code).toBe('CONFLICT')
-      expect(error.statusCode).toBe(409)
-      expect(error.name).toBe('ConflictError')
-    })
-  })
-
-  describe('InternalServerError', () => {
-    it('should create internal server error with default message', () => {
-      const error = new InternalServerError()
-
-      expect(error.message).toBe('Internal server error')
-      expect(error.code).toBe('INTERNAL_SERVER_ERROR')
-      expect(error.statusCode).toBe(500)
-      expect(error.name).toBe('InternalServerError')
-    })
-  })
-
-  describe('ServiceUnavailableError', () => {
-    it('should create service unavailable error', () => {
-      const error = new ServiceUnavailableError('Database is down')
-
-      expect(error.message).toBe('Database is down')
-      expect(error.code).toBe('SERVICE_UNAVAILABLE')
-      expect(error.statusCode).toBe(503)
-      expect(error.name).toBe('ServiceUnavailableError')
-    })
-  })
-})