Closed
Bug 1219695
Opened 9 years ago
Closed 9 years ago
Implement *test_privileged_app_contacts_prompt.py* as an integration test in JavaScript
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(feature-b2g:2.6+)
People
(Reporter: whsu, Assigned: mikehenrty)
References
Details
(Whiteboard: [gip-to-gij][systemsfe])
Attachments
(1 file)
Reporter | ||
Updated•9 years ago
|
Component: Gaia::UI Tests → Gaia::System
Updated•9 years ago
|
feature-b2g: --- → 2.6+
Assignee | ||
Comment 1•9 years ago
|
||
Johan, Is this test still valid? On a fresh build, if I open the UI tests - Priviliged app and go to "API/contacts" and then click "Insert fake contacts," I don't get a permission prompt. I see in the test that when we set up the profile we set the permissions level to "prompt," but looking at MDN [1] prompt should be the default. Do you know what is happening here? 1.) https://developer.mozilla.org/en-US/Apps/Build/App_permissions
Flags: needinfo?(jlorenzo)
Thanks for calling this out. This test was added in bug 948826 (as a result of bug 853356 comment 214). When the final patch landed, the contacts prompt was already forced[1]. However, the patch was backed out a couple of times before having it final form. It busted the builds. At [2], you can see the prompt not being explicitly set. None of the comments bug 948826 mentions this change. Based on the documentation and what happens on other products, I would expect the prompt to be displayed. Bug 853356 regressed the contacts prompt. Maybe we had another regression before that test landed. NI'ing Al, who might remember the context at that time. [1] https://github.com/mozilla-b2g/gaia/blob/9dced7f167784e330e4b67991d8ead934eb697ba/tests/python/gaia-ui-tests/gaiatest/tests/functional/system/test_privileged_app_contacts_prompt.py#L14 [2] https://github.com/mozilla-b2g/gaia/blob/24d9c772fe6eb62e9ad8ae63e6d4b2ed569f6299/tests/python/gaia-ui-tests/gaiatest/tests/functional/system/test_privileged_app_contacts_prompt.py
Flags: needinfo?(jlorenzo) → needinfo?(atsai)
Assignee | ||
Comment 3•9 years ago
|
||
Sounds like there might be a real bug in the product here. I can't test this functionality until that is fixed. So I'll wait to hear back from Al before moving forward here.
![]() |
||
Comment 4•9 years ago
|
||
Sorry. I took a PTO last Friday and couldn't reply in time. I'll take a look today.
![]() |
||
Comment 5•9 years ago
|
||
I think it's a bug. We should have a prompt while inserting/reading a contact via Privileged app first time.
Flags: needinfo?(atsai)
Based on the branches already impacted (see bug 1227011), this bug likely contains sensitive information. Restricting it to Mozilla's employees + cc'd contractors.
Group: mozilla-employee-confidential
Assignee | ||
Comment 7•9 years ago
|
||
Fernando, do you know who handles the contacts permission prompt these days?
Flags: needinfo?(ferjmoreno)
Assignee | ||
Comment 8•9 years ago
|
||
Sorry for the noise, it looks like most of the discussion is happening on bug 1219695. Let's move the conversation there and leave this for converting the test.
Flags: needinfo?(ferjmoreno)
Assignee | ||
Comment 9•9 years ago
|
||
Looks like the problem mentioned in comment 2 only affects pre-installed privileged apps, so we don't need to protect this one.
Group: mozilla-employee-confidential
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → mhenretty
Whiteboard: [gip-to-gij] → [gip-to-gij][systemsfe]
Target Milestone: --- → 2.6 S2 - 12/4
Comment 11•9 years ago
|
||
Assignee | ||
Comment 12•9 years ago
|
||
Comment on attachment 8695540 [details]
[gaia] mikehenrty:bug-1219695-contacts-prompt-test > mozilla-b2g:master
Guillaume, could you take a look here?
Attachment #8695540 -
Flags: review?(gmarty)
Comment 13•9 years ago
|
||
Comment on attachment 8695540 [details]
[gaia] mikehenrty:bug-1219695-contacts-prompt-test > mozilla-b2g:master
Looks good to me!
Attachment #8695540 -
Flags: review?(gmarty) → review+
Assignee | ||
Comment 14•9 years ago
|
||
master: https://github.com/mozilla-b2g/gaia/commit/6137497044ae1c24c06383caec6121567d64bf18
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 1230099
You need to log in
before you can comment on or make changes to this bug.
Description
•