1333065 - Intermittent test_capabilities.py TestCapabilities.test_additional_capabilities | AssertionError: u'/storage/sdcard/tests/profile' != '/tmp/tmpIqMQt9.mozrunner'

Status: RESOLVED FIXED

(Testing :: Marionette Client and Harness, defect)

Description

[Treeherder Bug Filer]

Filed by: hskupin [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=71162494&repo=autoland

https://queue.taskcluster.net/v1/task/KH5IkPMATBigQ7KTNcGWQw/runs/0/artifacts/public/logs/live_backing.log

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Andrew, do you know why mozrunner reports the local temp directory for the profile location but not the one from the device/emulator? I don't think that the current behavior is expected.

> current_profile = self.marionette.instance.runner.profile.profile
> self.assertEqual(self.caps["moz:profile"], current_profile)

which results in:

> test_capabilities.py TestCapabilities.test_additional_capabilities | AssertionError: u'/storage/sdcard/tests/profile' != '/tmp/tmpIqMQt9.mozrunner'

Comment 2

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

It keeps track of both. 

You want to use something like `self.marionette.instance.runner.device.app_ctx.remote_profile` in the Fennec case.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1333065 - Fix test_additional_capabilities to use the remote profile path for Fennec.

In case of Fennec the remote profile path has to be used for comparing the moz:profile
capability of Marionette.

Review commit: https://reviewboard.mozilla.org/r/106650/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/106650/

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Thanks Maja. I will try that.

Comment 5

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8829631 [details]
Bug 1333065 - Fix test_additional_capabilities to use the remote profile path for Fennec.

https://reviewboard.mozilla.org/r/106650/#review107836

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cef9cfc92fec
Fix test_additional_capabilities to use the remote profile path for Fennec. r=maja_zf

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/cef9cfc92fec

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/7cde5f78d3f5

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Please also uplift this test-only patch to aurora. Thanks.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e0aa7e76bdc5

Comment 11

[Intermittent Failures Robot]

5 failures in 749 pushes (0.007 failures/push) were associated with this bug in the last 7 days.  

Repository breakdown:
* try: 4
* autoland: 1

Platform breakdown:
* android-4-3-armv7-api15: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1333065&startday=2017-01-23&endday=2017-01-29&tree=all

Comment 12

[BMO Automation]

Moving bug to Testing::Marionette Client and Harness component per bug 1815831.