search

sanger / sequencescape

Web based LIMS
MIT License
84 stars 33 forks source link

Remove skipped tests #4134

Closed sdjmchattie closed 3 months ago

sdjmchattie commented 3 months ago

These come up on every pull request we submit for Sequencescape and create noise where real issues might be appearing. I have kept all the tests that I can, but these were added by Eduardo between 9 months and over a year ago and have not been looked at since. If they are going to be skipped that long, they don't hold any value and should be removed.

codeclimate[bot] commented 3 months ago

Code Climate has analyzed commit db110411 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 86.7% (0.1% change).

View more on Code Climate.