write missing Input API tests

RESOLVED FIXED

Status

Input Graveyard
Code Quality
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: willkg, Assigned: muricula, Mentored)

Tracking

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
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

4 years ago
Assignee: nobody → ikronquist
(Assignee)

Comment 2

4 years ago
PR submitted on github: https://github.com/mozilla/fjord/pull/319
Blocks: 1040773
PR 319 landed in master in: https://github.com/mozilla/fjord/commit/e4c7c6c5

Still outstanding:

* product/version combination
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Product: Input → Input Graveyard
You need to log in before you can comment on or make changes to this bug.