@aaronshaf/confluence-cli 0.1.15 → 1.0.0

package/src/test/file-rename.test.ts

@@ -1,305 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
-import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { handleFileRename } from '../cli/commands/file-rename.js';
-
-describe('handleFileRename', () => {
-  const testDir = join(import.meta.dir, '.test-file-rename');
-
-  beforeEach(() => {
-    // Clean up and create test directory
-    if (existsSync(testDir)) {
-      rmSync(testDir, { recursive: true });
-    }
-    mkdirSync(testDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    // Clean up test directory
-    if (existsSync(testDir)) {
-      rmSync(testDir, { recursive: true });
-    }
-  });
-
-  test('does not rename README.md even with different title', () => {
-    const filePath = join(testDir, 'README.md');
-    const content = `---
-title: "Engineering Wiki"
-page_id: page-123
----
-
-# Engineering Wiki
-
-Welcome to the wiki.
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'README.md', 'Engineering Wiki', content);
-
-    expect(result.wasRenamed).toBe(false);
-    expect(result.finalPath).toBe('README.md');
-    expect(existsSync(filePath)).toBe(true);
-    expect(readFileSync(filePath, 'utf-8')).toBe(content);
-  });
-
-  test('does not rename readme.md (lowercase) even with different title', () => {
-    const filePath = join(testDir, 'readme.md');
-    const content = `---
-title: "My Custom Title"
-page_id: page-456
----
-
-# My Custom Title
-
-Content here.
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'readme.md', 'My Custom Title', content);
-
-    expect(result.wasRenamed).toBe(false);
-    expect(result.finalPath).toBe('readme.md');
-    expect(existsSync(filePath)).toBe(true);
-  });
-
-  test('does not rename index.md even with different title', () => {
-    const filePath = join(testDir, 'index.md');
-    const content = `---
-title: "Home Page"
-page_id: page-789
----
-
-# Home Page
-
-Welcome.
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'index.md', 'Home Page', content);
-
-    expect(result.wasRenamed).toBe(false);
-    expect(result.finalPath).toBe('index.md');
-    expect(existsSync(filePath)).toBe(true);
-  });
-
-  test('renames regular file when title changes', () => {
-    const filePath = join(testDir, 'old-name.md');
-    const content = `---
-title: "New Name"
-page_id: page-101
----
-
-# New Name
-
-Content.
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'old-name.md', 'New Name', content);
-
-    expect(result.wasRenamed).toBe(true);
-    expect(result.finalPath).toBe('new-name.md');
-    expect(existsSync(join(testDir, 'new-name.md'))).toBe(true);
-    expect(existsSync(filePath)).toBe(false);
-  });
-
-  test('does not rename if target file already exists', () => {
-    const filePath = join(testDir, 'old-name.md');
-    const existingPath = join(testDir, 'new-name.md');
-    const content = `---
-title: "New Name"
----
-
-# New Name
-`;
-    writeFileSync(filePath, content);
-    writeFileSync(existingPath, 'existing content');
-
-    const result = handleFileRename(filePath, 'old-name.md', 'New Name', content);
-
-    expect(result.wasRenamed).toBe(false);
-    expect(result.finalPath).toBe('old-name.md');
-    expect(existsSync(filePath)).toBe(true);
-  });
-
-  test('preserves README.md in subdirectory', () => {
-    const subDir = join(testDir, 'docs');
-    mkdirSync(subDir);
-    const filePath = join(subDir, 'README.md');
-    const content = `---
-title: "Documentation Overview"
-page_id: page-docs
----
-
-# Documentation Overview
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'docs/README.md', 'Documentation Overview', content);
-
-    expect(result.wasRenamed).toBe(false);
-    expect(result.finalPath).toBe('docs/README.md');
-    expect(existsSync(filePath)).toBe(true);
-  });
-
-  test('handles file in subdirectory with rename', () => {
-    const subDir = join(testDir, 'guides');
-    mkdirSync(subDir);
-    const filePath = join(subDir, 'old-guide.md');
-    const content = `---
-title: "New Guide Name"
----
-
-# New Guide Name
-`;
-    writeFileSync(filePath, content);
-
-    const result = handleFileRename(filePath, 'guides/old-guide.md', 'New Guide Name', content);
-
-    expect(result.wasRenamed).toBe(true);
-    expect(result.finalPath).toBe('guides/new-guide-name.md');
-    expect(existsSync(join(subDir, 'new-guide-name.md'))).toBe(true);
-  });
-
-  test('updates references in other files when renamed with spaceRoot', () => {
-    // Create a file that links to another file
-    const linkingFile = join(testDir, 'index.md');
-    writeFileSync(
-      linkingFile,
-      `---
-title: Index
----
-
-See [Old Guide](./old-guide.md) for details.
-`,
-    );
-
-    // Create the file that will be renamed
-    const filePath = join(testDir, 'old-guide.md');
-    const content = `---
-title: "New Guide Name"
----
-
-# New Guide Name
-`;
-    writeFileSync(filePath, content);
-
-    // Rename with spaceRoot to trigger reference updates
-    const result = handleFileRename(filePath, 'old-guide.md', 'New Guide Name', content, testDir);
-
-    expect(result.wasRenamed).toBe(true);
-    expect(result.finalPath).toBe('new-guide-name.md');
-
-    // Verify the link in the other file was updated
-    const linkingContent = readFileSync(linkingFile, 'utf-8');
-    expect(linkingContent).toContain('./new-guide-name.md');
-    expect(linkingContent).not.toContain('./old-guide.md');
-  });
-
-  test('updates references without ./ prefix when renamed with spaceRoot', () => {
-    // Create a file that links without ./ prefix
-    const linkingFile = join(testDir, 'README.md');
-    writeFileSync(
-      linkingFile,
-      `---
-title: README
----
-
-See [Guide](old-guide.md) for details.
-`,
-    );
-
-    // Create the file that will be renamed
-    const filePath = join(testDir, 'old-guide.md');
-    const content = `---
-title: "New Guide Name"
----
-
-# New Guide Name
-`;
-    writeFileSync(filePath, content);
-
-    // Rename with spaceRoot to trigger reference updates
-    const result = handleFileRename(filePath, 'old-guide.md', 'New Guide Name', content, testDir);
-
-    expect(result.wasRenamed).toBe(true);
-
-    // Verify the link was updated (preserving no-prefix style)
-    const linkingContent = readFileSync(linkingFile, 'utf-8');
-    expect(linkingContent).toContain('new-guide-name.md');
-    expect(linkingContent).not.toContain('old-guide.md');
-  });
-
-  test('updates references in subdirectory files when renamed with spaceRoot', () => {
-    // Create subdirectory
-    const docsDir = join(testDir, 'docs');
-    mkdirSync(docsDir);
-
-    // Create a file in subdirectory that links to root file
-    const linkingFile = join(docsDir, 'guide.md');
-    writeFileSync(
-      linkingFile,
-      `---
-title: Guide
----
-
-See [Overview](../overview.md) for details.
-`,
-    );
-
-    // Create the file that will be renamed (at root)
-    const filePath = join(testDir, 'overview.md');
-    const content = `---
-title: "Project Overview"
----
-
-# Project Overview
-`;
-    writeFileSync(filePath, content);
-
-    // Rename with spaceRoot to trigger reference updates
-    const result = handleFileRename(filePath, 'overview.md', 'Project Overview', content, testDir);
-
-    expect(result.wasRenamed).toBe(true);
-    expect(result.finalPath).toBe('project-overview.md');
-
-    // Verify the relative link in subdirectory file was updated
-    const linkingContent = readFileSync(linkingFile, 'utf-8');
-    expect(linkingContent).toContain('../project-overview.md');
-    expect(linkingContent).not.toContain('../overview.md');
-  });
-
-  test('does not update references when no spaceRoot provided', () => {
-    // Create a file that links to another file
-    const linkingFile = join(testDir, 'index.md');
-    writeFileSync(
-      linkingFile,
-      `---
-title: Index
----
-
-See [Old Guide](./old-guide.md) for details.
-`,
-    );
-
-    // Create the file that will be renamed
-    const filePath = join(testDir, 'old-guide.md');
-    const content = `---
-title: "New Guide Name"
----
-
-# New Guide Name
-`;
-    writeFileSync(filePath, content);
-
-    // Rename WITHOUT spaceRoot - should not update references
-    const result = handleFileRename(filePath, 'old-guide.md', 'New Guide Name', content);
-
-    expect(result.wasRenamed).toBe(true);
-
-    // Verify the link in the other file was NOT updated (no spaceRoot)
-    const linkingContent = readFileSync(linkingFile, 'utf-8');
-    expect(linkingContent).toContain('./old-guide.md');
-    expect(linkingContent).not.toContain('./new-guide-name.md');
-  });
-});

package/src/test/folder-hierarchy.test.ts

@@ -1,337 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
-import { existsSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { HttpResponse, http } from 'msw';
-import { ConfluenceClient } from '../lib/confluence-client/client.js';
-import {
-  determineExpectedParent,
-  ensureFolderHierarchy,
-  FolderHierarchyError,
-  sanitizeFolderTitle,
-} from '../cli/commands/folder-hierarchy.js';
-import type { SpaceConfigWithState } from '../lib/space-config.js';
-import { server } from './setup-msw.js';
-import { createValidFolder } from './msw-schema-validation.js';
-
-const testConfig = {
-  confluenceUrl: 'https://test.atlassian.net',
-  email: 'test@example.com',
-  apiToken: 'test-token',
-};
-
-describe('folder-hierarchy', () => {
-  describe('sanitizeFolderTitle', () => {
-    test('returns unchanged title for valid names', () => {
-      const result = sanitizeFolderTitle('My Folder');
-      expect(result.sanitized).toBe('My Folder');
-      expect(result.wasModified).toBe(false);
-    });
-
-    test('replaces pipe character', () => {
-      const result = sanitizeFolderTitle('Folder | Name');
-      expect(result.sanitized).toBe('Folder - Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces backslash', () => {
-      const result = sanitizeFolderTitle('Folder\\Name');
-      expect(result.sanitized).toBe('Folder-Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces forward slash', () => {
-      const result = sanitizeFolderTitle('Folder/Name');
-      expect(result.sanitized).toBe('Folder-Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces colon', () => {
-      const result = sanitizeFolderTitle('Folder: Name');
-      expect(result.sanitized).toBe('Folder- Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces asterisk', () => {
-      const result = sanitizeFolderTitle('Folder*Name');
-      expect(result.sanitized).toBe('Folder-Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces question mark', () => {
-      const result = sanitizeFolderTitle('Folder?Name');
-      expect(result.sanitized).toBe('Folder-Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces double quotes', () => {
-      const result = sanitizeFolderTitle('Folder"Name');
-      expect(result.sanitized).toBe('Folder-Name');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces angle brackets', () => {
-      const result = sanitizeFolderTitle('Folder<Name>');
-      expect(result.sanitized).toBe('Folder-Name-');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('replaces multiple invalid characters', () => {
-      const result = sanitizeFolderTitle('A|B/C:D*E?F"G<H>I');
-      expect(result.sanitized).toBe('A-B-C-D-E-F-G-H-I');
-      expect(result.wasModified).toBe(true);
-    });
-
-    test('trims whitespace', () => {
-      const result = sanitizeFolderTitle('  Folder Name  ');
-      expect(result.sanitized).toBe('Folder Name');
-      expect(result.wasModified).toBe(true);
-    });
-  });
-
-  describe('ensureFolderHierarchy', () => {
-    let testDir: string;
-
-    beforeEach(() => {
-      testDir = mkdtempSync(join(tmpdir(), 'cn-folder-hierarchy-test-'));
-    });
-
-    afterEach(() => {
-      if (existsSync(testDir)) {
-        rmSync(testDir, { recursive: true });
-      }
-    });
-
-    const createSpaceConfig = (overrides?: Partial<SpaceConfigWithState>): SpaceConfigWithState => ({
-      spaceKey: 'TEST',
-      spaceId: 'space-123',
-      spaceName: 'Test Space',
-      pages: {},
-      folders: {},
-      ...overrides,
-    });
-
-    test('returns undefined parentId for root-level files', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      const result = await ensureFolderHierarchy(client, spaceConfig, testDir, 'readme.md', true);
-
-      expect(result.parentId).toBeUndefined();
-      expect(result.updatedConfig).toEqual(spaceConfig);
-    });
-
-    test('returns undefined parentId for files with ./ prefix at root', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      const result = await ensureFolderHierarchy(client, spaceConfig, testDir, './readme.md', true);
-
-      expect(result.parentId).toBeUndefined();
-    });
-
-    test('returns existing folder ID from local config', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig({
-        folders: {
-          'folder-abc': {
-            folderId: 'folder-abc',
-            title: 'docs',
-            localPath: 'docs',
-          },
-        },
-      });
-
-      const result = await ensureFolderHierarchy(client, spaceConfig, testDir, 'docs/guide.md', true);
-
-      expect(result.parentId).toBe('folder-abc');
-    });
-
-    test('returns nested folder ID from local config', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig({
-        folders: {
-          'folder-docs': {
-            folderId: 'folder-docs',
-            title: 'docs',
-            localPath: 'docs',
-          },
-          'folder-api': {
-            folderId: 'folder-api',
-            title: 'api',
-            parentId: 'folder-docs',
-            localPath: 'docs/api',
-          },
-        },
-      });
-
-      const result = await ensureFolderHierarchy(client, spaceConfig, testDir, 'docs/api/endpoints.md', true);
-
-      expect(result.parentId).toBe('folder-api');
-    });
-
-    test('throws error for path traversal attempts', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      try {
-        await ensureFolderHierarchy(client, spaceConfig, testDir, '../etc/passwd', true);
-        expect.unreachable('Should have thrown');
-      } catch (error) {
-        expect(error).toBeInstanceOf(FolderHierarchyError);
-        expect((error as FolderHierarchyError).message).toContain('path traversal');
-      }
-    });
-
-    test('throws error for deeply nested paths exceeding max depth', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      // Create a path with 11 levels (exceeds MAX_FOLDER_DEPTH of 10)
-      const deepPath = 'a/b/c/d/e/f/g/h/i/j/k/file.md';
-
-      try {
-        await ensureFolderHierarchy(client, spaceConfig, testDir, deepPath, true);
-        expect.unreachable('Should have thrown');
-      } catch (error) {
-        expect(error).toBeInstanceOf(FolderHierarchyError);
-        expect((error as FolderHierarchyError).message).toContain('too deep');
-      }
-    });
-
-    test('dry run returns undefined when folder needs creation', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      // Mock: folder does not exist on Confluence
-      server.use(
-        http.get('*/wiki/api/v2/spaces/space-123/folders', () => {
-          return HttpResponse.json({ results: [] });
-        }),
-      );
-
-      const result = await ensureFolderHierarchy(client, spaceConfig, testDir, 'newdir/file.md', true);
-
-      // Dry run can't continue without a real folder ID
-      expect(result.parentId).toBeUndefined();
-    });
-  });
-
-  describe('determineExpectedParent', () => {
-    let testDir: string;
-
-    beforeEach(() => {
-      testDir = mkdtempSync(join(tmpdir(), 'cn-determine-parent-test-'));
-    });
-
-    afterEach(() => {
-      if (existsSync(testDir)) {
-        rmSync(testDir, { recursive: true });
-      }
-    });
-
-    const createSpaceConfig = (overrides?: Partial<SpaceConfigWithState>): SpaceConfigWithState => ({
-      spaceKey: 'TEST',
-      spaceId: 'space-123',
-      spaceName: 'Test Space',
-      pages: {},
-      folders: {},
-      ...overrides,
-    });
-
-    test('returns undefined parentId for root-level files', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      const result = await determineExpectedParent(client, spaceConfig, testDir, 'readme.md', true);
-
-      expect(result.parentId).toBeUndefined();
-      expect(result.updatedConfig).toEqual(spaceConfig);
-    });
-
-    test('returns undefined for files with . directory', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      // This tests the early return for dirPath === '.'
-      const result = await determineExpectedParent(client, spaceConfig, testDir, './file.md', true);
-
-      expect(result.parentId).toBeUndefined();
-    });
-
-    test('returns existing folder ID for nested file', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig({
-        folders: {
-          'folder-xyz': {
-            folderId: 'folder-xyz',
-            title: 'docs',
-            localPath: 'docs',
-          },
-        },
-      });
-
-      const result = await determineExpectedParent(client, spaceConfig, testDir, 'docs/page.md', true);
-
-      expect(result.parentId).toBe('folder-xyz');
-    });
-
-    test('returns deeply nested folder ID', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig({
-        folders: {
-          'folder-docs': {
-            folderId: 'folder-docs',
-            title: 'docs',
-            localPath: 'docs',
-          },
-          'folder-api': {
-            folderId: 'folder-api',
-            title: 'api',
-            parentId: 'folder-docs',
-            localPath: 'docs/api',
-          },
-          'folder-v2': {
-            folderId: 'folder-v2',
-            title: 'v2',
-            parentId: 'folder-api',
-            localPath: 'docs/api/v2',
-          },
-        },
-      });
-
-      const result = await determineExpectedParent(client, spaceConfig, testDir, 'docs/api/v2/endpoints.md', true);
-
-      expect(result.parentId).toBe('folder-v2');
-    });
-
-    test('handles path normalization with ./ prefix', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig({
-        folders: {
-          'folder-docs': {
-            folderId: 'folder-docs',
-            title: 'docs',
-            localPath: 'docs',
-          },
-        },
-      });
-
-      const result = await determineExpectedParent(client, spaceConfig, testDir, './docs/guide.md', true);
-
-      expect(result.parentId).toBe('folder-docs');
-    });
-
-    test('propagates FolderHierarchyError from ensureFolderHierarchy', async () => {
-      const client = new ConfluenceClient(testConfig);
-      const spaceConfig = createSpaceConfig();
-
-      try {
-        await determineExpectedParent(client, spaceConfig, testDir, '../outside/file.md', true);
-        expect.unreachable('Should have thrown');
-      } catch (error) {
-        expect(error).toBeInstanceOf(FolderHierarchyError);
-      }
-    });
-  });
-});