Telemetry tests are currently xpcshell. This makes Telemetry incompatible with workers and OS.File. We should port them to mochitests before we consider porting Telemetry I/O to OS.File.

Porting test_nsITelemetry.js. Some of the conversion was done manually, some with regexps. Taras, what do you think of the result? A few notes: - there were a number of errors and inconsistencies in the original code, I have tried to fix them along the way; - atm, this test leaks a string buffer – not quite sure where.

Comment on attachment 669122 [details] [diff] [review] Porting test_nsITelemetry.js Nathan owns this now.

Comment on attachment 669122 [details] [diff] [review] Porting test_nsITelemetry.js Review of attachment 669122 [details] [diff] [review]: ----------------------------------------------------------------- Looks sane enough to me.

There is ongoing work to make Components go away...I can't find the bug right now, my searches are not working in bugzilla or zimbra. I'm sure folks would appreciate if you looked up what was going on there and tried to write the tests according to the New World Order. :)

TUrns out that we may not need this at all: bug 808161.