890487 - Test failure "getProperty: Unknown property - identity.encrypted" in testSecurity/testDVCertificate.js

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)

Description

[mario garbi]

This started happening today on MAC OS :

http://mozmill-ci.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ee7c5dd
http://mozmill-ci.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ee7d0cd
http://mozmill-ci.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ee7d529

We can reproduce this locally so it might be a regression in Firefox.

Comment 1

[Andreea Matei [:AndreeaMatei]]

Lets get this skipped until we find the responsible changeset and fix it. Already started to fail a lot.

Comment 2

[Cosmin Malutan, [:cosmin-malutan]]

Attachment: skip patch v1.0

This issue is caused by a change in the name of the identity.encrypted and identity.mixed_content labels from chrome://browser/locale/browser.properties.

Comment 3

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 771605 [details] [diff] [review]
skip patch v1.0

Review of attachment 771605 [details] [diff] [review]:
-----------------------------------------------------------------

Some tests don't have teardownModule(), be careful there and please test the patch. Thanks!

Comment 4

[Andreea Matei [:AndreeaMatei]]

It might be bug 838402 that has caused this. Changeset:
http://hg.mozilla.org/mozilla-central/rev/e6b2cb47bddd

Comment 5

[Cosmin Malutan, [:cosmin-malutan]]

Attachment: skip patch v1.1

Modified patch based on review.

Comment 6

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 771608 [details] [diff] [review]
skip patch v1.1

Review of attachment 771608 [details] [diff] [review]:
-----------------------------------------------------------------

Skipped:
http://hg.mozilla.org/qa/mozmill-tests/rev/f3771baed213 (default)

Comment 7

[Cosmin Malutan, [:cosmin-malutan]]

Attachment: patch_v1.0

Here are the changes from firefox which made tasts to fail:
http://hg.mozilla.org/mozilla-central/rev/e6b2cb47bddd

I changed the labels name from tests so the tests won't fail anymore.

Here are the reports to see they work now:
http://mozmill-crowd.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ef71fbc
http://mozmill-crowd.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ef5f4b5
http://mozmill-crowd.blargon7.com/#/functional/report/a1b02004612785c13cf7c6bf1ef0b6d8

Comment 8

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 771639 [details] [diff] [review]
patch_v1.0

Review of attachment 771639 [details] [diff] [review]:
-----------------------------------------------------------------

Great, thanks!
Landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/e26ea3c00876 (default)

Comment 9

[u279076]

Happened today with the daily mozilla-aurora testrun:
> http://mozmill-daily.blargon7.com/#/functional/report/bccc2ea9ac1e478fa074c699d874b2ac

Comment 10

[u279076]

(In reply to Anthony Hughes, Mozilla QA (:ashughes) from comment #9)
> Happened today with the daily mozilla-aurora testrun:
> > http://mozmill-daily.blargon7.com/#/functional/report/bccc2ea9ac1e478fa074c699d874b2ac

Note this failure is also seen in testGreenLarry.js

Comment 11

[Dave Hunt [:davehunt] [he/him] ⌚BST]

This was caused by missing the update of the mozmill-tests repository on the CI after a merge.