Closed Bug 1085300 Opened 10 years ago Closed 10 years ago

[Loop][Testing] Basic test infrastructure with grunt

Categories

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

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ferjm, Assigned: ferjm)

References

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → ferjmoreno
Blocks: Loopmov_1_1
Attachment #8507806 - Flags: review?(borja.bugzilla)
Attachment #8507806 - Flags: review?(borja.bugzilla) → review?(josea.olivera)
Comment on attachment 8507806 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/firefoxos-loop-client/pull/215/files

LGTM. r=me

I left a comment on the PR. Basically I encourage you to add a default task (the tests one) but It's up to you. Some notes in the README file about the grunt stuff being added would be nice but at this point it is not really necessary.

As you are moving everything to the app/ folder and it seems risky I ran kind of smoke test. Everything works and it seems nothing gets broken when taking this patch. The downside of taking this patch is when uplifting the patches we will still need on the 1.1 branch.
Attachment #8507806 - Flags: review?(josea.olivera) → review+
Thanks! The default task is added in bug 1085485 and I'll add the README today
https://github.com/mozilla-b2g/firefoxos-loop-client/commit/f83a3c5aa90c0f78a8047c7efadace01ad01325b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: