Closed Bug 901897 Opened 12 years ago Closed 12 years ago

Perma-red test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643

Categories

(Core :: DOM: Device Interfaces, defect)

24 Branch
x86
Gonk (Firefox OS)
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: cbook, Assigned: allstars.chh)

Details

seems this was introduced with the https://tbpl.mozilla.org/?tree=B2g-Inbound&rev=ede868d4787 b2g_emulator b2g-inbound opt test marionette-webapi on 2013-08-05 21:14:15 PDT for push ede868d47878 slave: talos-r3-fed-021 ImportError: cannot import name OrderedDict AssertionError: See errors below and more information in Bug 880643 TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643 Return code: 2560 Marionette exited with return code 2560: harness failures # TBPL FAILURE # https://tbpl.mozilla.org/php/getParsedLog.php?id=26196079&tree=B2g-Inbound Yoshi, could you take a look ?
OS: Mac OS X → Gonk (Firefox OS)
Summary: TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643 → Intermittent TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643
Hi, Tomcat Sorry for the failure. Sure I'd take a look here, but doesn't try-server have this test enabled? My previous try is https://tbpl.mozilla.org/?tree=Try&rev=ae6cd888b2d7 But it didn't have the failure in Comment 0.
Assignee: nobody → allstars.chh
(In reply to Yoshi Huang[:allstars.chh][:yoshi] from comment #1) > but doesn't try-server have this test enabled? > My previous try is https://tbpl.mozilla.org/?tree=Try&rev=ae6cd888b2d7 > But it didn't have the failure in Comment 0. Ed, Ryan: see above (i guess you can answer this question better than i can do). Do you know if this tests are enabled on try. If not can we enable that tests. Seems the reason for this test is "the test is for checking whether the backend js code is well written according to jslint check"
Flags: needinfo?(ryanvm)
Flags: needinfo?(emorley)
Looks like the parent cset of the Try push was c2b375f3a909? Which was from July 30. Is that from before this test landed? Yoshi, did you push against b2g-inbound tip?
Flags: needinfo?(ryanvm)
Flags: needinfo?(emorley)
Backed out. Sorry Yoshi, but while I understand that this test is ultimately checking code quality, not correctness, we can't leave this in if it's causing the test suite to fail. https://hg.mozilla.org/integration/b2g-inbound/rev/cd6bda58b682
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #3) > Looks like the parent cset of the Try push was c2b375f3a909? Which was from > July 30. Is that from before this test landed? Yes it was. Bug 880643 landed on July 31.
Summary: Intermittent TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643 → Perma-red TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643
(Appearing in my missing intermittent-failure keyword bug whine, tweaking summary to avoid this)
Summary: Perma-red TEST-UNEXPECTED-FAIL | test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643 → Perma-red test_ril_code_quality.py TestRILCodeQuality.test_ril_consts | See errors above and more information in Bug 880643
You need to log in before you can comment on or make changes to this bug.