fix(tests): restore missing e2e fixtures, re-add bun test to CI - #3
Open
catesandrew wants to merge 3 commits into
Open
catesandrew wants to merge 3 commits into
catesandrew wants to merge 3 commits into
Conversation
8 pre-existing e2e-smoke test failures traced to two committed test fixtures (examples/mixed.log, examples/mixed-2.log) that had no git history at all -- silently excluded from every commit by a contributor's global gitignore *.log rule, with no git-status signal that they were ever missing. Recreates both fixtures with content satisfying every numeric assertion across all 8 failing tests (entry counts, json/text splits, filter+query match counts against the CLI's actual parseLine/filter/query logic). Adds an explicit `!examples/*.log` repo-level gitignore exception so this can't silently recur for any contributor. Re-adds `bun test` to CI (previously typecheck-only, per this repo's own dated comment explaining the omission). 93/93 tests pass, typecheck clean. Part of agent-ops' epic-39s (CI/Pipeline Reliability Hardening) -- log-cli is the pilot repo for agent-ops' dispatch pipeline, whose auto-merge gate depends on this repo's own CI actually being green. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Adds a Topics line summarizing deployment target, stack, and shared suite membership, generated from a fleet-wide scan. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
e2e-smoketest failures traced to two committed test fixtures (examples/mixed.log,examples/mixed-2.log) with no git history at all -- silently excluded from every commit by a contributor's global*.loggitignore rule, with nogit statussignal they were ever missing.parseLine/filter/query/summarylogic.!examples/*.logrepo-level gitignore exception so this can't silently recur for any contributor.bun testto CI (previously typecheck-only).Test plan
bun test-- 93/93 pass, 0 failbun run typecheck-- cleangit check-ignore -vshows the negation winning;git statusnow tracks the fixtures)Part of
agent-ops' epic-39s (CI/Pipeline Reliability Hardening) -- this repo is the pilot for agent-ops' dispatch pipeline, whose auto-merge gate depends on this repo's own CI actually being green.🤖 Generated with Claude Code