Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DRY up test helpers #2308

Merged
merged 1 commit into from
Mar 6, 2023
Merged

DRY up test helpers #2308

merged 1 commit into from
Mar 6, 2023

Conversation

mark-dce
Copy link
Contributor

@mark-dce mark-dce commented Mar 6, 2023

Move the Warden test helper setup to the rails_helper.rb file since it's used in more system specs than not.

Also add `# frozen_string_literal: true since we're checking all these files.

@mark-dce
DRY up test helpers
6404c0c 
Move the Warden test helper setup to the rails_helper.rb file since
it's used in more system specs than not.

Also add `# frozen_string_literal: true since we're checking all
these files.
@coveralls
Copy link

coveralls commented Mar 6, 2023 •
edited

Coverage Status

Coverage: 91.291% (+2.3%) from 89.008% when pulling 6404c0c on dry_warden into d6e6698 on main.

@mark-dce mark-dce merged commit 95ee6fb into main Mar 6, 2023
@mark-dce mark-dce deleted the dry_warden branch March 6, 2023 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mark-dce @coveralls