faf-taf-git 2.4.1 → 2.4.2

package/test-standalone.ts

@@ -1,87 +0,0 @@
-/**
- * Standalone test script to simulate the action
- * Tests the action logic without GitHub Actions environment
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-import { parseJestOutput } from './src/parsers/jest';
-import { updateTafFile } from './src/taf-core';
-
-async function testAction() {
-  console.log('Starting standalone action test...\n');
-
-  // Simulate Jest output from faf-cli
-  const jestOutput = `
-PASS src/commands/taf-init.test.ts
-PASS src/commands/taf-log.test.ts
-PASS src/commands/taf-validate.test.ts
-PASS src/commands/taf-stats.test.ts
-
-Test Suites: 18 passed, 18 total
-Tests:       173 passed, 173 total
-Snapshots:   0 total
-Time:        2.512 s
-Ran all test suites.
-`;
-
-  console.log('1. Parsing Jest output...');
-  const testResults = parseJestOutput(jestOutput);
-
-  if (!testResults) {
-    console.error('❌ Failed to parse Jest output');
-    process.exit(1);
-  }
-
-  console.log('✅ Parsed successfully:');
-  console.log(`   - Total: ${testResults.total}`);
-  console.log(`   - Passed: ${testResults.passed}`);
-  console.log(`   - Failed: ${testResults.failed}`);
-  console.log(`   - Result: ${testResults.result}\n`);
-
-  // Test with faf-cli .taf file
-  const tafPath = path.join('/Users/wolfejam/FAF/cli', '.taf');
-
-  console.log('2. Checking .taf file...');
-  if (!fs.existsSync(tafPath)) {
-    console.error('❌ No .taf file found at:', tafPath);
-    process.exit(1);
-  }
-  console.log('✅ Found .taf file\n');
-
-  console.log('3. Reading current .taf file...');
-  const beforeContent = fs.readFileSync(tafPath, 'utf-8');
-  const beforeLines = beforeContent.split('\n').filter(line => line.includes('timestamp')).length;
-  console.log(`✅ Current test runs: ${beforeLines}\n`);
-
-  console.log('4. Updating .taf file...');
-  const updated = await updateTafFile(tafPath, testResults);
-
-  if (!updated) {
-    console.error('❌ Failed to update .taf file');
-    process.exit(1);
-  }
-  console.log('✅ .taf file updated\n');
-
-  console.log('5. Verifying update...');
-  const afterContent = fs.readFileSync(tafPath, 'utf-8');
-  const afterLines = afterContent.split('\n').filter(line => line.includes('timestamp')).length;
-  console.log(`✅ Test runs after update: ${afterLines}`);
-
-  if (afterLines !== beforeLines + 1) {
-    console.error('❌ Expected one additional run');
-    process.exit(1);
-  }
-
-  console.log('\n✅ Standalone test PASSED');
-  console.log('\nThe action successfully:');
-  console.log('  - Parsed Jest output (173/173 tests)');
-  console.log('  - Updated .taf file');
-  console.log('  - Added new test run entry');
-  console.log('  - Maintained proper format');
-}
-
-testAction().catch(error => {
-  console.error('❌ Test failed:', error);
-  process.exit(1);
-});

package/tests/action-bundle.test.ts

@@ -1,34 +0,0 @@
-/**
- * The built Action bundle — dist/index.js, what action.yml runs — loads and
- * reaches the parse step under this Node (CI runs 24.x, the runtime
- * action.yml declares).
- *
- * An open-ended `overrides.undici: ">=6.28.0"` resolved undici 8 (Node >=
- * 22.19), which crashed the bundle at load on Node 20, before any input was
- * read. The src-level tests never executed the bundle, so nothing caught it.
- */
-import { spawnSync } from 'child_process';
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-import { UNPARSEABLE_OUTPUT } from '../src/parsers';
-
-describe('Action bundle (dist/index.js)', () => {
-  it('loads and fails an unparseable test output with the parser list', () => {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'taf-bundle-'));
-    const file = path.join(dir, 'test-output.txt');
-    fs.writeFileSync(file, 'no test summary here\n');
-
-    // Actions pass inputs as INPUT_<NAME> env vars; stdout is captured here,
-    // so the ::error:: line never reaches a real runner's log.
-    const run = spawnSync(process.execPath, [path.join(__dirname, '..', 'dist', 'index.js')], {
-      env: { ...process.env, 'INPUT_TEST-OUTPUT-FILE': file },
-      encoding: 'utf-8',
-    });
-    fs.rmSync(dir, { recursive: true, force: true });
-
-    expect(run.stderr).not.toMatch(/TypeError/);
-    expect(run.stdout).toContain(`::error::${UNPARSEABLE_OUTPUT}`);
-    expect(run.status).toBe(1);
-  });
-});

package/tests/action-parse-error.test.ts

@@ -1,47 +0,0 @@
-/**
- * The Action's parse error names every parser the dispatcher tries.
- *
- * It said "Supported: Jest, Vitest." while Bun (2.2.0) and WJTTC (2.3.0)
- * were already live. The CLI and the Action now build the message from the
- * same parser table that parseTestOutput walks.
- */
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-
-// Factory mocks: no real @actions/* module (or its undici network stack) loads.
-jest.mock('@actions/core', () => ({
-  getInput: jest.fn(),
-  info: jest.fn(),
-  warning: jest.fn(),
-  debug: jest.fn(),
-  setOutput: jest.fn(),
-  setFailed: jest.fn(),
-}));
-jest.mock('@actions/exec', () => ({ exec: jest.fn() }));
-
-import * as core from '@actions/core';
-
-describe('Action: unparseable test output', () => {
-  it('fails with a message naming Bun, WJTTC, Jest and Vitest', () => {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'taf-action-'));
-    const file = path.join(dir, 'test-output.txt');
-    fs.writeFileSync(file, 'no test summary here\n');
-    (core.getInput as jest.Mock).mockImplementation((name: string) =>
-      name === 'test-output-file' ? file : '',
-    );
-
-    // src/index.ts runs the Action on load; the parse failure is reached
-    // before its first await, so setFailed has been called when this returns.
-    jest.isolateModules(() => {
-      require('../src/index');
-    });
-
-    const failures = (core.setFailed as jest.Mock).mock.calls.map((c) => String(c[0]));
-    expect(failures).toHaveLength(1);
-    for (const parser of ['Bun', 'WJTTC', 'Jest', 'Vitest']) {
-      expect(failures[0]).toContain(parser);
-    }
-    fs.rmSync(dir, { recursive: true, force: true });
-  });
-});

package/tests/badge.test.ts

@@ -1,260 +0,0 @@
-/**
- * Badge Generator Tests
- *
- * Tests SVG output validity, .taf parsing, and edge cases.
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-import * as os from 'os';
-import { renderBadgeSvg, generateBadge } from '../src/badge';
-
-function createTempDir(prefix: string): string {
-  return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-}
-
-// ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-// SVG RENDERING
-// ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-
-describe('SVG Rendering', () => {
-  test('should produce valid SVG with label and value', () => {
-    const svg = renderBadgeSvg('TAF', '10/10 passing', '#003366');
-
-    expect(svg).toContain('<svg');
-    expect(svg).toContain('</svg>');
-    expect(svg).toContain('TAF');
-    expect(svg).toContain('10/10 passing');
-  });
-
-  test('should include correct color', () => {
-    const svg = renderBadgeSvg('TAF', '10/10 passing', '#003366');
-
-    expect(svg).toContain('fill="#003366"');
-  });
-
-  test('should include accessibility attributes', () => {
-    const svg = renderBadgeSvg('TAF', '5/5 passing', '#003366');
-
-    expect(svg).toContain('role="img"');
-    expect(svg).toContain('aria-label="TAF: 5/5 passing"');
-    expect(svg).toContain('<title>TAF: 5/5 passing</title>');
-  });
-
-  test('should use Verdana font', () => {
-    const svg = renderBadgeSvg('TAF', 'test', '#555');
-
-    expect(svg).toContain('Verdana');
-  });
-
-  test('should have 20px height', () => {
-    const svg = renderBadgeSvg('TAF', 'test', '#555');
-
-    expect(svg).toContain('height="20"');
-  });
-
-  test('should have rounded corners via clipPath', () => {
-    const svg = renderBadgeSvg('TAF', 'test', '#555');
-
-    expect(svg).toContain('rx="3"');
-    expect(svg).toContain('clip-path="url(#r)"');
-  });
-
-  test('should handle custom label', () => {
-    const svg = renderBadgeSvg('tests', '99/100 passing', '#E32400');
-
-    expect(svg).toContain('tests');
-    expect(svg).toContain('99/100 passing');
-    expect(svg).toContain('#E32400');
-  });
-});
-
-// ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-// BADGE GENERATION FROM .taf
-// ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-
-describe('Badge Generation', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir('badge-test-');
-  });
-
-  afterEach(() => {
-    fs.rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  test('should generate navy badge when all tests pass', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: PASSED
-    tests:
-      total: 38
-      passed: 38
-      failed: 0
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('38/38 passing');
-    expect(result.svg).toContain('#003366');
-  });
-
-  test('should generate red badge when tests fail', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: FAILED
-    tests:
-      total: 38
-      passed: 35
-      failed: 3
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('35/38 passing');
-    expect(result.svg).toContain('#E32400');
-  });
-
-  test('should generate gray badge when .taf is missing', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    // Don't create the file
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('no data');
-    expect(result.svg).toContain('#9F9F9F');
-  });
-
-  test('should generate gray badge when test_history is empty', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history: []
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('no data');
-    expect(result.svg).toContain('#9F9F9F');
-  });
-
-  test('should use latest run from multiple entries', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: FAILED
-    tests:
-      total: 10
-      passed: 7
-      failed: 3
-  - timestamp: "2026-01-02T00:00:00Z"
-    result: PASSED
-    tests:
-      total: 10
-      passed: 10
-      failed: 0
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('10/10 passing');
-    expect(result.svg).toContain('#003366');
-  });
-
-  test('should use custom label', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: PASSED
-    tests:
-      total: 5
-      passed: 5
-      failed: 0
-`);
-
-    const result = generateBadge({ tafPath, label: 'tests' });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('tests');
-    expect(result.svg).not.toContain('>TAF<');
-  });
-
-  test('should handle single test run', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: PASSED
-    tests:
-      total: 1
-      passed: 1
-      failed: 0
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('1/1 passing');
-  });
-
-  test('should handle large test counts', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: PASSED
-    tests:
-      total: 10000
-      passed: 10000
-      failed: 0
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('10000/10000 passing');
-  });
-
-  test('should handle invalid YAML gracefully', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, '{{{{ not yaml at all');
-
-    const result = generateBadge({ tafPath });
-
-    // Should either return no-data badge or error, but never throw
-    expect(result).toBeDefined();
-  });
-
-  test('should handle .taf with missing tests field', () => {
-    const tafPath = path.join(tempDir, '.taf');
-    fs.writeFileSync(tafPath, `format_version: 1.0.0
-project: test
-test_history:
-  - timestamp: "2026-01-01T00:00:00Z"
-    result: PASSED
-`);
-
-    const result = generateBadge({ tafPath });
-
-    expect(result.success).toBe(true);
-    expect(result.svg).toContain('no data');
-  });
-});

package/tests/bun-parser.test.ts

@@ -1,173 +0,0 @@
-/**
- * Tests for Bun output parser
- */
-
-import { parseBunOutput } from '../src/parsers/bun';
-
-describe('Bun Parser', () => {
-  describe('parseBunOutput', () => {
-    it('should parse all passing tests (minimal output)', () => {
-      const output = `
- 173 pass
- 0 fail
- 612 expect() calls
-Ran 173 tests across 8 files. [4.2s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result).not.toBeNull();
-      expect(result?.total).toBe(173);
-      expect(result?.passed).toBe(173);
-      expect(result?.failed).toBe(0);
-      expect(result?.skipped).toBeUndefined();
-      expect(result?.result).toBe('PASSED');
-    });
-
-    it('should parse the grok-faf-mcp 1.4.9 shape (pass + skip + todo + fail=0)', () => {
-      const output = `
- 232 pass
- 1 skip
- 1 todo
- 0 fail
- 848 expect() calls
-Ran 234 tests across 9 files. [21.53s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result).not.toBeNull();
-      expect(result?.total).toBe(234);
-      expect(result?.passed).toBe(232);
-      expect(result?.failed).toBe(0);
-      // skipped aggregates skip + todo (parity with vitest convention)
-      expect(result?.skipped).toBe(2);
-      expect(result?.result).toBe('PASSED');
-    });
-
-    it('should mark result FAILED when any tests fail', () => {
-      const output = `
- 170 pass
- 3 fail
- 612 expect() calls
-Ran 173 tests across 8 files. [5.1s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result).not.toBeNull();
-      expect(result?.total).toBe(173);
-      expect(result?.passed).toBe(170);
-      expect(result?.failed).toBe(3);
-      expect(result?.result).toBe('FAILED');
-    });
-
-    it('should handle skip-only output', () => {
-      const output = `
- 170 pass
- 3 skip
- 0 fail
- 612 expect() calls
-Ran 173 tests across 8 files. [5.0s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result?.total).toBe(173);
-      expect(result?.passed).toBe(170);
-      expect(result?.skipped).toBe(3);
-      expect(result?.result).toBe('PASSED');
-    });
-
-    it('should strip ANSI color codes', () => {
-      const output =
-        '\x1b[32m 232 pass\x1b[0m\n' +
-        '\x1b[33m 1 skip\x1b[0m\n' +
-        '\x1b[33m 1 todo\x1b[0m\n' +
-        '\x1b[31m 0 fail\x1b[0m\n' +
-        ' 848 expect() calls\n' +
-        'Ran 234 tests across 9 files. [21.53s]\n';
-
-      const result = parseBunOutput(output);
-
-      expect(result?.total).toBe(234);
-      expect(result?.passed).toBe(232);
-      expect(result?.skipped).toBe(2);
-    });
-
-    it('should handle CRLF line endings', () => {
-      const output =
-        ' 173 pass\r\n 0 fail\r\nRan 173 tests across 8 files. [4.2s]\r\n';
-
-      const result = parseBunOutput(output);
-
-      expect(result?.total).toBe(173);
-      expect(result?.passed).toBe(173);
-    });
-
-    it('should ignore an appended jest-shim "Tests:" line (legacy ci.yml conversion)', () => {
-      // Some consumer CIs append a jest-shaped summary so the legacy jest
-      // parser could read bun output. With bun support, the appended line
-      // should be ignored — the canonical Ran-line wins.
-      const output = `
- 232 pass
- 1 skip
- 1 todo
- 0 fail
- 848 expect() calls
-Ran 234 tests across 9 files. [21.53s]
-Tests: 232 passed, 232 total
-`;
-
-      const result = parseBunOutput(output);
-
-      // Truthful total from "Ran 234" — NOT 232 from the jest-shim line.
-      expect(result?.total).toBe(234);
-      expect(result?.passed).toBe(232);
-      expect(result?.skipped).toBe(2);
-    });
-
-    it('should return null when no "Ran N tests" line is present (non-bun output)', () => {
-      const output = `
-Test Suites: 18 passed, 18 total
-Tests:       173 passed, 173 total
-Snapshots:   0 total
-Time:        2.5s
-`;
-
-      const result = parseBunOutput(output);
-
-      // Defers to the next parser in the dispatcher chain.
-      expect(result).toBeNull();
-    });
-
-    it('should return null on empty input', () => {
-      expect(parseBunOutput('')).toBeNull();
-    });
-
-    it('should return null when "Ran 0 tests" (nothing actually ran)', () => {
-      const output = `
- 0 pass
- 0 fail
-Ran 0 tests across 0 files. [0.1s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result).toBeNull();
-    });
-
-    it('should handle singular forms gracefully (1 test)', () => {
-      const output = `
- 1 pass
- 0 fail
-Ran 1 test across 1 file. [0.5s]
-`;
-
-      const result = parseBunOutput(output);
-
-      expect(result?.total).toBe(1);
-      expect(result?.passed).toBe(1);
-    });
-  });
-});

package/tests/cli-version.test.ts

@@ -1,31 +0,0 @@
-/**
- * `faf-taf-git --version` is a stranger's first command. Before 2.4.1 it fell
- * through to "No test output file specified" and exit 1 (found by /dev-notes
- * on the published 2.4.0).
- */
-import { spawnSync } from 'child_process';
-import * as fs from 'fs';
-import * as path from 'path';
-import { cliVersion } from '../src/cli';
-
-const root = path.join(__dirname, '..');
-const pkgVersion = JSON.parse(fs.readFileSync(path.join(root, 'package.json'), 'utf-8')).version;
-const bin = path.join(root, 'dist', 'cli.js');
-
-describe('--version', () => {
-  test('cliVersion() is the package.json version', () => {
-    expect(cliVersion()).toBe(pkgVersion);
-  });
-
-  test.each(['--version', '-V'])('the built bin answers %s with the version and exit 0', (flag) => {
-    const r = spawnSync(process.execPath, [bin, flag], { encoding: 'utf-8' });
-    expect(r.status).toBe(0);
-    expect(r.stdout.trim()).toBe(pkgVersion);
-  });
-
-  test('-v is still --verbose, not the version', () => {
-    const r = spawnSync(process.execPath, [bin, '-v'], { encoding: 'utf-8' });
-    expect(r.status).toBe(1);
-    expect(r.stderr).toContain('No test output file specified');
-  });
-});