test_ipc.html overrides TestRunner.log, but fails to override TestRunner.error. This leads to failures not being counted and test runs going green on TBPL even when tests fail. See <https://tbpl.mozilla.org/php/getParsedLog.php?id=13147301&full=1&branch=mozilla-inbound> for an example.
Kyle, please fix ASAP.
*** This bug has been marked as a duplicate of bug 769017 ***
*** Bug 769017 has been marked as a duplicate of this bug. ***
I'm going to disable test_ipc.html if this isn't fixed by tomorrow.
(In reply to :Ms2ger from comment #4)
> I'm going to disable test_ipc.html if this isn't fixed by tomorrow.
By attaching a patch and requesting a review from an IndexedDB peer? Great. Otherwise, please don't.
I'm on vacation this week and won't have any time to do this, but the benefit of running these tests and exercising this code on all platforms far outweighs missing some random orange logging for broken child process behavior. We have found plenty of broken stuff form logging just the parent side already.
Then fix it yourself, as bent is too lazy to do his job.
And get your facts in order, bent is a peer.
(In reply to :Ms2ger from comment #0)
> test_ipc.html overrides TestRunner.log, but fails to override
> TestRunner.error. This leads to failures not being counted and test runs
> going green on TBPL even when tests fail. See
> php?id=13147301&full=1&branch=mozilla-inbound> for an example.
> Kyle, please fix ASAP.
onTestMessage tries to handle this with a regex, but it probably isn't the right regex. It probably would make sense to transmit both message and error and map them to things more like exact parallels.
(In reply to :Ms2ger from comment #8)
As I tried to explain in comment 5 this is not a very serious issue. It is simply one of many bugs that I need to fix. We're currently catching some, but not all, test failures. You propose that we should disable this test. Doing so would of course mean that we would not catch *any* failures. I don't see how that helps.
I'm currently working on other bugs which are higher priority but I will fix this as soon as I have the time to do so (unless someone else wants to fix it first, of course).
It's a five-line patch. I don't understand why you want to waste time arguing about the fix rather that doing the work.
Disabled again, since clearly nothing is going to happen otherwise
([leave open] needs to be in the whiteboard or the merging tool will close the bug)
(In reply to :Ms2ger from comment #13)
> Disabled again, since clearly nothing is going to happen otherwise
Ms2ger, you disabled these tests *again* w/o following due process. As many people have already told you, this is on bent's list of things to do very soon. It's just that there are other things ahead if this in his immediate priority queue and disabling these tests isn't going to change that priority queue or help these tests get fixed any faster.
The politics around this test disabling aside; the weird thing is that when the test was re-enabled in comment 16 - native Android mochitest-3 went permaorange with bug 781837. Do we have any ideas why?
Aren't the imported tests before indexedDB? Are you sure that there's correlation here and not just coincidence?
Press down arrow once on:
https://tbpl.mozilla.org/?jobname=Android Tegra 250 mozilla-central opt test mochitest-3&noignore=1&rev=2dfbe4b9403b
The second is not so clear visually (you have to ignore the test_event_propagation.html failures), but both seem to confirm that re-enabling test_ipc.html somehow turned bug 781837 permaorange - unless I'm missing something?
(In reply to Kyle Huey [:khuey] (email@example.com) from comment #18)
> Aren't the imported tests before indexedDB? Are you sure that there's
> correlation here and not just coincidence?
Yes I am now.
Filed bug 783513.
Created attachment 655653 [details] [diff] [review]
This fixes the failure case and makes sure that the harness fails if it isn't run in a child process.
Comment on attachment 655653 [details] [diff] [review]
Review of attachment 655653 [details] [diff] [review]:
Please separate the test_ipc fixes and the test_blob stuff.