gh-here 3.0.2 → 3.1.0

package/tests/httpService.test.js

@@ -1,268 +0,0 @@
-// Tests for HTTP Service abstraction
-// Run with: node tests/httpService.test.js
-
-// Simple mock implementation for jest.fn()
-function jest_fn() {
-  const calls = [];
-  const mockFn = (...args) => {
-    calls.push(args);
-    return mockFn._returnValue;
-  };
-  mockFn.mock = { calls };
-  mockFn.mockResolvedValue = (value) => {
-    mockFn._returnValue = Promise.resolve(value);
-    return mockFn;
-  };
-  return mockFn;
-}
-
-// Replace jest with our simple implementation
-const jest = { fn: jest_fn };
-
-// Mock fetch for testing
-class MockResponse {
-  constructor(data, options = {}) {
-    this._data = data;
-    this.ok = options.ok !== false;
-    this.status = options.status || (this.ok ? 200 : 500);
-    this.statusText = options.statusText || (this.ok ? 'OK' : 'Internal Server Error');
-  }
-
-  async json() {
-    return this._data;
-  }
-
-  async text() {
-    return typeof this._data === 'string' ? this._data : JSON.stringify(this._data);
-  }
-}
-
-// Mock fetch implementation
-function createMockFetch(responseData, options = {}) {
-  return jest.fn().mockResolvedValue(new MockResponse(responseData, options));
-}
-
-// HTTP Service abstraction - pure functions, easily testable
-function createHttpService(fetchFn = fetch) {
-  return {
-    // File operations
-    async getFileContent(filePath) {
-      const response = await fetchFn(`/api/file-content?path=${encodeURIComponent(filePath)}`);
-      if (!response.ok) {
-        throw new Error(`HTTP ${response.status}: ${response.statusText}`);
-      }
-      return response.text();
-    },
-
-    async saveFile(filePath, content) {
-      const response = await fetchFn('/api/save-file', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ path: filePath, content })
-      });
-      return response.json();
-    },
-
-    async createFile(path, filename) {
-      const response = await fetchFn('/api/create-file', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ path, filename })
-      });
-      return response.json();
-    },
-
-    async deleteFile(path) {
-      const response = await fetchFn('/api/delete', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ path })
-      });
-      return response.json();
-    },
-
-    async renameFile(oldPath, newPath) {
-      const response = await fetchFn('/api/rename', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ oldPath, newPath })
-      });
-      return response.json();
-    },
-
-    // Folder operations
-    async createFolder(path, foldername) {
-      const response = await fetchFn('/api/create-folder', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ path, foldername })
-      });
-      return response.json();
-    },
-
-    // Git operations
-    async getGitStatus(currentPath) {
-      const response = await fetchFn(`/api/git-status?currentPath=${encodeURIComponent(currentPath)}`);
-      return response.json();
-    },
-
-    async getGitDiff(filePath) {
-      const response = await fetchFn(`/api/git-diff?path=${encodeURIComponent(filePath)}`);
-      return response.json();
-    },
-
-    async commitSelectedFiles(files, message) {
-      const response = await fetchFn('/api/git-commit-selected', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ files, message })
-      });
-      return response.json();
-    }
-  };
-}
-
-// Simple test framework (since we don't have a real test runner)
-function test(name, fn) {
-  try {
-    fn();
-    console.log(`✅ ${name}`);
-  } catch (error) {
-    console.log(`❌ ${name}: ${error.message}`);
-  }
-}
-
-function assertEqual(actual, expected, message = '') {
-  if (actual !== expected) {
-    throw new Error(`Expected "${expected}" but got "${actual}". ${message}`);
-  }
-}
-
-function assertDeepEqual(actual, expected, message = '') {
-  const actualStr = JSON.stringify(actual);
-  const expectedStr = JSON.stringify(expected);
-  if (actualStr !== expectedStr) {
-    throw new Error(`Expected ${expectedStr} but got ${actualStr}. ${message}`);
-  }
-}
-
-// Test cases
-console.log('🧪 Testing HTTP Service\n');
-
-test('gets file content correctly', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse('file content'));
-  const httpService = createHttpService(mockFetch);
-  
-  const content = await httpService.getFileContent('src/app.js');
-  assertEqual(content, 'file content');
-  assertEqual(mockFetch.mock.calls[0][0], '/api/file-content?path=src%2Fapp.js');
-});
-
-test('saves file with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.saveFile('src/app.js', 'new content');
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/save-file');
-  assertEqual(options.method, 'POST');
-  assertDeepEqual(JSON.parse(options.body), { path: 'src/app.js', content: 'new content' });
-});
-
-test('creates file with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.createFile('src', 'newfile.js');
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/create-file');
-  assertDeepEqual(JSON.parse(options.body), { path: 'src', filename: 'newfile.js' });
-});
-
-test('deletes file with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.deleteFile('src/oldfile.js');
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/delete');
-  assertDeepEqual(JSON.parse(options.body), { path: 'src/oldfile.js' });
-});
-
-test('renames file with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.renameFile('src/old.js', 'src/new.js');
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/rename');
-  assertDeepEqual(JSON.parse(options.body), { oldPath: 'src/old.js', newPath: 'src/new.js' });
-});
-
-test('creates folder with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.createFolder('src', 'components');
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/create-folder');
-  assertDeepEqual(JSON.parse(options.body), { path: 'src', foldername: 'components' });
-});
-
-test('gets git status with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ files: [] }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.getGitStatus('src');
-  assertDeepEqual(result, { files: [] });
-  
-  assertEqual(mockFetch.mock.calls[0][0], '/api/git-status?currentPath=src');
-});
-
-test('gets git diff with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ diff: 'diff content' }));
-  const httpService = createHttpService(mockFetch);
-  
-  const result = await httpService.getGitDiff('src/app.js');
-  assertDeepEqual(result, { diff: 'diff content' });
-  
-  assertEqual(mockFetch.mock.calls[0][0], '/api/git-diff?path=src%2Fapp.js');
-});
-
-test('commits selected files with correct parameters', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse({ success: true }));
-  const httpService = createHttpService(mockFetch);
-  
-  const files = ['src/app.js', 'src/utils.js'];
-  const message = 'Fix bugs';
-  const result = await httpService.commitSelectedFiles(files, message);
-  assertDeepEqual(result, { success: true });
-  
-  const [url, options] = mockFetch.mock.calls[0];
-  assertEqual(url, '/api/git-commit-selected');
-  assertDeepEqual(JSON.parse(options.body), { files, message });
-});
-
-test('handles HTTP errors correctly', async () => {
-  const mockFetch = jest.fn().mockResolvedValue(new MockResponse('Not found', { ok: false, status: 404, statusText: 'Not Found' }));
-  const httpService = createHttpService(mockFetch);
-  
-  try {
-    await httpService.getFileContent('nonexistent.js');
-    throw new Error('Should have thrown an error');
-  } catch (error) {
-    assertEqual(error.message, 'HTTP 404: Not Found');
-  }
-});
-
-console.log('\n🎉 HTTP Service tests complete!');

package/tests/languageDetection.test.js

@@ -1,145 +0,0 @@
-// Simple unit tests for language detection
-// Run with: node tests/languageDetection.test.js
-
-// Import the function (in a real setup, you'd use proper modules)
-// For now, we'll copy the function to test it standalone
-
-function getLanguageFromExtension(filename) {
-  const ext = filename.split('.').pop().toLowerCase();
-  const languageMap = {
-    // JavaScript family
-    'js': 'javascript',
-    'mjs': 'javascript', 
-    'jsx': 'javascript',
-    'ts': 'typescript',
-    'tsx': 'typescript',
-    
-    // Web languages
-    'html': 'html',
-    'htm': 'html',
-    'css': 'css',
-    'scss': 'scss',
-    'sass': 'sass',
-    'less': 'less',
-    
-    // Data formats
-    'json': 'json',
-    'xml': 'xml',
-    'yaml': 'yaml',
-    'yml': 'yaml',
-    
-    // Programming languages
-    'py': 'python',
-    'java': 'java',
-    'go': 'go',
-    'rs': 'rust',
-    'php': 'php',
-    'rb': 'ruby',
-    'swift': 'swift',
-    'kt': 'kotlin',
-    'dart': 'dart',
-    
-    // Systems languages
-    'c': 'c',
-    'cpp': 'cpp',
-    'cc': 'cpp',
-    'cxx': 'cpp',
-    'h': 'c',
-    'hpp': 'cpp',
-    
-    // Shell and scripts
-    'sh': 'shell',
-    'bash': 'shell',
-    'zsh': 'shell',
-    'fish': 'shell',
-    'ps1': 'powershell',
-    
-    // Other languages
-    'sql': 'sql',
-    'r': 'r',
-    'scala': 'scala',
-    'clj': 'clojure',
-    'lua': 'lua',
-    'pl': 'perl',
-    'groovy': 'groovy',
-    
-    // Config and text
-    'md': 'markdown',
-    'txt': 'plaintext',
-    'log': 'plaintext'
-  };
-  
-  // Special filename handling
-  const basename = filename.toLowerCase();
-  if (basename === 'dockerfile' || basename.startsWith('dockerfile.')) return 'dockerfile';
-  if (basename === 'makefile') return 'makefile';
-  if (basename.startsWith('.env')) return 'dotenv';
-  if (basename === 'package.json' || basename === 'composer.json') return 'json';
-  
-  return languageMap[ext] || 'plaintext';
-}
-
-// Simple test framework
-function test(name, fn) {
-  try {
-    fn();
-    console.log(`✅ ${name}`);
-  } catch (error) {
-    console.log(`❌ ${name}: ${error.message}`);
-  }
-}
-
-function assertEqual(actual, expected, message = '') {
-  if (actual !== expected) {
-    throw new Error(`Expected "${expected}" but got "${actual}". ${message}`);
-  }
-}
-
-// Test cases
-console.log('🧪 Testing Language Detection Function\n');
-
-test('detects JavaScript files', () => {
-  assertEqual(getLanguageFromExtension('app.js'), 'javascript');
-  assertEqual(getLanguageFromExtension('module.mjs'), 'javascript');
-  assertEqual(getLanguageFromExtension('Component.jsx'), 'javascript');
-});
-
-test('detects TypeScript files', () => {
-  assertEqual(getLanguageFromExtension('main.ts'), 'typescript');
-  assertEqual(getLanguageFromExtension('Component.tsx'), 'typescript');
-});
-
-test('detects web languages', () => {
-  assertEqual(getLanguageFromExtension('index.html'), 'html');
-  assertEqual(getLanguageFromExtension('styles.css'), 'css');
-  assertEqual(getLanguageFromExtension('styles.scss'), 'scss');
-});
-
-test('detects programming languages', () => {
-  assertEqual(getLanguageFromExtension('script.py'), 'python');
-  assertEqual(getLanguageFromExtension('Main.java'), 'java');
-  assertEqual(getLanguageFromExtension('main.go'), 'go');
-  assertEqual(getLanguageFromExtension('lib.rs'), 'rust');
-});
-
-test('detects special filenames', () => {
-  assertEqual(getLanguageFromExtension('Dockerfile'), 'dockerfile');
-  assertEqual(getLanguageFromExtension('dockerfile.prod'), 'dockerfile');
-  assertEqual(getLanguageFromExtension('Makefile'), 'makefile');
-  assertEqual(getLanguageFromExtension('.env'), 'dotenv');
-  assertEqual(getLanguageFromExtension('.env.local'), 'dotenv');
-  assertEqual(getLanguageFromExtension('package.json'), 'json');
-});
-
-test('defaults to plaintext for unknown extensions', () => {
-  assertEqual(getLanguageFromExtension('file.unknown'), 'plaintext');
-  assertEqual(getLanguageFromExtension('README'), 'plaintext');
-  assertEqual(getLanguageFromExtension('file.xyz'), 'plaintext');
-});
-
-test('handles files without extensions', () => {
-  assertEqual(getLanguageFromExtension('README'), 'plaintext');
-  assertEqual(getLanguageFromExtension('LICENSE'), 'plaintext');
-});
-
-console.log('\n🎉 Language detection tests complete!');

package/tests/pathUtils.test.js

@@ -1,136 +0,0 @@
-// Tests for Path Utilities
-// Run with: node tests/pathUtils.test.js
-
-// Path utility functions extracted for testability
-const PathUtils = {
-  // Extract current path from URL parameters
-  getCurrentPath() {
-    const currentUrl = new URL(window.location.href);
-    return currentUrl.searchParams.get('path') || '';
-  },
-
-  // Navigate to parent directory
-  getParentPath(currentPath) {
-    if (!currentPath || currentPath === '') {
-      return null; // Already at root
-    }
-    
-    const pathParts = currentPath.split('/').filter(p => p);
-    if (pathParts.length === 0) {
-      return null; // Already at root
-    }
-    
-    pathParts.pop();
-    return pathParts.join('/');
-  },
-
-  // Build file path from directory and filename
-  buildFilePath(currentPath, filename) {
-    return currentPath ? `${currentPath}/${filename}` : filename;
-  },
-
-  // Get filename from full path
-  getFileName(filePath) {
-    return filePath.split('/').pop() || 'file.txt';
-  },
-
-  // Build URL with encoded path parameter
-  buildPathUrl(basePath, targetPath) {
-    return targetPath ? `${basePath}?path=${encodeURIComponent(targetPath)}` : basePath;
-  },
-
-  // Extract directory path from file path
-  getDirectoryPath(filePath) {
-    const parts = filePath.split('/').filter(p => p);
-    if (parts.length <= 1) return '';
-    return parts.slice(0, -1).join('/');
-  }
-};
-
-// Simple test framework
-function test(name, fn) {
-  try {
-    fn();
-    console.log(`✅ ${name}`);
-  } catch (error) {
-    console.log(`❌ ${name}: ${error.message}`);
-  }
-}
-
-function assertEqual(actual, expected, message = '') {
-  if (actual !== expected) {
-    throw new Error(`Expected "${expected}" but got "${actual}". ${message}`);
-  }
-}
-
-function assertNull(actual, message = '') {
-  if (actual !== null) {
-    throw new Error(`Expected null but got "${actual}". ${message}`);
-  }
-}
-
-// Test cases
-console.log('🧪 Testing Path Utilities\n');
-
-test('gets parent path correctly', () => {
-  assertEqual(PathUtils.getParentPath('src/components/App.js'), 'src/components');
-  assertEqual(PathUtils.getParentPath('src/App.js'), 'src');
-  assertEqual(PathUtils.getParentPath('App.js'), '');
-  assertNull(PathUtils.getParentPath(''));
-  assertNull(PathUtils.getParentPath(null));
-});
-
-test('builds file paths correctly', () => {
-  assertEqual(PathUtils.buildFilePath('src/components', 'App.js'), 'src/components/App.js');
-  assertEqual(PathUtils.buildFilePath('src', 'App.js'), 'src/App.js');
-  assertEqual(PathUtils.buildFilePath('', 'App.js'), 'App.js');
-  assertEqual(PathUtils.buildFilePath(null, 'App.js'), 'App.js');
-});
-
-test('gets filename from path correctly', () => {
-  assertEqual(PathUtils.getFileName('src/components/App.js'), 'App.js');
-  assertEqual(PathUtils.getFileName('App.js'), 'App.js');
-  assertEqual(PathUtils.getFileName(''), 'file.txt');
-  assertEqual(PathUtils.getFileName('src/components/'), 'file.txt');
-});
-
-test('builds URLs with path parameters', () => {
-  assertEqual(PathUtils.buildPathUrl('/', 'src/components'), '/?path=src%2Fcomponents');
-  assertEqual(PathUtils.buildPathUrl('/new', 'src'), '/new?path=src');
-  assertEqual(PathUtils.buildPathUrl('/', ''), '/');
-  assertEqual(PathUtils.buildPathUrl('/new', null), '/new');
-});
-
-test('gets directory path from file path', () => {
-  assertEqual(PathUtils.getDirectoryPath('src/components/App.js'), 'src/components');
-  assertEqual(PathUtils.getDirectoryPath('src/App.js'), 'src');
-  assertEqual(PathUtils.getDirectoryPath('App.js'), '');
-  assertEqual(PathUtils.getDirectoryPath(''), '');
-});
-
-test('handles git commit dialog scenarios correctly', () => {
-  // Test cases that match the commit dialog usage pattern
-  // where we extract directory from file.name
-  assertEqual(PathUtils.getDirectoryPath('src/components/Button.tsx'), 'src/components');
-  assertEqual(PathUtils.getDirectoryPath('lib/utils/helper.js'), 'lib/utils');
-  assertEqual(PathUtils.getDirectoryPath('tests/unit/test.js'), 'tests/unit');
-  assertEqual(PathUtils.getDirectoryPath('package.json'), ''); // Root level file
-  assertEqual(PathUtils.getDirectoryPath('docs/README.md'), 'docs');
-  
-  // Edge cases from commit dialog
-  assertEqual(PathUtils.getDirectoryPath('very/deeply/nested/folder/structure/file.txt'), 'very/deeply/nested/folder/structure');
-  assertEqual(PathUtils.getDirectoryPath('single-level.txt'), '');
-});
-
-test('handles edge cases in parent path', () => {
-  assertEqual(PathUtils.getParentPath('a/b/c/d/e'), 'a/b/c/d');
-  assertEqual(PathUtils.getParentPath('single'), '');
-  assertNull(PathUtils.getParentPath(''));
-});
-
-test('handles edge cases in file path building', () => {
-  assertEqual(PathUtils.buildFilePath('a/b/c', 'file.txt'), 'a/b/c/file.txt');
-  assertEqual(PathUtils.buildFilePath('', 'README.md'), 'README.md');
-});
-
-console.log('\n🎉 Path utilities tests complete!');