[Stingray] Rename all _test folder as test under tv_apps/

RESOLVED FIXED

Status

Firefox OS
Gaia::TV
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: dwi2, Assigned: dwi2)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
This is precondition for enabling unit test of TV apps.
1. Rename all _test folder as test under tv_apps/
2. Fix all these unit tests.
(Assignee)

Updated

3 years ago
Assignee: nobody → tzhuang
Status: NEW → ASSIGNED
Created attachment 8590047 [details] [review]
[gaia] dwi2:bug1152638 > mozilla-b2g:master
(Assignee)

Comment 2

3 years ago
Comment on attachment 8590047 [details] [review]
[gaia] dwi2:bug1152638 > mozilla-b2g:master

Hi John, Rex,

Please help to review the patch. 
This patch rename all '_test' folder to 'test', and fix interactive_notification_test.js in smart-system. 
When we done moving tv_apps/ to apps/ in bug 1142372, all these unit tests will be enabled by default.
Attachment #8590047 - Flags: review?(rexboy)
Attachment #8590047 - Flags: review?(im)
Comment on attachment 8590047 [details] [review]
[gaia] dwi2:bug1152638 > mozilla-b2g:master

Looks good to me. Thanks for enabling unit-tests in TV.
Attachment #8590047 - Flags: review?(im) → review+
Comment on attachment 8590047 [details] [review]
[gaia] dwi2:bug1152638 > mozilla-b2g:master

Looks good to me on the card_manager part.

Maybe someday we can move test of card_manager to some shared place if possible.
Attachment #8590047 - Flags: review?(rexboy) → review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Whiteboard: [partner-cherry-pick]

Updated

3 years ago
Whiteboard: [partner-cherry-pick]
You need to log in before you can comment on or make changes to this bug.