Closed
Bug 554805
Opened 15 years ago
Closed 7 years ago
Look at how to test failure cases for bootstrap.js
Categories
(Toolkit :: Add-ons Manager, defect, P2)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: mossop, Unassigned)
References
Details
(Whiteboard: [rewrite])
From bug 553169 comment 21:
> Is it worth checking that bootstrap.js exists and won't throw an exception for
> a
> syntax error?
>
> Actually, I'd like to see tests written for bootstrap.js failure conditions:
> * where bootstrap.js throws an exception in enable,
> * where bootstrap.js throws an exception in disable,
> * where bootstrap.js just has a syntax error and can't be parsed
> * where bootstrap.js throws an exception before enable and disable are
> defined
> * where bootstrap.js throws an exception after enable and disable are defined
> * where bootstrap.js throws an exception after enable is defined, but before
> disable is defined.
|
||
Updated•15 years ago
|
Flags: in-testsuite?
Flags: in-litmus-
|
||
Comment 1•15 years ago
|
||
Let's add a couple more combinations to this, swapping disable and enable:
function disable() {
// do something
}
function enable() {
throw new Error("oh no");
}
|
|
||
Comment 2•15 years ago
|
||
Clarification: swapping disable and enable, to see what happens when disable is defined, but enable can't be defined due to a syntax or script error being thrown.
|
Reporter | |
Updated•15 years ago
|
Assignee: dtownsend → nobody
|
||
Comment 3•7 years ago
|
||
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•