Refactor unit tests in /voting

RESOLVED FIXED

Status

Mozilla Reps
reps.mozilla.org
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: hoosteeno, Assigned: tasos)

Tracking

Details

(Whiteboard: [kb=1048716] )

See tracking bug 895064 for general goals of this effort. We'll add any additional app-specific changes in comments.
Whiteboard: [kb=1048716]
(Assignee)

Updated

4 years ago
Assignee: nobody → tasos
Status: NEW → ASSIGNED
(Assignee)

Updated

4 years ago
Status: ASSIGNED → NEW
(Assignee)

Comment 1

3 years ago
Every time a new commit is made in the voting app, new tests replace the old ones. Let's keep this bug as a reminder to keep track of the process and close it when all tests are updated.
Assignee: tasos → nobody
(Assignee)

Updated

2 years ago
Assignee: nobody → tasos
Status: NEW → ASSIGNED

Comment 2

2 years ago
Commit pushed to master at https://github.com/mozilla/remo

https://github.com/mozilla/remo/commit/0e626a32703c81d3f64582a9848346e0aff9d6e8
Refactor ReMo tests.

* [fix bug 895074] Refactor voting tests.
* [fix bug 895073] Refactor tests in reports app.
* [fix bug 895068] Refactor unit tests in base app.
* [fix bug 895070] Refactor tests in the profiles app.
* [fix bug 895069] Refactor unit tests in events app.
* Refactor dashboard tests.
* Small PEP8 changes in order to use 99 chars per line.
* Fix PollFactory creation.
** When PollFactory was evaluated in __init__.py `Group` tables didn't exist
** Make `valid_groups` lazy evaluated
* Fix ical template rendering.
* Fix testing mocked mozillians API calls.
* Fix edit featured reps as non-rep.

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.