Last modified: 2014-03-03 18:10:22 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T45654, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 43654 - Add unit tests for GuidedTour
Add unit tests for GuidedTour
Status: ASSIGNED
Product: MediaWiki extensions
Classification: Unclassified
GuidedTour (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Matthew Flaschen
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2013-01-05 05:46 UTC by Matthew Flaschen
Modified: 2014-03-03 18:10 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Matthew Flaschen 2013-01-05 05:46:14 UTC 
Add some unit tests for GuidedTour.  These should focus on our logic, e.g. the shouldSkip methods, pingServer, initGuider wrapper, etc.
Comment 1 Matthew Flaschen 2013-01-25 23:20:15 UTC 
Some QUnit tests have now been added, though more are needed.
Comment 2 Andre Klapper 2014-02-28 14:50:35 UTC 
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.
Comment 3 Matthew Flaschen 2014-03-03 18:10:22 UTC 
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.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links