Oleg Valter
Oleg Valter
Important note: `/coverage` directory is now mounted in dev mode (to generate the necessary assets, set the SimpleCov's formatter to `SimpleCov::Formatter::HTMLFormatter` or use the multi-formatter). If you run the tests...
Probably won't move it from the draft status for a couple more days (want to improve coverage after the latest changes first), but I believe it's ready otherwise. Made re-review...
Just a quick test to check whether #1571 is enough to fix up the error (apparently, not)
> Now that it isn't just me looking at this branch, I should make explicit: > > The tests added on this branch will never pass. They fail deliberately in...
Also it seems like system test 3.1 and 3.2 differ in that only the 3.1 test has the "already logged in" error displayed if I am not mistaken?
> The tests run in a random order each time, so if behaviour is inconsistent it suggests there is still state being carried over from one test to the next....
After doing some cursory testing, from what I can see, one of the issues is that preemptive `log_out` doesn't work simply because it needs a page to be visited -...
> Now that CircleCI is back up I have merged in `develop` and the tests now show basic_user sees zero posts in Main even locally (previously locally 7 were showing...
Related: #1652 (enables `Style/HashSyntax` in `either` mode until we decide which one to use)
`ruby19_no_mixed_keys` makes the most sense to me given that we all seem to agree that the `ruby19` syntax is preferable. No mixed keys seems like a no-brainer here as being...