Skip to content

test: port comprehensive test cases from just-bash#49

Merged
chaliy merged 1 commit intomainfrom
claude/audit-bash-test-cases-yCN58
Feb 2, 2026
Merged

test: port comprehensive test cases from just-bash#49
chaliy merged 1 commit intomainfrom
claude/audit-bash-test-cases-yCN58

Conversation

@chaliy
Copy link
Contributor

@chaliy chaliy commented Feb 2, 2026

Summary

Significantly expand test coverage inspired by Vercel's just-bash project:

  • AWK: 89 tests (was 19) - operators, loops, functions, printf formats
  • SED: 65 tests (was 17) - addresses, ranges, regex, hold space
  • JQ: 92 tests (was 21) - filters, functions, operators, string ops
  • GREP: 55 tests (was 15) - flags, context, regex patterns

Also expand bash builtin tests:

  • echo: 26 tests (escape sequences, flags, quoting)
  • sort/uniq: 31 tests (numeric, reverse, field sorting)
  • cut/tr: 35 tests (character/field modes, classes)
  • wc: 22 tests (all counting modes)
  • date: 31 tests (format specifiers)

Test Coverage Summary

Before After
~281 total tests 510+ total tests
72 active in CI 301 active in CI
62 passing 168 passing
10 skipped 133 skipped

New tests marked with ### skip: document unimplemented features and serve as a roadmap for future work.

Test plan

  • cargo test --features network passes
  • cargo test --test spec_tests passes (all 4 categories: AWK, SED, JQ, GREP)
  • cargo fmt --check passes
  • cargo clippy passes
  • Updated KNOWN_LIMITATIONS.md with new test counts

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Significantly expand test coverage inspired by Vercel's just-bash project:

- AWK: 89 tests (was 19) - operators, loops, functions, printf formats
- SED: 65 tests (was 17) - addresses, ranges, regex, hold space
- JQ: 92 tests (was 21) - filters, functions, operators, string ops
- GREP: 55 tests (was 15) - flags, context, regex patterns

Also expand bash builtin tests:
- echo: 26 tests (escape sequences, flags, quoting)
- sort/uniq: 31 tests (numeric, reverse, field sorting)
- cut/tr: 35 tests (character/field modes, classes)
- wc: 22 tests (all counting modes)
- date: 31 tests (format specifiers)

Total spec tests increased from ~281 to 510+
Active CI tests: 301 (168 pass, 133 skip for unimplemented features)

https://claude.ai/code/session_01GEyr3K2pKHhpEPhkReEgS9
@chaliy chaliy force-pushed the claude/audit-bash-test-cases-yCN58 branch from 194a20b to 3f83d3e Compare February 2, 2026 05:04
@chaliy chaliy merged commit 29be2e7 into main Feb 2, 2026
6 checks passed
@chaliy chaliy deleted the claude/audit-bash-test-cases-yCN58 branch February 2, 2026 05:10
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