Last modified: 2014-03-03 18:10:22 UTC
Add some unit tests for GuidedTour. These should focus on our logic, e.g. the shouldSkip methods, pingServer, initGuider wrapper, etc.
Some QUnit tests have now been added, though more are needed.
Matthew Flaschen: You assigned this issue to yourself 12 months ago. Could you please provide a status update and inform us whether you are still working (or still plan to work) on this issue? Only in case you do not plan to work on this issue anymore, should the assignee be set back to default and the bug status changed from ASSIGNED to NEW/UNCONFIRMED? Thanks.
This is kind of a general bug. However, I'm working on an API change right now (https://gerrit.wikimedia.org/r/#/c/116228/), so this is a good time to work on unit tests. I'll try to add some more, then mark this fixed. We can add more tests in the future, and possibly make bugs if something in particular needs to be tested but isn't yet.