Closed Bug 898409 Opened 7 years ago Closed 7 years ago

[User Story] Data Settings Runtime Customization by SIM

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:koi+)

RESOLVED FIXED
blocking-b2g koi+

People

(Reporter: pdol, Assigned: aus)

References

Details

(Keywords: feature, verifyme, Whiteboard: [ucid:System23, FT:systems-fe, KOI:P1][systemsfe],QARegressExclude)

Attachments

(1 file, 1 obsolete file)

User Story:

As an OEM, I want to be able to specify which data APN, Identifer, Password, HTTP proxy host and HTTP proxy port should be used on the device based on the MNC/MCC setting from the SIM card inserted in order to target customizations to locales without needing to use separate builds.


Acceptance Criteria:

1. If a certain data settings (including APN, Identifier, Password, HTTP proxy host and HTTP proxy port) are specified to be included for an MNC/MCC combination, and a SIM card with that MNC/MCC combination is inserted into the device the data settings used by the device are those specified.
2. If no SIM card is inserted, the specified default data settings are used by the device.

Note: this functionality already exists - this user story is just to ensure the inclusion of integration tests and QA test verification.
Whiteboard: [ucid:System23] → [ucid:System23, FT:systems-fe, KOI:P1]
Flags: in-moztrap?(jsmith)
This bug should tackle the bugs that are marked blocking at the same time. Data, MMS and A-GPS settings are all handled by the same underlying code.
Blocks: 898410, 891732
Ghislain 'Aus' Lacroix changed story state to started in Pivotal Tracker
Assignee: nobody → aus
Status: NEW → ASSIGNED
Flags: in-moztrap?(jsmith) → in-moztrap?(jhammink)
Attachment #800951 - Attachment mime type: text/plain → text/html
This patch fixes dependent bug 898410.
Attachment #800951 - Flags: review?(alive) → review?(kaze)
Comment on attachment 800951 [details]
Patch - v2 - Update Operator Variant to use new common helper, Add much needed unit tests!

apn.json is dynamically generated: please don’t update it manually. Instead, you should add your data to apn_conf_local.xml and regenerate apn.json.

Feel free to r? me again when you update your PR.
Attachment #800951 - Flags: review?(kaze)
Comment on attachment 800951 [details]
Patch - v2 - Update Operator Variant to use new common helper, Add much needed unit tests!

Same pull-request but with the requested updates. Used apns_conf_local.xml to add the appropriate test network values and regenerated apn.json. Other minor updates to ensure settings load appropriately and to unit tests.
Attachment #800951 - Attachment description: Patch - v1 - Update Operator Variant to use new common helper, Add much needed unit tests! → Patch - v2 - Update Operator Variant to use new common helper, Add much needed unit tests!
Attachment #800951 - Flags: review?(kaze)
From IRC:

kaze			will r+ as soon as bugzilla is back			1:43
kaze			yes!

Looks like he hasn't gotten to hit bugzilla but the pull request is green and ready to merge so I'm going to go ahead and do that.
Fixed: https://github.com/mozilla-b2g/gaia/commit/cfcea746250d7c6ea01a389c3d7714c471f28641
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Ghislain 'Aus' Lacroix changed story state to finished in Pivotal Tracker
Ghislain 'Aus' Lacroix changed story state to accepted in Pivotal Tracker
Ghislain 'Aus' Lacroix changed story state to accepted in Pivotal Tracker
Keywords: verifyme
Comment on attachment 800951 [details]
Patch - v2 - Update Operator Variant to use new common helper, Add much needed unit tests!

Thanks for merging. :-)
Attachment #800951 - Flags: review?(kaze) → review+
Original test failed wifi tests due to mozPower not being set writable. All this pull request does is revert the revert on github, and add a patch on top to fix writable property, as well as remove unused stubMozPower variable.

Need this fixed to land Bug 891725.
Attachment #800951 - Attachment is obsolete: true
Attachment #803046 - Flags: review?(kaze)
Whiteboard: [ucid:System23, FT:systems-fe, KOI:P1] → [ucid:System23, FT:systems-fe, KOI:P1][systemsfe]
(In reply to Anthony Ricaud (:rik) from comment #13)
> I had to revert this since it is failing one unit test.
> 
> https://github.com/mozilla-b2g/gaia/commit/
> 8b15a3359fc30f3703808763e976cfcab3812724
> 
> [system] WiFi > "before each" hook:
> TypeError: setting a property that has only a getter
> at (anonymous) (http://system.gaiamobile.org:8080/test/unit/wifi_test.js:104)
> at wrapper
> (http://test-agent.gaiamobile.org:8080/common/test/mocha_generators.js:60)
> at run
> (http://test-agent.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3709)
> at next
> (http://test-agent.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3973)
> at (anonymous)
> (http://test-agent.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3984)
> at (anonymous)
> (http://test-agent.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4932)

This also caused bug 915218 as well. Thanks for backing this out.
Depends on: 915218
Comment on attachment 803046 [details] [diff] [review]
Patch - v3 - Update Operator Variant to use new common helper, Add much needed unit tests!

Looks good to me, but redirecting the review to Jose who knows this part better than anyone.
Attachment #803046 - Flags: review?(kaze) → review?(josea.olivera)
Comment on attachment 803046 [details] [diff] [review]
Patch - v3 - Update Operator Variant to use new common helper, Add much needed unit tests!

PR closed. Clearing the review flag here.
Attachment #803046 - Flags: review?(josea.olivera)
Landed as part of pull request for Bug 891725, which also contained revert revert and fix. A sad mess I will try not to repeat.

https://github.com/mozilla-b2g/gaia/pull/12121
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
blocking-b2g: koi? → koi+
Flags: in-moztrap?(jhammink) → in-moztrap?
Aus - In order to test this, I need to get a sample customization showcasing how this would be setup in a real customization. Can you provide a sample?
Flags: needinfo?(aus)
See https://github.com/mozilla-b2g/gaia/tree/master/shared/resources/apn for the README.md

See https://github.com/mozilla-b2g/gaia/blob/master/shared/resources/apn.json#L3179 for what is the final output file that we actually load and parse at run-time. I pointed you to the test data we use.

The 'default' settings are what is used for all basic data connectivity.
Flags: needinfo?(aus)
Flags: in-moztrap? → in-moztrap?(jhammink)
Whiteboard: [ucid:System23, FT:systems-fe, KOI:P1][systemsfe] → [ucid:System23, FT:systems-fe, KOI:P1][systemsfe],QARegressExclude
You need to log in before you can comment on or make changes to this bug.