Closed Bug 590195 Opened 14 years ago Closed 14 years ago

ProfileManager should have a full xpcshell test suite

Categories

(Testing Graveyard :: ProfileManager, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: k0scist, Unassigned)

References

Details

There are no tests for ProfileManager....and there should be!  I don't see any need for continuous integration or anything fancy at this point in time, but a nice set of unit/integration tests would go some ways to ensure against regressions.  

See http://k0s.org/mozilla/hg/ProfileManager/file/tip/profilemanager/tests/test.py for a very basic example for the python straw man

I'm imagining a script that uses the command line interface.  If other options are taken, bug 586003 should not block
A very minimal implementation is done:

http://hg.mozilla.org/automation/profilemanager/file/d27926f11d60/tests/testCLI.py

This should be extended and probably made more functional as well.  In addition, xpcshell tests should be written.

I'll leave this bug open for now mostly as a reminder for xpcshell tests.  If no action is taken in a finite time window, its probably better to close and move on.
ProfileManager now support xpcshell tests, a couple of simple ones are at http://hg.mozilla.org/automation/profilemanager/file/tip/xpcshell/unit.

More tests still need to be written!
Summary: ProfileManager should have a test suite → ProfileManager should have a full xpcshell test suite before checking into m-c
Whiteboard: [blocker-1.0]
Summary: ProfileManager should have a full xpcshell test suite before checking into m-c → ProfileManager should have a full xpcshell test suite
We now have xpcshell tests for all the main internal API's.  I'll continue to add/modify tests as needed, but am marking this as finished for now.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [blocker-1.0]
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.