@bestend/confluence-cli 1.16.1 → 2.0.0

package/tests/confluence-client.test.js

@@ -1,458 +0,0 @@
-const ConfluenceClient = require('../lib/confluence-client');
-const MockAdapter = require('axios-mock-adapter');
-
-describe('ConfluenceClient', () => {
-  let client;
-  
-  beforeEach(() => {
-    client = new ConfluenceClient({
-      domain: 'test.atlassian.net',
-      token: 'test-token'
-    });
-  });
-
-  describe('api path handling', () => {
-    test('defaults to /rest/api when path is not provided', () => {
-      const defaultClient = new ConfluenceClient({
-        domain: 'example.com',
-        token: 'no-path-token'
-      });
-
-      expect(defaultClient.baseURL).toBe('https://example.com/rest/api');
-    });
-
-    test('normalizes custom api paths', () => {
-      const customClient = new ConfluenceClient({
-        domain: 'cloud.example',
-        token: 'custom-path',
-        apiPath: 'wiki/rest/api/'
-      });
-
-      expect(customClient.baseURL).toBe('https://cloud.example/wiki/rest/api');
-    });
-  });
-
-  describe('authentication setup', () => {
-    test('uses bearer token headers by default', () => {
-      const bearerClient = new ConfluenceClient({
-        domain: 'test.atlassian.net',
-        token: 'bearer-token'
-      });
-
-      expect(bearerClient.client.defaults.headers.Authorization).toBe('Bearer bearer-token');
-    });
-
-    test('builds basic auth headers when email is provided', () => {
-      const basicClient = new ConfluenceClient({
-        domain: 'test.atlassian.net',
-        token: 'basic-token',
-        authType: 'basic',
-        email: 'user@example.com'
-      });
-
-      const encoded = Buffer.from('user@example.com:basic-token').toString('base64');
-      expect(basicClient.client.defaults.headers.Authorization).toBe(`Basic ${encoded}`);
-    });
-
-    test('throws when basic auth is missing an email', () => {
-      expect(() => new ConfluenceClient({
-        domain: 'test.atlassian.net',
-        token: 'missing-email',
-        authType: 'basic'
-      })).toThrow('Basic authentication requires an email address.');
-    });
-  });
-
-  describe('extractPageId', () => {
-    test('should return numeric page ID as is', async () => {
-      expect(await client.extractPageId('123456789')).toBe('123456789');
-      expect(await client.extractPageId(123456789)).toBe(123456789);
-    });
-
-    test('should extract page ID from URL with pageId parameter', async () => {
-      const url = 'https://test.atlassian.net/wiki/spaces/TEST/pages/123456789/Page+Title';
-      expect(await client.extractPageId(url + '?pageId=987654321')).toBe('987654321');
-    });
-
-    test('should resolve display URLs', async () => {
-      // Mock the API response for display URL resolution
-      const mock = new MockAdapter(client.client);
-
-      mock.onGet('/content').reply(200, {
-        results: [{
-          id: '12345',
-          title: 'Page Title',
-          _links: { webui: '/display/TEST/Page+Title' }
-        }]
-      });
-
-      const displayUrl = 'https://test.atlassian.net/display/TEST/Page+Title';
-      expect(await client.extractPageId(displayUrl)).toBe('12345');
-
-      mock.restore();
-    });
-
-    test('should resolve nested display URLs', async () => {
-      // Mock the API response for display URL resolution
-      const mock = new MockAdapter(client.client);
-
-      mock.onGet('/content').reply(200, {
-        results: [{
-          id: '67890',
-          title: 'Child Page',
-          _links: { webui: '/display/TEST/Parent/Child+Page' }
-        }]
-      });
-
-      const displayUrl = 'https://test.atlassian.net/display/TEST/Parent/Child+Page';
-      expect(await client.extractPageId(displayUrl)).toBe('67890');
-
-      mock.restore();
-    });
-
-    test('should throw error when display URL cannot be resolved', async () => {
-      const mock = new MockAdapter(client.client);
-
-      // Mock empty result
-      mock.onGet('/content').reply(200, {
-        results: []
-      });
-
-      const displayUrl = 'https://test.atlassian.net/display/TEST/NonExistentPage';
-      await expect(client.extractPageId(displayUrl)).rejects.toThrow(/Could not resolve page ID/);
-
-      mock.restore();
-    });
-  });
-
-  describe('markdownToStorage', () => {
-    test('should convert basic markdown to native Confluence storage format', () => {
-      const markdown = '# Hello World\n\nThis is a **test** page with *italic* text.';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<h1>Hello World</h1>');
-      expect(result).toContain('<p>This is a <strong>test</strong> page with <em>italic</em> text.</p>');
-      expect(result).not.toContain('<ac:structured-macro ac:name="html">');
-    });
-
-    test('should convert code blocks to Confluence code macro', () => {
-      const markdown = '```javascript\nconsole.log("Hello World");\n```';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<ac:structured-macro ac:name="code">');
-      expect(result).toContain('<ac:parameter ac:name="language">javascript</ac:parameter>');
-      expect(result).toContain('console.log("Hello World");');
-    });
-
-    test('should convert lists to native Confluence format', () => {
-      const markdown = '- Item 1\n- Item 2\n\n1. First\n2. Second';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<ul>');
-      expect(result).toContain('<li><p>Item 1</p></li>');
-      expect(result).toContain('<ol>');
-      expect(result).toContain('<li><p>First</p></li>');
-    });
-
-    test('should convert Confluence admonitions', () => {
-      const markdown = '[!info]\nThis is an info message';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<ac:structured-macro ac:name="info">');
-      expect(result).toContain('This is an info message');
-    });
-
-    test('should convert tables to native Confluence format', () => {
-      const markdown = '| Header 1 | Header 2 |\n|----------|----------|\n| Cell 1   | Cell 2   |';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<table>');
-      expect(result).toContain('<th><p>Header 1</p></th>');
-      expect(result).toContain('<td><p>Cell 1</p></td>');
-    });
-
-    test('should keep links as standard HTML anchors', () => {
-      const markdown = '[Example Link](https://example.com)';
-      const result = client.markdownToStorage(markdown);
-      
-      expect(result).toContain('<a href="https://example.com">');
-      expect(result).toContain('Example Link');
-    });
-  });
-
-  describe('markdownToNativeStorage', () => {
-    test('should act as an alias to htmlToConfluenceStorage via markdown render', () => {
-      const markdown = '# Native Storage Test';
-      const result = client.markdownToNativeStorage(markdown);
-
-      expect(result).toContain('<h1>Native Storage Test</h1>');
-    });
-
-    test('should handle code blocks correctly', () => {
-      const markdown = '```javascript\nconst a = 1;\n```';
-      const result = client.markdownToNativeStorage(markdown);
-
-      expect(result).toContain('<ac:structured-macro ac:name="code">');
-      expect(result).toContain('const a = 1;');
-    });
-  });
-
-  describe('storageToMarkdown', () => {
-    test('should convert Confluence storage format to markdown', () => {
-      const storage = '<h1>Hello World</h1><p>This is a <strong>test</strong> page.</p>';
-      const result = client.storageToMarkdown(storage);
-      
-      expect(result).toContain('# Hello World');
-      expect(result).toContain('**test**');
-    });
-
-    test('should convert Confluence code macro to markdown', () => {
-      const storage = '<ac:structured-macro ac:name="code"><ac:parameter ac:name="language">javascript</ac:parameter><ac:plain-text-body><![CDATA[console.log("Hello");]]></ac:plain-text-body></ac:structured-macro>';
-      const result = client.storageToMarkdown(storage);
-      
-      expect(result).toContain('```javascript');
-      expect(result).toContain('console.log("Hello");');
-      expect(result).toContain('```');
-    });
-
-    test('should convert Confluence macros to admonitions', () => {
-      const storage = '<ac:structured-macro ac:name="info"><ac:rich-text-body><p>This is info</p></ac:rich-text-body></ac:structured-macro>';
-      const result = client.storageToMarkdown(storage);
-      
-      expect(result).toContain('[!info]');
-      expect(result).toContain('This is info');
-    });
-
-    test('should convert Confluence links to markdown', () => {
-      const storage = '<ac:link><ri:url ri:value="https://example.com" /><ac:plain-text-link-body><![CDATA[Example]]></ac:plain-text-link-body></ac:link>';
-      const result = client.storageToMarkdown(storage);
-      
-      expect(result).toContain('[Example](https://example.com)');
-    });
-  });
-
-  describe('htmlToMarkdown', () => {
-    test('should convert basic HTML to markdown', () => {
-      const html = '<h2>Title</h2><p>Some <strong>bold</strong> and <em>italic</em> text.</p>';
-      const result = client.htmlToMarkdown(html);
-      
-      expect(result).toContain('## Title');
-      expect(result).toContain('**bold**');
-      expect(result).toContain('*italic*');
-    });
-
-    test('should convert HTML lists to markdown', () => {
-      const html = '<ul><li><p>Item 1</p></li><li><p>Item 2</p></li></ul>';
-      const result = client.htmlToMarkdown(html);
-      
-      expect(result).toContain('- Item 1');
-      expect(result).toContain('- Item 2');
-    });
-
-    test('should convert HTML tables to markdown', () => {
-      const html = '<table><tr><th><p>Header</p></th></tr><tr><td><p>Cell</p></td></tr></table>';
-      const result = client.htmlToMarkdown(html);
-      
-      expect(result).toContain('| Header |');
-      expect(result).toContain('| --- |');
-      expect(result).toContain('| Cell |');
-    });
-  });
-
-  describe('page creation and updates', () => {
-    test('should have required methods for page management', () => {
-      expect(typeof client.createPage).toBe('function');
-      expect(typeof client.updatePage).toBe('function');
-      expect(typeof client.getPageForEdit).toBe('function');
-      expect(typeof client.createChildPage).toBe('function');
-      expect(typeof client.findPageByTitle).toBe('function');
-      expect(typeof client.deletePage).toBe('function');
-    });
-  });
-
-  describe('deletePage', () => {
-    test('should delete a page by ID', async () => {
-      const mock = new MockAdapter(client.client);
-      mock.onDelete('/content/123456789').reply(204);
-
-      await expect(client.deletePage('123456789')).resolves.toEqual({ id: '123456789' });
-
-      mock.restore();
-    });
-
-    test('should delete a page by URL', async () => {
-      const mock = new MockAdapter(client.client);
-      mock.onDelete('/content/987654321').reply(204);
-
-      await expect(
-        client.deletePage('https://test.atlassian.net/wiki/viewpage.action?pageId=987654321')
-      ).resolves.toEqual({ id: '987654321' });
-
-      mock.restore();
-    });
-  });
-
-  describe('page tree operations', () => {
-    test('should have required methods for tree operations', () => {
-      expect(typeof client.getChildPages).toBe('function');
-      expect(typeof client.getAllDescendantPages).toBe('function');
-      expect(typeof client.copyPageTree).toBe('function');
-      expect(typeof client.buildPageTree).toBe('function');
-      expect(typeof client.shouldExcludePage).toBe('function');
-    });
-
-    test('should correctly exclude pages based on patterns', () => {
-      const patterns = ['temp*', 'test*', '*draft*'];
-
-      expect(client.shouldExcludePage('temporary document', patterns)).toBe(true);
-      expect(client.shouldExcludePage('test page', patterns)).toBe(true);
-      expect(client.shouldExcludePage('my draft page', patterns)).toBe(true);
-      expect(client.shouldExcludePage('normal document', patterns)).toBe(false);
-      expect(client.shouldExcludePage('production page', patterns)).toBe(false);
-    });
-
-    test('should handle empty exclude patterns', () => {
-      expect(client.shouldExcludePage('any page', [])).toBe(false);
-      expect(client.shouldExcludePage('any page', null)).toBe(false);
-      expect(client.shouldExcludePage('any page', undefined)).toBe(false);
-    });
-
-    test('globToRegExp should escape regex metacharacters and match case-insensitively', () => {
-      const patterns = [
-        'file.name*',   // dot should be literal
-        '[draft]?',     // brackets should be literal
-        'Plan (Q1)?',   // parentheses literal, ? wildcard
-        'DATA*SET',     // case-insensitive
-      ];
-      const rx = patterns.map(p => client.globToRegExp(p));
-      expect('file.name.v1').toMatch(rx[0]);
-      expect('filexname').not.toMatch(rx[0]);
-      expect('[draft]1').toMatch(rx[1]);
-      expect('[draft]AB').not.toMatch(rx[1]);
-      expect('Plan (Q1)A').toMatch(rx[2]);
-      expect('Plan Q1A').not.toMatch(rx[2]);
-      expect('data big set').toMatch(rx[3]);
-    });
-
-    test('buildPageTree should link children by parentId and collect orphans at root', () => {
-      const rootId = 'root';
-      const pages = [
-        { id: 'a', title: 'A', parentId: rootId },
-        { id: 'b', title: 'B', parentId: 'a' },
-        { id: 'c', title: 'C', parentId: 'missing' }, // orphan
-      ];
-      const tree = client.buildPageTree(pages, rootId);
-      // tree should contain A and C at top-level (B is child of A)
-      const topTitles = tree.map(n => n.title).sort();
-      expect(topTitles).toEqual(['A', 'C']);
-      const a = tree.find(n => n.title === 'A');
-      expect(a.children.map(n => n.title)).toEqual(['B']);
-    });
-
-    test('exclude parser should tolerate spaces and empty items', () => {
-      const raw = ' temp* , , *draft* ,,test? ';
-      const patterns = raw.split(',').map(p => p.trim()).filter(Boolean);
-      expect(patterns).toEqual(['temp*', '*draft*', 'test?']);
-      expect(client.shouldExcludePage('temp file', patterns)).toBe(true);
-      expect(client.shouldExcludePage('my draft page', patterns)).toBe(true);
-      expect(client.shouldExcludePage('test1', patterns)).toBe(true);
-      expect(client.shouldExcludePage('production', patterns)).toBe(false);
-    });
-  });
-
-  describe('comments', () => {
-    test('should list comments with location filter', async () => {
-      const mock = new MockAdapter(client.client);
-      mock.onGet('/content/123/child/comment').reply(config => {
-        expect(config.params.location).toBe('inline');
-        expect(config.params.expand).toContain('body.storage');
-        expect(config.params.expand).toContain('ancestors');
-        return [200, {
-          results: [
-            {
-              id: 'c1',
-              status: 'current',
-              body: { storage: { value: '<p>Hello</p>' } },
-              history: { createdBy: { displayName: 'Ada' }, createdDate: '2025-01-01' },
-              version: { number: 1 },
-              ancestors: [{ id: 'c0', type: 'comment' }],
-              extensions: {
-                location: 'inline',
-                inlineProperties: { selection: 'Hello', originalSelection: 'Hello' },
-                resolution: { status: 'open' }
-              }
-            }
-          ],
-          _links: { next: '/rest/api/content/123/child/comment?start=2' }
-        }];
-      });
-
-      const page = await client.listComments('123', { location: 'inline' });
-      expect(page.results).toHaveLength(1);
-      expect(page.results[0].location).toBe('inline');
-      expect(page.results[0].resolution).toBe('open');
-      expect(page.results[0].parentId).toBe('c0');
-      expect(page.nextStart).toBe(2);
-
-      mock.restore();
-    });
-
-    test('should create inline comment with inline properties', async () => {
-      const mock = new MockAdapter(client.client);
-      mock.onPost('/content').reply(config => {
-        const payload = JSON.parse(config.data);
-        expect(payload.type).toBe('comment');
-        expect(payload.container.id).toBe('123');
-        expect(payload.body.storage.value).toBe('<p>Hi</p>');
-        expect(payload.ancestors[0].id).toBe('c0');
-        expect(payload.extensions.location).toBe('inline');
-        expect(payload.extensions.inlineProperties.originalSelection).toBe('Hi');
-        expect(payload.extensions.inlineProperties.markerRef).toBe('comment-1');
-        return [200, { id: 'c1', type: 'comment' }];
-      });
-
-      await client.createComment('123', '<p>Hi</p>', 'storage', {
-        parentId: 'c0',
-        location: 'inline',
-        inlineProperties: {
-          selection: 'Hi',
-          originalSelection: 'Hi',
-          markerRef: 'comment-1'
-        }
-      });
-
-      mock.restore();
-    });
-
-    test('should delete a comment by ID', async () => {
-      const mock = new MockAdapter(client.client);
-      mock.onDelete('/content/456').reply(204);
-
-      await expect(client.deleteComment('456')).resolves.toEqual({ id: '456' });
-
-      mock.restore();
-    });
-  });
-
-  describe('attachments', () => {
-    test('should have required methods for attachment handling', () => {
-      expect(typeof client.listAttachments).toBe('function');
-      expect(typeof client.getAllAttachments).toBe('function');
-      expect(typeof client.downloadAttachment).toBe('function');
-    });
-
-    test('matchesPattern should respect glob patterns', () => {
-      expect(client.matchesPattern('report.png', '*.png')).toBe(true);
-      expect(client.matchesPattern('report.png', '*.jpg')).toBe(false);
-      expect(client.matchesPattern('report.png', ['*.jpg', 'report.*'])).toBe(true);
-      expect(client.matchesPattern('report.png', null)).toBe(true);
-      expect(client.matchesPattern('report.png', [])).toBe(true);
-    });
-
-    test('parseNextStart should read start query param when present', () => {
-      expect(client.parseNextStart('/rest/api/content/1/child/attachment?start=25')).toBe(25);
-      expect(client.parseNextStart('/rest/api/content/1/child/attachment?limit=50')).toBeNull();
-      expect(client.parseNextStart(null)).toBeNull();
-    });
-  });
-});