Closed
Bug 1108967
Opened 10 years ago
Closed 10 years ago
[NFC] NFC Test app for privilege API
Categories
(Firefox OS Graveyard :: NFC, defect)
Tracking
(b2g-v2.2 fixed, b2g-master fixed)
RESOLVED
FIXED
2.2 S4 (23jan)
People
(Reporter: allstars.chh, Assigned: allstars.chh)
References
Details
Attachments
(1 file, 2 obsolete files)
46 bytes,
text/x-github-pull-request
|
timdream
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details | Review |
We should also create a test app in UITest-privilege for the privilege NFC APIs.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → allstars.chh
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
My original impl is to put the tests in uitest-privilege, but QA would like to make it as a seperate app to leverage their testing. After discussed with Tim he agrees to put it as a seperate app.
Assignee | ||
Comment 3•10 years ago
|
||
With Alison's help to turn this into a single app.
Attachment #8541406 -
Attachment is obsolete: true
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8551221 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Attachment #8552335 -
Flags: review?(timdream)
Updated•10 years ago
|
Attachment #8552335 -
Flags: review?(timdream) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8552335 [details] [review]
Gaia Pull Request
[Approval Request Comment]
This is the NFC Test app for testing NFC Privilege API.
And we'd like to uplift to v2.2 so QA can easily run some test.
Current existing apps using NFC API are all still certified (like Gallery, Music, Video, Contacts and System Browser), so QA cannot test NFC Privilege API base on those existing apps.
[Bug caused by] (feature/regressing bug #):
Bug 1042851
[User impact] if declined:
QA cannot test the NFC v2.2 feature.
[Testing completed]:
Manually.
[Risk to taking this patch] (and alternatives if risky):
No, this is just a test app, shouldn't affect any existing Gaia app.
[String changes made]:
No
Attachment #8552335 -
Flags: approval-gaia-v2.2?
Comment 7•10 years ago
|
||
Comment on attachment 8552335 [details] [review]
Gaia Pull Request
looks good to land as its low risk and helps QA to test.
Attachment #8552335 -
Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Comment 8•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•