Open
Bug 490147
Opened 16 years ago
Updated 2 years ago
xpcshell-tests: replace head files by setup functions & imports?
Categories
(Testing :: XPCShell Harness, enhancement)
Testing
XPCShell Harness
Tracking
(Not tracked)
UNCONFIRMED
People
(Reporter: sgautherie, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug, )
Details
While reviewing bug 384339, Jeff wrote (iirc):
{
frankly, if I were to do it again I'd not have head/tail files; you get a setup function and a teardown function, and you put whatever you want in them (or not define them if not necessary)
and you could load files if you wanted to keep common functionality separate
make the dependencies explicit, not implicit by location
that's long-term, not for now
}
Comment 1•8 years ago
|
||
Bug 503613 has removed the tail file functionality, so updating title.
I agree with comment 0 that it'd be good to make the dependencies explicit, this would be especially good for helping eslint to work better. For that, I think we really need bug 1308512 to give us a better module/file import system.
Depends on: esm-ification
Summary: xpcshell-tests: replace head+tail files by setup+teardown functions → xpcshell-tests: replace head files by setup functions & imports?
Updated•7 years ago
|
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•