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)

x86_64
Linux
defect
Not set
normal

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.
(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
Attachment #8455658 - Flags: review?(zcampbell)
(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)
Sure, I've already lined up test changes to deal with the 'data model' changes.
Flags: needinfo?(zcampbell)
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
Attachment #8455658 - Flags: review?(zcampbell) → review+
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 → ---
(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?
Attachment #8456438 - Flags: review?(mozbugs.retornam)
Flags: needinfo?(dwong)
Attachment #8456438 - Flags: review?(mozbugs.retornam) → review+
Thanks guys!
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: