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)

x86
macOS
defect

Tracking

()

People

(Reporter: ehsan.akhgari, Unassigned)

Details

Sebastian, is this something you're interested to work on?
(No obligation, feel free to say no if not interested!)
(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 =)
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!
Assignee: nobody → 46b

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: 46b → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.