search

jazzband / django-push-notifications

Send push notifications to mobile devices through GCM or APNS in Django.
MIT License
2.26k stars 613 forks source link

chore: dry tests, use newer DRF on tests #641

Closed bertonha closed 2 years ago

bertonha commented 2 years ago

Looking at the docs of tox-gh-action

we could not have the test logic on both places, reducing the duplication and complexity of testing

the new DRF 3.13.0 is full compatible with django 4.0 and it does not need anymore the pytz package

codecov[bot] commented 2 years ago

Codecov Report

Merging #641 (cb4c30e) into master (554f381) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #641   +/-   ##
=======================================
  Coverage   68.33%   68.33%           
=======================================
  Files          25       25           
  Lines        1102     1102           
  Branches      240      240           
=======================================
  Hits          753      753           
  Misses        312      312           
  Partials       37       37

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 554f381...cb4c30e. Read the comment docs.