Loading
refactor: #3621381 Drop the redundant local access-denied step
Issue: #3621381 Remove the local "am denied access to" step, now redundant with @vardot/varbase-e2e 2.0.4
Why
@vardot/varbase-e2e 2.0.4 ships the same step this recipe defined locally. Nothing was broken yet — no scenario uses that sentence, so the run was clean — but it is a duplicate waiting to bite: the same local step took out 4 scenarios in Varbase AI Figma.
Then I am denied access to "/admin/config/ai/figma"- Ours:
tests/step-definitions/ai-figma.steps.js:89 - The library's:
navigation.steps.js:264
What changed
- Deleted the local step and its doc block from
tests/step-definitions/ai-figma.steps.js. - Dependency
^2→^2.0.4. - No feature file changes.
Keep the library one: it passes on any refusal a site can give (403, 404, an access-denied page, a redirect to the log-in form) and it is documented. Ours only checked the status code.
Verified
npx cucumber-js --dry-run: 6 scenarios, 91 steps, nothing ambiguous, nothing undefined. The browser suite needs a live site, so CI confirms the rest.
AI-Generated: Yes
Checkpoints:
- File an issue
- Addition/Change/Update/Fix
- Testing to ensure no regression
- Automated unit testing coverage
- Automated functional testing coverage
- UX/UI designer responsibilities
- Readability
- Accessibility
- Performance
- Security
- Developer Documentation
- User Guide Documentation
- Reviewed by human
- Code review by maintainers
- Full testing and approval
- Credit contributors
- Review with the product owner
- Release notes snippet
- Release
Edited by Rajab Natshah