Closed Bug 1164722 Opened 10 years ago Closed 10 years ago

Temporarily disable Notification's RTL tests

Categories

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

Product:
Firefox OS Graveyard
Component:
Gaia::System
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: zbraniecki, Assigned: zbraniecki)

Details

Attachments

(2 files)

[gaia] zbraniecki:1164722-temporarily-disable-notification-rtl-tests > mozilla-b2g:master
46 bytes, text/x-github-pull-request
kgrandon
: review+
Details | Review
[gaia] zbraniecki:1164722-reenable-notification-rtl-tests > mozilla-b2g:master
46 bytes, text/x-github-pull-request
Details | Review
In order to fix bug 1137593 we need to disable some tests, land the gecko patch, land the gaia patch, reenable the tests.
Comment on attachment 8605584 [details] [review] [gaia] zbraniecki:1164722-temporarily-disable-notification-rtl-tests > mozilla-b2g:master Kevin, can you r? this for me. I need to disable those tests to land the Gecko patch and then gaia patch in bug 1137593. I'll reenable them when those two are in.
Attachment #8605584 - Flags: review?(kgrandon)
Comment on attachment 8605584 [details] [review] [gaia] zbraniecki:1164722-temporarily-disable-notification-rtl-tests > mozilla-b2g:master Seems ok to me. I assume you have tested that these tests pass with your gecko and gaia work together?
Attachment #8605584 - Flags: review?(kgrandon) → review+
Yes, I tested it locally and they do.
Commit: https://github.com/mozilla-b2g/gaia/commit/5e4041cc2db39be8e265cb646a43591bbb4ef100 I'll keep this bug open and back out the patch when the patches from bug 1137593 land.
Assignee: nobody → gandalf
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: