Closed Bug 1201370 Opened 4 years ago Closed 4 years ago

Remote cppunit test is unable to launch x86 emulator correctly

Categories

(Testing :: General, defect)

defect
Not set

Tracking

(firefox44 fixed)

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: edgar, Assigned: jdai)

References

Details

Attachments

(2 files)

remotecppunittests doesn't tell mozrunner which kind of arch should be launch, so mozrunner use the default one [2]. We have to find some way to pass the arch information to remotecppunuttests.

[1] https://dxr.mozilla.org/mozilla-central/source/testing/remotecppunittests.py?offset=0#230
[2] https://dxr.mozilla.org/mozilla-central/source/testing/mozbase/mozrunner/mozrunner/runners.py?offset=0#87
Blocks: 1192134
Depends on: 1188330
QA Contact: jdai
Assignee: nobody → jdai
QA Contact: jdai
Bug 1201370 - Add emulator architecture parameter to cppunit test; r?ahal
Attachment #8664621 - Flags: review?(ahalberstadt)
Since I can't retrigger failed test, I put 2 try link for you to reference.

Try link:
With x86-kk try error but others are green:
https://treeherder.allizom.org/#/jobs?repo=try&revision=f825cdfbfc3e
Without x86-kk try error:
https://treeherder.allizom.org/#/jobs?repo=try&revision=0b018ffcf7e6
Attachment #8664621 - Flags: review?(ahalberstadt)
Comment on attachment 8664621 [details]
MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test. r=ahal

https://reviewboard.mozilla.org/r/20017/#review18071

::: testing/remotecppunittests.py:230
(Diff revision 1)
> -
> +        self.add_option("--emulator", action="store",
> +                    type="string", dest = "emulator",

nit: the action, type and dest parameters aren't needed here as those are already the default values.

::: testing/remotecppunittests.py:232
(Diff revision 1)
> +                    help = "Architecture of emulator to use: x86 or arm")

You should add a 'choices' parameter, that way it'll fail fast if someone types something other than 'arm' or 'x86':
https://docs.python.org/2.7/library/argparse.html#choices
Comment on attachment 8664621 [details]
MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test. r=ahal

Bug 1201370 - Add emulator parameter to cppunit test; r?ahal
Attachment #8664621 - Attachment description: MozReview Request: Bug 1201370 - Add emulator architecture parameter to cppunit test; r?ahal → MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test; r?ahal
Attachment #8664621 - Flags: review?(ahalberstadt)
Attachment #8664621 - Flags: review?(ahalberstadt) → review+
Comment on attachment 8664621 [details]
MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test. r=ahal

https://reviewboard.mozilla.org/r/20017/#review18195

Thanks!
Comment on attachment 8664621 [details]
MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test. r=ahal

Bug 1201370 - Add emulator parameter to cppunit test. r=ahal
Attachment #8664621 - Attachment description: MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test; r?ahal → MozReview Request: Bug 1201370 - Add emulator parameter to cppunit test. r=ahal
Attachment #8664621 - Flags: review+ → review?(ahalberstadt)
Attachment #8664621 - Flags: review?(ahalberstadt) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/6e77b81bc2a7
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.