Skip to content

style: Fix Capybara offenses - #2743

Merged
mroderick merged 7 commits into
masterfrom
fix/rubocop-capybara
Jul 28, 2026
Merged

style: Fix Capybara offenses#2743
mroderick merged 7 commits into
masterfrom
fix/rubocop-capybara

Conversation

@mroderick

Copy link
Copy Markdown
Collaborator

This PR fixes the Capybara cops.

Seven commits:

  1. style: Fix Capybara/RSpec/NegationMatcherAfterVisit offense — 1 offense, manual (wait with positive matcher first)
  2. style: Fix Capybara/RSpec/VisibilityMatcher offense — 1 offense, autocorrected
  3. style: Fix Capybara/SpecificFinders offenses — 6 offenses, autocorrected
  4. style: Fix Capybara/RSpec/CurrentPathExpectation offenses — 17 offenses, autocorrected
  5. style: Fix Capybara/RSpec/HaveSelector offenses — 21 offenses, autocorrected
  6. style: Fix Capybara/RSpec/NegationMatcher offenses — 49 offenses, autocorrected
  7. style: Fix Capybara/RSpec/HaveContent offenses — 339 offenses, autocorrected

Each commit removes the relevant exclusion from .rubocop_todo.yml and regenerates it.

  • Offense count: 1834 → 1386 (−448)
  • bundle exec rubocop is green
  • Affected specs pass (267 examples, 0 failures)

Fix Capybara/RSpec/NegationMatcherAfterVisit in
spec/features/admin/chapters_spec.rb and regenerate .rubocop_todo.yml.
Fix 6 offenses and regenerate .rubocop_todo.yml.
Fix 17 offenses and regenerate .rubocop_todo.yml.
Fix 21 offenses and regenerate .rubocop_todo.yml.
Fix 49 offenses and regenerate .rubocop_todo.yml.
Fix 339 offenses and regenerate .rubocop_todo.yml.
@mroderick
mroderick marked this pull request as ready for review July 28, 2026 07:12
@mroderick
mroderick merged commit 9ad2436 into master Jul 28, 2026
9 checks passed
@mroderick
mroderick deleted the fix/rubocop-capybara branch July 28, 2026 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant