Closed
Bug 1038079
Opened 10 years ago
Closed 10 years ago
Change all IMEI in all testvars files to be an array with one or both IMEI codes.
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: zcampbell, Assigned: dwong)
Details
Attachments
(2 files)
In order to be able to test with dual IMEI numbers we need to update the testvars files to be an array.
Comment 1•10 years ago
|
||
(In reply to Zac C (:zac) from comment #0) > In order to be able to test with dual IMEI numbers we need to update the > testvars files to be an array. This has to be done manually since there is no way to do this automatically.I'll have to take all the phones down to do this at a sitting. We should have kept the dual IMEI pull requests i submitted earlier.
Assignee: nobody → dwong
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8455658 -
Flags: review?(zcampbell)
Comment 3•10 years ago
|
||
(In reply to Dylan Chun Wong [:dwong] from comment #2) > Created attachment 8455658 [details] [review] > PR to change all testvars to use arrays for IMEI's Zac, can you take a look at the pull request since this changes for the Hamachi's as well,
Flags: needinfo?(zcampbell)
Reporter | ||
Comment 4•10 years ago
|
||
Sure, I've already lined up test changes to deal with the 'data model' changes.
Flags: needinfo?(zcampbell)
Reporter | ||
Comment 5•10 years ago
|
||
Merged! https://github.com/mozilla/webqa-credentials/commit/ef3a910070c2d1145a1bb383fa7657f8a725ca75 Thanks Dylan for your hard work on this.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•10 years ago
|
Attachment #8455658 -
Flags: review?(zcampbell) → review+
Comment 6•10 years ago
|
||
I looked over the PR and I see that in 2 of the files the imei list contains the same code twice. The files in question are: b2g-2.json b2g-17.2.json
Status: RESOLVED → REOPENED
Flags: needinfo?(dwong)
Resolution: FIXED → ---
Comment 7•10 years ago
|
||
(In reply to Robert Chira [:RobertC] from comment #6) > I looked over the PR and I see that in 2 of the files the imei list contains > the same code twice. > The files in question are: > b2g-2.json > b2g-17.2.json Dwong can you please look at this?
Assignee | ||
Comment 8•10 years ago
|
||
Attachment #8456438 -
Flags: review?(mozbugs.retornam)
Flags: needinfo?(dwong)
Updated•10 years ago
|
Attachment #8456438 -
Flags: review?(mozbugs.retornam) → review+
Comment 9•10 years ago
|
||
Thanks guys!
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Comment 10•10 years ago
|
||
https://github.com/mozilla/webqa-credentials/commit/eb3597586fd41a95cdeaf66c208396223107bfe2
You need to log in
before you can comment on or make changes to this bug.
Description
•