Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testing required for calculate_AP method #152

Closed
markgdawson opened this issue Aug 12, 2019 · 1 comment · Fixed by markgdawson/cogs3#46
Closed

Testing required for calculate_AP method #152

markgdawson opened this issue Aug 12, 2019 · 1 comment · Fixed by markgdawson/cogs3#46
Assignees
@edbennett
Labels
urgent needs fixing before next PR to live

Comments

@markgdawson
Copy link
Collaborator

This functionality appears twice, in model and in manage.py script. Both need testing.
@markgdawson markgdawson added the urgent needs fixing before next PR to live label Aug 12, 2019
@edbennett edbennett self-assigned this Aug 13, 2019
edbennett added a commit that referenced this issue Aug 13, 2019
@edbennett
add tests for Project.AP(); fix #152
4d0b2f2
@edbennett edbennett mentioned this issue Aug 13, 2019
Merged
@edbennett
Copy link
Owner

testing of remaining functionality is covered by #131, so once m4rkd#46 is merged, this can be closed

edbennett added a commit that referenced this issue Aug 14, 2019
@edbennett
add tests for Project.AP(); fix #152
29b8f32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edbennett edbennett

Labels
urgent needs fixing before next PR to live
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor Project.AP() and add unit tests for it edbennett/cogs3
2 participants
@edbennett @markgdawson