Skip to content
This repository was archived by the owner on Mar 14, 2023. It is now read-only.

Adding tests for set_assignee #117

Merged
merged 2 commits into from
Mar 25, 2018
Merged

Adding tests for set_assignee #117

merged 2 commits into from
Mar 25, 2018

Conversation

davidalber
Copy link
Collaborator

@davidalber davidalber commented Mar 23, 2018 •
edited
Loading

This PR adds tests for set_assignee. It also removes a global statement for a global variable not being assigned to.

$ nosetests -s --with-coverage --cover-package=highfive --cover-erase --cover-html
.......................................................
Name                   Stmts   Miss  Cover
------------------------------------------
highfive/__init__.py       0      0   100%
highfive/irc.py           33      4    88%
highfive/newpr.py        282     75    73%
------------------------------------------
TOTAL                    315     79    75%
----------------------------------------------------------------------
Ran 55 tests in 0.567s

OK

@davidalber davidalber mentioned this pull request Mar 25, 2018
Merged
@nrc nrc merged commit d2741d5 into rust-lang:master Mar 25, 2018
@nrc
Copy link
Member

nrc commented Mar 25, 2018

Thanks!

@davidalber davidalber deleted the test-set-assignee branch March 25, 2018 08:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidalber @nrc