Conversation
logout helper from Warden::Test::Helpers doesn't actually perform the logout (so as testing lifecycle callbacks can decide when it happens)
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
On The specific change I made in the db was: |
Just in case we have one more bug on our hands - what's the exact error you get? |
There was a problem hiding this comment.
Couldn't repro yesterday's behavior, weird. I'm sorry; it was something about a nil or null that seemed like it might have been about a time value (deleted_at?), but I didn't record it since I expected to be able to repro on demand, oops.
This behaves correctly for me on the branch. However, develop also behaves correctly for me, so I'm not sure what that means. If Art is happy then I'm fine with merging.
My apologies, I forgot to include the part that actually makes the import reproducible - the remember me checkbox must be set when logging in. Please do try again - this is a critical path, so I would like to be extra sure the fix works as expected (and that I've correctly identified the root problem). |
closes #591
To test the change:
deletedtotruefor that user directly in the database);