Skip to content

Conversation

@noelsaw1
Copy link
Contributor

feat: Add Test Suite V2 with modular architecture

  • Complete rewrite of fixture test framework
  • New modular design with separate libraries (utils, precheck, runner, reporter)
  • Robust JSON parsing handles polluted stdout
  • Better error messages and logging
  • All 8 fixture tests now pass consistently
  • Fixed path handling for absolute paths with spaces
  • Version bump to 1.2.1

Fixes test failures caused by:

  • Scanner stdout pollution from pattern library manager
  • Shell quoting issues with bash -c
  • Poor error reporting in monolithic design

Known issue: Scanner produces different results with relative vs absolute paths
(some patterns not detected with absolute paths - needs separate fix)

noelsaw1 and others added 17 commits January 9, 2026 20:11
git add .
git commit -m "Fix CI test failures: Add jq dependency and TTY availability check

- Install jq in GitHub Actions workflow
- Add TTY availability check to prevent /dev/tty errors in CI
- Improve test suite with trace mode and error handling
- Add CI environment emulator for local testing
- Remove dead text parsing fallback
- Fix indentation inconsistencies

Tests now pass 10/10 in both local and CI environments."
Co-authored-by: noelsaw1 <56978803+noelsaw1@users.noreply.github.com>
Co-authored-by: noelsaw1 <56978803+noelsaw1@users.noreply.github.com>
Fix Required
Change: Add quotes around $PATHS → "$PATHS" in 3 locations

Lines to modify:

Line 4164: $PATHS → "$PATHS"
Line 4940: $PATHS → "$PATHS"
Line 4945: $PATHS → "$PATHS"
Line 5009: $PATHS → "$PATHS"
Total changes: 4 lines (literally adding 2 characters per line)
@noelsaw1 noelsaw1 merged commit a02dae9 into development Jan 12, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants