Closed
Bug 927642
Opened 11 years ago
Closed 11 years ago
Readd integration tests for notification lang/direction in gaia
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(b2g-v1.2 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v1.2 | --- | fixed |
People
(Reporter: qdot, Assigned: qdot)
References
Details
Attachments
(2 files, 4 obsolete files)
46 bytes,
patch
|
mikehenrty
:
review+
|
Details | Diff | Splinter Review |
46 bytes,
patch
|
mikehenrty
:
review+
|
Details | Diff | Splinter Review |
Due to some conflict issues between 915002 and 899574, lang/dir integration tests for 915002 got stomped. Readd tests. Blocking because it should've been part of 915002, which blocked koi+.
Assignee | ||
Comment 1•11 years ago
|
||
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] [Bug caused by] (feature/regressing bug #): Bug 915002 [User impact] if declined: Less testing [Testing completed]: This IS tests. [Risk to taking this patch] (and alternatives if risky): None [String changes made]: None
Attachment #818112 -
Flags: review?(mhenretty)
Attachment #818112 -
Flags: approval-gaia-v1.2?(anygregor)
Updated•11 years ago
|
blocking-b2g: koi? → ---
Comment 2•11 years ago
|
||
Comment on attachment 818112 [details] [review] Patch 1 (v1) - Add lang/dir integration tests for notifications test-only. Doesn't need approval.
Attachment #818112 -
Flags: approval-gaia-v1.2?(anygregor)
Comment 3•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/041bb1ec91ab
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•11 years ago
|
||
Merge was accidental, Subsequently backed out
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 5•11 years ago
|
||
Comment on attachment 818112 [details] [review] Patch 1 (v1) - Add lang/dir integration tests for notifications Comment's on github. I don't think we are going to uplift Notification.get() (bug 899574) to 1.2, so these tests can't get uplifted either since they use that feature.
Assignee | ||
Comment 6•11 years ago
|
||
Attachment #819250 -
Flags: review?(mhenretty)
Assignee | ||
Comment 7•11 years ago
|
||
Updated to address review comments
Attachment #818112 -
Attachment is obsolete: true
Attachment #818112 -
Flags: review?(mhenretty)
Attachment #819255 -
Flags: review?(mhenretty)
Comment 8•11 years ago
|
||
Comment on attachment 819255 [details] [review] Patch 1 (v2) - Add lang/dir integration tests for notifications LGTM
Attachment #819255 -
Flags: review?(mhenretty) → review+
Comment 9•11 years ago
|
||
Comment on attachment 819250 [details] [diff] [review] Patch 2 (v1) - Gaia v1.2 - Add lang/dir integration tests for notifications Review of attachment 819250 [details] [diff] [review]: ----------------------------------------------------------------- LGTM ::: apps/system/test/marionette/notification_test.js @@ +1,4 @@ > var assert = require('assert'), > + NotificationTest = require('./lib/notification').NotificationTest, > + NotificationList = require('./lib/notification').NotificationList, > + Marionette = require('marionette-client'), Nit: don't think we use this.
Attachment #819250 -
Flags: review?(mhenretty) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Asking for rereview because this changed pretty massively.
Attachment #819255 -
Attachment is obsolete: true
Attachment #824118 -
Flags: review?(mhenretty)
Assignee | ||
Comment 11•11 years ago
|
||
Fixed nits and optional argument issues.
Attachment #824118 -
Attachment is obsolete: true
Attachment #824118 -
Flags: review?(mhenretty)
Attachment #824179 -
Flags: review?(mhenretty)
Comment 12•11 years ago
|
||
Comment on attachment 824179 [details] [diff] [review] Patch 1 (v4) - Add lang/dir integration tests for notifications LGTM. Let's wait for Travis to go green and land.
Attachment #824179 -
Flags: review?(mhenretty) → review+
Assignee | ||
Comment 13•11 years ago
|
||
Patch 1 Landed to master https://github.com/qdot/gaia/commit/ab40624fc6947b8cb7eed68bac09e6ecbdc285ef
Assignee | ||
Comment 15•11 years ago
|
||
Rereview of the tests for v1.2. Only real difference between m-c and this is the persisting of the notification by hanging on wrappedJSObject.
Attachment #819250 -
Attachment is obsolete: true
Attachment #824398 -
Flags: review?(mhenretty)
Updated•11 years ago
|
Attachment #824398 -
Flags: review?(mhenretty) → review+
Comment 16•11 years ago
|
||
v1.2: https://github.com/mozilla-b2g/gaia/commit/063a22e36a0641970176206e1ed3998202e14d37
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-b2g-v1.2:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•