Closed blueyed closed 6 years ago
See the report at: https://codecov.io/gh/GetBlimp/django-rest-framework-jwt/pull/382/tree
I would like to have this before doing any other cleanup.
Regarding those oauth tests and what not, I'd say we get rid of them. If I recall we just had those after #4.
@jpadilla Can you please activate/install the codecov integration on https://codecov.io/gh/GetBlimp/django-rest-framework-jwt?
@blueyed done!
:exclamation: No coverage uploaded for pull request base (
master@6b01b47
). Click here to learn what that means. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #382 +/- ##
=========================================
Coverage ? 90.62%
=========================================
Files ? 14
Lines ? 843
Branches ? 29
=========================================
Hits ? 764
Misses ? 66
Partials ? 13
Flag | Coverage Δ | |
---|---|---|
#codecov | 90.62% <ø> (?) |
|
#dj110 | 87.42% <ø> (?) |
|
#dj111 | 87.42% <ø> (?) |
|
#dj18 | 89.79% <ø> (?) |
|
#dj19 | 89.79% <ø> (?) |
|
#drf31 | 89.79% <ø> (?) |
|
#drf32 | 89.79% <ø> (?) |
|
#drf33 | 89.79% <ø> (?) |
|
#drf34 | 90.62% <ø> (?) |
|
#drf35 | 90.27% <ø> (?) |
|
#drf36 | 90.27% <ø> (?) |
|
#py27 | 90.62% <ø> (?) |
|
#py33 | 89.44% <ø> (?) |
|
#py34 | 89.44% <ø> (?) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 6b01b47...5fed46e. Read the comment docs.
TODO:
@jpadilla djangorestframework-oauth is only supported for py27? Is it necessary for it to cover / test it?