Closed Bug 919047 Opened 11 years ago Closed 11 years ago

Run unit tests on TBPL startup if debug=1

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
Now that TBPL has a debug mode (bug 918741), I finally have a good place to run the unit tests in MVTime.js (bug 610602).
Attachment #808017 - Flags: review?(emorley)
Comment on attachment 808017 [details] [diff] [review]
patch

Review of attachment 808017 [details] [diff] [review]:
-----------------------------------------------------------------

Great idea, thank you :-)

nit: console.error() ? Though happy to defer to you on convention (in which case, maybe also change the .error in Data.js to .log).
Attachment #808017 - Flags: review?(emorley) → review+
(And sorry didn't do this quick review sooner, was away much of Sat-Mon)
Pushed with nit fixed:
https://hg.mozilla.org/webtools/tbpl/rev/ecbd85696df3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 920995
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: