Closed
Bug 1030905
Opened 10 years ago
Closed 10 years ago
write missing Input API tests
Categories
(Input Graveyard :: Code Quality, defect)
Input Graveyard
Code Quality
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: muricula, Mentored)
References
Details
I recently added a GET API for Input feedback. I wrote tests for many of the parameters, but there's a FIXME in the code listing the parameters that have no tests. The code being tested is in fjord/feedback/api_views.py. The tests are in fjord/feedback/tests/test_api.py. This bug covers writing tests for the following parameters: * date_start * date_end * delta * product/version combination
Reporter | ||
Comment 1•10 years ago
|
||
Making this a mentored bug. This involves writing the specified tests that are missing. It's probably the case that you can cut and paste existing test code with some changes to test the new parameters. It's possible that the code being tested is broken. If you think that's the case, let me know. If you're interested in working on this bug, either reply in this bug's comments or ping me on IRC (I'm willkg).
Mentor: willkg
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → ikronquist
Assignee | ||
Comment 2•10 years ago
|
||
PR submitted on github: https://github.com/mozilla/fjord/pull/319
Reporter | ||
Comment 3•10 years ago
|
||
PR 319 landed in master in: https://github.com/mozilla/fjord/commit/e4c7c6c5 Still outstanding: * product/version combination
Reporter | ||
Comment 4•10 years ago
|
||
Added the outstanding tests in https://github.com/mozilla/fjord/commit/4887c49d Since this is specific to the tests and no other changes were made, I'm marking this as FIXED.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Input → Input Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•