fix: CI test summary uses NO_COLOR instead of sed hack (#985)

* fix: strip ANSI colors before grepping test summary

The mock test output uses ANSI escape codes for colored ✓/✗/━━━
characters, so the grep in the Post summary step couldn't match
them. Strip colors with sed first.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: use NO_COLOR standard instead of sed to strip ANSI codes

mock.sh now respects the NO_COLOR env var (https://no-color.org/).
CI sets NO_COLOR=1 so grep matches ✓/✗/━━━ cleanly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ahmed Abushagur 2026-02-13 11:26:41 -08:00 committed by GitHub
parent 0ed8a29004
commit d501b5eb1d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 12 additions and 6 deletions

View file

@ -17,6 +17,8 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4
- name: Run mock tests
env:
NO_COLOR: 1
run: |
bash test/mock.sh 2>&1 | tee /tmp/mock-output.log
- name: Post summary