Open
Bug 647064
Opened 14 years ago
Updated 2 years ago
Remove the old transaction tests, and replace them with something sane if needed
Categories
(Core :: DOM: Editor, defect)
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
Details
Reporter | ||
Comment 1•14 years ago
|
||
Sebastian, is this something you're interested to work on?
Reporter | ||
Comment 2•14 years ago
|
||
(No obligation, feel free to say no if not interested!)
Comment 3•14 years ago
|
||
(In reply to comment #1)
> Sebastian, is this something you're interested to work on?
Sounds quite interesting. I would be glad to try myself on this one =)
Reporter | ||
Comment 4•14 years ago
|
||
So, the test in question is <http://mxr.mozilla.org/mozilla-central/source/editor/txmgr/tests/TestTXMgr.cpp>. It's supposed to test the transaction manager component, represented by nsITransactionManager <http://mxr.mozilla.org/mozilla-central/source/editor/txmgr/idl/nsITransactionManager.idl#56>.
If you look at the main function in the test file, there are a bunch of tests performed in that test program. The stress tests are needless (and honestly, ridiculous!). I think we should convert the rest of the tests to xpcshell tests <https://developer.mozilla.org/en/Writing_xpcshell-based_unit_tests>.
And I also think that we should have several separate xpcshell test files, each for one of the main categories in this test.
I think this will be an interesting bug for you because you'll learn a lot about the nsITransactionManager, how XPCOM components work, how to write your own components, and how to write xpcshell tests.
Let me know if you need help in any step of the work here. And thanks!
Updated•14 years ago
|
Assignee: nobody → 46b
Comment 5•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: 46b → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•