ssl-hep / ServiceX_App

REST Front End to ServiceX
BSD 3-Clause "New" or "Revised" License
3 stars 10 forks source link

Reduce boilerplate in unit tests #103

Closed AndrewEckart closed 3 years ago

AndrewEckart commented 3 years ago

Thanks to #96, we can reduce the amount of boilerplate in the unit test suite considerably.

In general, tests now work as follows:

There's still quite a bit of duplicated code throughout the unit test suite that should be refactored to Pytest fixtures, but this is a good start.

codecov[bot] commented 3 years ago

Codecov Report

Merging #103 (ed9166a) into develop (994b970) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #103   +/-   ##
========================================
  Coverage    90.75%   90.75%           
========================================
  Files           46       46           
  Lines         1287     1287           
  Branches       107      107           
========================================
  Hits          1168     1168           
  Misses         106      106           
  Partials        13       13           

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 994b970...ed9166a. Read the comment docs.