Skip to content

Spike add password change decoration for authentication token renewal - #47

Open
gonzalo-bulnes wants to merge 7 commits into
fix-authentication-token-not-renewed-after-password-changefrom
spike-add-password-change-decoration-for-authentication-token-renewal
Open

Spike add password change decoration for authentication token renewal#47
gonzalo-bulnes wants to merge 7 commits into
fix-authentication-token-not-renewed-after-password-changefrom
spike-add-password-change-decoration-for-authentication-token-renewal

Conversation

@gonzalo-bulnes

Copy link
Copy Markdown
Owner

As a developer
In order to get an insight of which way could lead to effectively decorating the Devise::PasswordsController#update action
And to be able to decide which is the better way to write the corresponding RSpec example
I want to explore some of the possible implementations

This spike is part of #40.

Minor fix RSpec expectation
Minor fix Cucumber step misspelling
Marking this integration test as pending increses the visibility
of the ActsAsTokenAuthenticatable#renew_authentication_token! unit tests.

This commit must be reverted once the test suite is green again.
@gonzalo-bulnes

Copy link
Copy Markdown
Owner Author

Relevant Devise code:

@gonzalo-bulnes
gonzalo-bulnes force-pushed the fix-authentication-token-not-renewed-after-password-change branch from 1f1ece4 to 998c5d0 Compare August 27, 2014 15:51
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.

1 participant