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.

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

[Issue #1839] Refactor secret_shares controller spec to request spec #1875

Conversation

camilacampos
Copy link
Contributor

Description

This refactors the tests associated with the SecretSharesController to use RSpec request type tests instead of controller

Corresponding Issue

Fixes #1839
Parent issue: #1815


Reviewing this pull request? Check out our Code Review Practices guide if you haven't already!

Copy link
Member

@julianguyen julianguyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solid work! Thanks for working on this :D Just one piece of feedback and we should be good to merge!

spec/requests/secret_shares_spec.rb Outdated Show resolved Hide resolved
spec/requests/secret_shares_spec.rb Outdated Show resolved Hide resolved
@camilacampos camilacampos force-pushed the issue-1839/refactor-secret-shares-spec branch from 587ea6d to 49589b3 Compare October 4, 2020 18:33
Copy link
Member

@julianguyen julianguyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Go ahead and merge! 🙌

@camilacampos camilacampos merged commit 4bee6f4 into ifmeorg:main Oct 4, 2020
@welcome
Copy link

welcome bot commented Oct 4, 2020

Thank you for merging this pull request with us! If you haven't already, in another pull request, please add yourself to our About page.

@camilacampos camilacampos deleted the issue-1839/refactor-secret-shares-spec branch October 4, 2020 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor spec/controllers/secret_shares_controller_spec.rb to spec/request/secret_share_spec.rb
2 participants