976520 - test_browserElement_inproc_ErrorSecurity.html seem to be permanently failing when switching to mochitest.ini manifests

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

as part of our big push to reduce complexity and add full support for a single manifest, we are moving entries from b2g.json -> mochitest.ini files.  This is going well except for two tests:
test_browserElement_[NoAttr|NoWhitelist].html

These tests have been failing in bug 970290 intermittently, now they appear to be permanent.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

I r+'d a patch yesterday that converts the dom/browser-element/ tests to mochitest.ini, if it is landing today, I would rather edit that, otherwise I will edit the .json file.

Comment 2

[Kan-Ru Chen [:kanru] (UTC+9)]

I will soon land patches for bug 970290 that make these two tests pass.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

oh, looking forward to those, I will retest on try once you have those landed.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

now that those two tests are passing, we have test_browserElement_inproc_ErrorSecurity.html failing:
https://tbpl.mozilla.org/?tree=Try&rev=5dab0a650aa6

only on emulator opt though!

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

odd, this test is disabled according to the mochitest.ini file:
http://dxr.mozilla.org/mozilla-central/source/dom/browser-element/mochitest/mochitest.ini#176

but in the mochitest-oop.ini, it is enabled:
http://dxr.mozilla.org/mozilla-central/source/dom/browser-element/mochitest/mochitest-oop.ini#11

that mochitest-oop.ini doesn't have the proper support files, but we seem to be executing the support file.

this is really odd because the patch we are testing doesn't change any b2g.json or mochitest.ini file for browser-element tests.

jlebar, you wrote this test originally, any thoughts why it might be causing troubles for us?

Comment 6

[(not currently active) Ted Mielczarek]

It looks like Vivien originally disabled this test and added it to the list that wound up in mochitest-oop:
http://hg.mozilla.org/mozilla-central/rev/ec2f1b9a9726

(Don't get too hung up on the manifests, I did my best to translate from the Makefiles to manifests, but there may have been something lost in translation.)

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

so should we be disabling this?

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

this test was originally disabled in b2g.json, it is an error in the patch we are testing.  So since it was originally disabled, we can mark it as such in the appropriate patch and move on in life.