Closed Bug 1249311 Opened 8 years ago Closed 8 years ago

Desktop FTU/Slideshow needs unit test coverage

Categories

(Hello (Loop) :: Client, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Iteration:
47.3 - Mar 7

People

(Reporter: standard8, Assigned: vidhuran2012)

References

Details

(Whiteboard: [btpp-fix-now])

Attachments

(1 file)

The FTU slideshow landed without any tests. We need to add some tests to cover it.

I would propose some unit tests to cover the slideshow.js (not the vendor file in detail necessarily), and a mochitest to check the general functionality.
Blocks: 1248604
Rank: 15
Priority: -- → P1
Whiteboard: [triage] → [btpp-fix-now]
Rank: 15 → 12
Note: this is for unit tests to run in karma / with the index.html files. Don't forget to update the test/index.html to reference the additional index.html that'll be required.
Summary: Desktop FTU/Slideshow needs test coverage → Desktop FTU/Slideshow needs unit test coverage
Assignee: nobody → vidhuran2012
Attachment #8723529 - Flags: review?(standard8)
Note: comment 0 mentioned about mochitest - that's going to happen in bug 1250107.
Comment on attachment 8723529 [details] [review]
[loop] Vidhuran:bug1249311-ftu-unittests > mozilla:master

Hi Vidhuran, sorry about the delay here, we've been busy getting releases together.

Thank you for the patch, I've just left some comments in github for some minor changes.

Please could you also rebase it against latest master when you update it?

Then if you re-request review to me, we should be able to land it quite quickly.
Attachment #8723529 - Flags: review?(standard8)
Updated the patch based on review comments. Requesting another review.
Attachment #8723529 - Flags: review?(standard8)
Comment on attachment 8723529 [details] [review]
[loop] Vidhuran:bug1249311-ftu-unittests > mozilla:master

That's great, thank you!
Attachment #8723529 - Flags: review?(standard8) → review+
https://github.com/mozilla/loop/commit/c158dc119dfe32a9ed2429229b8aac42781d8ddf
Status: NEW → RESOLVED
Iteration: --- → 47.3 - Mar 7
Closed: 8 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: