Add tests for promise-queue and tables

Quick coverage wins for previously untested utility modules.

Author: jdalton

test/unit/lib/promise-queue.test.mts

@@ -0,0 +1,75 @@
+import { describe, expect, it } from 'vitest'
+
+import { PromiseQueue } from '../../../registry/src/lib/promise-queue'
+
+describe('PromiseQueue', () => {
+  it('executes tasks with concurrency limit', async () => {
+    const queue = new PromiseQueue(2)
+    let running = 0
+    let maxRunning = 0
+
+    const task = async () => {
+      running += 1
+      maxRunning = Math.max(maxRunning, running)
+      await new Promise(resolve => setTimeout(resolve, 10))
+      running -= 1
+      return 'done'
+    }
+
+    const results = await Promise.all([
+      queue.add(task),
+      queue.add(task),
+      queue.add(task),
+      queue.add(task),
+    ])
+
+    expect(results).toEqual(['done', 'done', 'done', 'done'])
+    expect(maxRunning).toBeLessThanOrEqual(2)
+  })
+
+  it('handles errors in tasks', async () => {
+    const queue = new PromiseQueue(1)
+
+    const failingTask = async () => {
+      throw new Error('task failed')
+    }
+
+    await expect(queue.add(failingTask)).rejects.toThrow('task failed')
+  })
+
+  it('drops oldest task when maxQueueLength exceeded', async () => {
+    const queue = new PromiseQueue(1, 2)
+    const results: string[] = []
+
+    const slowTask = async (id: string) => {
+      await new Promise(resolve => setTimeout(resolve, 20))
+      results.push(id)
+      return id
+    }
+
+    const promises = [
+      queue.add(() => slowTask('1')),
+      queue.add(() => slowTask('2')),
+      queue.add(() => slowTask('3')),
+      queue.add(() => slowTask('4')),
+    ]
+
+    await Promise.allSettled(promises)
+    expect(results.length).toBeGreaterThan(0)
+  })
+
+  it('throws error for invalid maxConcurrency', () => {
+    expect(() => new PromiseQueue(0)).toThrow('maxConcurrency must be at least 1')
+    expect(() => new PromiseQueue(-1)).toThrow(
+      'maxConcurrency must be at least 1',
+    )
+  })
+
+  it('handles empty queue', async () => {
+    const queue = new PromiseQueue(2)
+    const task = async () => 'result'
+
+    const result = await queue.add(task)
+    expect(result).toBe('result')
+  })
+})

test/unit/lib/tables.test.mts

@@ -0,0 +1,111 @@
+import { describe, expect, it } from 'vitest'
+
+import {
+  formatSimpleTable,
+  formatTable,
+} from '../../../registry/src/lib/tables'
+
+describe('tables', () => {
+  describe('formatSimpleTable', () => {
+    it('renders simple table', () => {
+      const result = formatSimpleTable(
+        [
+          { key: 'name', header: 'Name' },
+          { key: 'value', header: 'Value' },
+        ],
+        [
+          { name: 'foo', value: '123' },
+          { name: 'bar', value: '456' },
+        ],
+      )
+      expect(result).toContain('Name')
+      expect(result).toContain('Value')
+      expect(result).toContain('foo')
+      expect(result).toContain('bar')
+      expect(result).toContain('123')
+      expect(result).toContain('456')
+    })
+
+    it('aligns columns', () => {
+      const result = formatSimpleTable(
+        [
+          { key: 'left', header: 'Left', align: 'left' as const },
+          { key: 'right', header: 'Right', align: 'right' as const },
+          { key: 'center', header: 'Center', align: 'center' as const },
+        ],
+        [{ left: 'A', right: 'B', center: 'C' }],
+      )
+      expect(result).toContain('Left')
+      expect(result).toContain('Right')
+      expect(result).toContain('Center')
+    })
+
+    it('applies column colors', () => {
+      const result = formatSimpleTable(
+        [
+          {
+            key: 'name',
+            header: 'Name',
+            color: (v: string) => `<${v}>`,
+          },
+        ],
+        [{ name: 'test' }],
+      )
+      expect(result).toContain('<test>')
+    })
+
+    it('handles fixed column widths', () => {
+      const result = formatSimpleTable(
+        [{ key: 'name', header: 'Name', width: 20 }],
+        [{ name: 'short' }],
+      )
+      expect(result).toContain('Name')
+      expect(result).toContain('short')
+    })
+
+    it('handles empty data', () => {
+      const result = formatSimpleTable(
+        [
+          { key: 'name', header: 'Name' },
+          { key: 'value', header: 'Value' },
+        ],
+        [],
+      )
+      expect(result).toContain('Name')
+      expect(result).toContain('Value')
+    })
+  })
+
+  describe('formatTable', () => {
+    it('renders table with borders', () => {
+      const result = formatTable(
+        [
+          { key: 'name', header: 'Name' },
+          { key: 'age', header: 'Age' },
+        ],
+        [
+          { name: 'Alice', age: '30' },
+          { name: 'Bob', age: '25' },
+        ],
+      )
+      expect(result).toContain('Name')
+      expect(result).toContain('Age')
+      expect(result).toContain('Alice')
+      expect(result).toContain('Bob')
+    })
+
+    it('handles alignment in bordered table', () => {
+      const result = formatTable(
+        [
+          { key: 'num', header: 'Number', align: 'right' as const },
+          { key: 'text', header: 'Text', align: 'left' as const },
+        ],
+        [{ num: '42', text: 'hello' }],
+      )
+      expect(result).toContain('Number')
+      expect(result).toContain('Text')
+      expect(result).toContain('42')
+      expect(result).toContain('hello')
+    })
+  })
+})