Fix README tests to match restructured documentation - #2
Open
msarg44 wants to merge 1 commit into
Open
Conversation
The README was rewritten on 2026-08-12 (commit 0f3aa88) which replaced the old ## Features section with ## Why Cain and dropped the explicit OWASP skill enumeration. The two corresponding tests (test_features_section and test_owasp_coverage) were not updated, causing the test suite to fail. Changes: - Replaced test_features_section with test_sections_present that validates all current README sections including the new ## Why Cain and the ##⚠️ Legal & Ethical Use heading (with emoji) - Replaced test_owasp_coverage with test_skills_coverage that checks for capabilities actually mentioned in the new README: cloud module, read-only, and benchmark
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.
The README was rewritten today (commit 0f3aa88) which replaced the old
## Featuressection with## Why Cainand dropped the explicit OWASP skill enumeration. Two corresponding tests were not updated, causing the test suite to fail.Changes:
test_features_sectionwithtest_sections_presentthat validates all current README sections, including## Why Cainand## ⚠️ Legal & Ethical Use(with emoji)test_owasp_coveragewithtest_skills_coveragethat checks for capabilities actually mentioned in the new README: cloud module, read-only, and benchmarkTest results: