fix: enforce BDD/E2E in dev-story — Gherkin gate + E2E required for UI flows#2276
Open
pablontiv wants to merge 1 commit intobmad-code-org:mainfrom
Open
fix: enforce BDD/E2E in dev-story — Gherkin gate + E2E required for UI flows#2276pablontiv wants to merge 1 commit intobmad-code-org:mainfrom
pablontiv wants to merge 1 commit intobmad-code-org:mainfrom
Conversation
…I flows - Step 1: add BDD/SDD/E2E pre-check gate; warns when TEA is installed but no ATDD checklist exists, and warns when checklist has no Gherkin scenarios - Step 5: red phase now identifies test type (unit vs E2E) before writing the failing test, so UI tasks explicitly require E2E red phase - Step 6: E2E tests required for all user-facing flows (removes the 'when story requirements demand them' opt-in conditional) - Step 9 DoD: same language fix — E2E mandatory for UI flows
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.
What
Four targeted changes to
src/bmm-skills/4-implementation/bmad-dev-story/workflow.mdthat enforce BDD/E2E discipline when the TEA module is present.Why
Without these changes, stories with user-facing flows complete without E2E tests and without Gherkin-described acceptance criteria, even when the TEA module is installed and configured. See #2275.
Note: this extends the intent of #2272 (ATDD gate) with Gherkin content verification and E2E opt-out (rather than opt-in).
How
Given/When/Then)when story requirements demand themopt-in conditional)Testing
Verified locally against a project with TEA module installed:
bmad-dev-storynow warns at Step 1 when no ATDD checklist is found, and warns when the checklist contains no Gherkin scenarios.