https://github.com/publiclab/plots2/pull/5410 was a great solution for the bug identified in #5405, but I'd like to write a test to prevent this from happening again. It could be a system test:
Or it could be an integration test, something like this:
Basically we want to try logging in, like here:
https://github.com/publiclab/plots2/blob/master/test/integration/login_flow_test.rb#L22
but with the wrong password, then enter the right password, and show that it really redirects to the dashboard as the solution in #5410 changed.
For anyone looking to learn more about testing, this is a good one!!!
@40404notfound would you be interested in this one at all? We'd love your help! Thanks again!
@jywarren Sure! I am very glad to help.
@40404notfound how's the work going in on this one? Just checking in in case you are stuck!
@IshaGupta18 Thanks a lot ! I just had an extremely busy week so I had to postpone my plan in fixing this issue. Don't worry. I am definitely going to fix this in a few days.
Hi, @jywarren @IshaGupta18 , I just added a test case in #5579 for exposing the bug in #5410. The new test should be able to distinguish a buggy one from a correct one.
Most helpful comment
@jywarren Sure! I am very glad to help.