Test Pilot needs better unit tests

RESOLVED FIXED

Status

Mozilla Labs Graveyard
Test Pilot
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: Jono Xia, Unassigned)

Tracking

Details

(Whiteboard: needs-integration)

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
It needs a better unit test framework for one thing, and it needs much more unit test coverage.
(Reporter)

Comment 1

8 years ago
Created attachment 500465 [details] [diff] [review]
Some unit test improvements

This patch adds tests for the test pilot task's date-based state-change logic. To implement them it was necessary to add a parameter to the TestPilotExperiment constructor to pass in a stub date provider object.
Attachment #500465 - Flags: review?(dtownsend)
(Reporter)

Updated

8 years ago
Whiteboard: needs-integration
Attachment #500465 - Flags: review?(dtownsend) → review+
Landed in mozilla-central: http://hg.mozilla.org/mozilla-central/rev/486c465a73fc
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Updated

2 years ago
Product: Mozilla Labs → Mozilla Labs Graveyard
You need to log in before you can comment on or make changes to this bug.