uchchwhash commented 3 years ago

Sorry @benjimin for the mess.

benjimin commented 3 years ago

Could you also rename the test "broken_test_pixeldrill", so that it doesn't resemble a new test framework feature ("xtest") and doesn't refer to Mangroves (unless the test is supposed to)?

whatnick commented 3 years ago

Proper way is to mark the test as xfail via pytest. https://docs.pytest.org/en/6.2.x/skipping.html

codecov[bot] commented 3 years ago

Codecov Report

Merging #107 (861ea00) into master (ad447ee) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 861ea00 differs from pull request most recent head 083e99a. Consider uploading reports for the commit 083e99a to get more accurate results

@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   72.36%   72.36%           
=======================================
  Files           8        8           
  Lines         644      644           
=======================================
  Hits          466      466           
  Misses        178      178

opendatacube / datacube-wps

Mark bad test better. #107

Codecov Report