🌱 Add /list-e2e-steps Claude Code command#2508
🌱 Add /list-e2e-steps Claude Code command#2508pedjak wants to merge 1 commit intooperator-framework:mainfrom
/list-e2e-steps Claude Code command#2508Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
pedjak
changed the title
/list-e2e-steps Claude Code command
pedjak
force-pushed
the
list-e2e-steps
branch
from
08f7dbf to
8601200
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds tooling and documentation to improve the discoverability and usability of Godog e2e step definitions. It introduces a Claude Code command that generates a categorized reference of all step handlers, adds comprehensive doc comments to exported step handler functions, and updates the e2e README with usage instructions.
Changes:
- Added
.claude/commands/list-e2e-steps.mdwith instructions for generating a categorized reference of all Godog step definitions - Added doc comments to ~30 exported step handler functions describing their behavior and polling characteristics
- Updated
test/e2e/README.mdwith instructions on using the new/list-e2e-stepsClaude Code command
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.claude/commands/list-e2e-steps.md |
New Claude Code skill defining instructions for generating a comprehensive categorized reference of all e2e step definitions with parameters, DocString expectations, and polling behavior |
test/e2e/steps/steps.go |
Added doc comments to exported step handler functions documenting their behavior, parameters, and whether they poll with timeout |
test/e2e/README.md |
Added section referencing the new /list-e2e-steps Claude Code command for discovering available step definitions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.claude/commands/list-e2e-steps.md
Outdated
| 7. **Generic Resource Operations** - Get, delete, restore, match arbitrary resources | ||
| 8. **Test Operator Control** - Marking test-operator deployment ready/not-ready | ||
| 9. **Metrics** - Fetching and validating Prometheus metrics | ||
| 10. **CRD Patching** - Setting minimum values on CRD fields |
There was a problem hiding this comment.
The category numbering skips from category 5 to category 7. Category 6 is missing. The list should be renumbered sequentially: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10.
| 7. **Generic Resource Operations** - Get, delete, restore, match arbitrary resources | |
| 8. **Test Operator Control** - Marking test-operator deployment ready/not-ready | |
| 9. **Metrics** - Fetching and validating Prometheus metrics | |
| 10. **CRD Patching** - Setting minimum values on CRD fields | |
| 6. **Generic Resource Operations** - Get, delete, restore, match arbitrary resources | |
| 7. **Test Operator Control** - Marking test-operator deployment ready/not-ready | |
| 8. **Metrics** - Fetching and validating Prometheus metrics | |
| 9. **CRD Patching** - Setting minimum values on CRD fields |
test/e2e/steps/steps.go
Outdated
| } | ||
|
|
||
| // ResourceIsApplied applies the provided YAML resource to the cluster and in case of ClusterExtension it captures its name in the test context | ||
| // so it could be referred in later steps with ${NAME} |
There was a problem hiding this comment.
Grammatical error: "so it could be referred" should be "so it can be referred" or "so that it can be referred to".
| // so it could be referred in later steps with ${NAME} | |
| // so that it can be referred to in later steps with ${NAME} |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2508 +/- ##
==========================================
+ Coverage 69.84% 73.25% +3.41%
==========================================
Files 102 102
Lines 8496 8504 +8
==========================================
+ Hits 5934 6230 +296
+ Misses 2090 1799 -291
- Partials 472 475 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Add a Claude Code skill that generates a categorized reference of all Godog e2e step definitions. Add doc comments to exported step handler functions documenting their behavior and polling characteristics. Update e2e README with usage instructions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
pedjak
force-pushed
the
list-e2e-steps
branch
from
8601200 to
f30883c
Compare
1 participant
Description
Add a Claude Code skill that generates a categorized reference of all Godog e2e step definitions. Add doc comments to exported step handler functions documenting their behavior and polling characteristics.
Update e2e README with usage instructions.
Reviewer Checklist
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com