Fail CI if a test named in the README table does not exist
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 75/100
Research direction
Start with the 'What the tests cover' table in README.md and the fn names in src/test.rs, which the script must match against. Write scripts/check-readme-tests.sh using only bash and grep, then add it as a step in .github/workflows/ci.yml. Done when the script passes on main, fails naming the old identifier after a local test rename, and passes bash -n. The 'and the five tests after it' phrase needs a decision, which the issue asks you to make and report.
Written by the indexing model from the issue text.
Description
Written against commit 78d0a89; later commits may have moved things, so check the code first.
Size: Trivial
Description
The README has a table of "What the tests cover" that names tests (an_address_that_qualifies_and_authorizes_is_let_in_and_recorded, a_deactivated_policy_lets_nobody_in, ...). CONTRIBUTING.md welcomes "a clearer name for a test". Rename a test and the README keeps pointing at a name that no longer exists, which turns a documented claim into an unchecked one. Some entries are abbreviated with an ellipsis (an_address_below_the_minimum_…, a_policy_that_does_not_exist_… "and the five tests after it").
Current state
README.md"What the tests cover" table.src/test.rs: 19fntests plus helpers.
What to build
A short script (scripts/check-readme-tests.sh, bash and grep only, no new dependency) that extracts every backticked identifier in that table, treats one ending in _… or … as a prefix, and fails naming any that matches no fn in src/test.rs. Run it from CI. Decide how to treat the "and the five tests after it" phrase (suggest rewriting it to list the names) and say what you did.
Acceptance criteria
- The script passes on
maintoday. - Renaming a test locally makes it fail with the old name in the message.
- It runs as a step in
.github/workflows/ci.yml. -
bash -npasses.
Out of scope
Checking that the README's description of each test is accurate.
Verification
Run it, rename a test, run it again, revert.
- Dominant language
- Rust
- Stars
- 2
- Forks
- 2
- Avg merge
- 49m
- Merged PRs (30d)
- 1
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Sorogate/example-consumer
-
documentation good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Sorogate/example-consumer#17 ·
-
documentation good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Sorogate/example-consumer#16 · 1 comment · 1 reaction ·
-
documentation good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Sorogate/example-consumer#15 ·
-
enhancement good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Sorogate/example-consumer#14 ·
-
enhancement good first issue help wanted
Difficulty 2/5 Half a day Newbie friendliness 80/100
Sorogate/example-consumer#12 ·
All issues in Sorogate/example-consumer
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
mishraprafful/multihull#150 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
voidzero-dev/vite-plus#2970 ·
Maintainers usually reply within 1 day
-
ai_p2 comp-parquet-reader-v3
Difficulty 2/5 Half a day Newbie friendliness 66/100
ClickHouse/ClickHouse#124986 ·
Maintainers usually reply within 1 day
-
bug(ktuner): exporter directories hide daemon processesPossibly taken @iloveeyjafjalla claimed this today. Opencomponent:ktuner
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
agentic-os-org/ANOLISA#6905 · 1 comment ·
Maintainers usually reply within 1 day
-
[Feature]: [P3] engine-rs: the package source hash should ignore line endings and untracked filesOpen
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
maniator/verticopolis#880 ·
Maintainers usually reply within 1 day