diff --git a/rook/director/director.py b/rook/director/director.py index cb3500f..beb6a17 100644 --- a/rook/director/director.py +++ b/rook/director/director.py @@ -48,7 +48,9 @@ def __init__(self, coll, inputs): self._check_apply_fixes() def use_fixes(self): - return CONFIG[f"project:{self.project}"].get("use_fixes", False) + # TODO: don't use fixes + return False + # return CONFIG[f"project:{self.project}"].get("use_fixes", False) def _check_apply_fixes(self): if ( @@ -118,20 +120,22 @@ def _resolve(self): # If we got here: then WPS will be used, because `self.use_original_files == False` def requires_fixes(self): - if not self.use_fixes(): - return False + # TODO: don't use fixes + return False + # if not self.use_fixes(): + # return False - if self.search_result: - ds_ids = self.search_result.files() - else: - ds_ids = self.coll - for ds_id in ds_ids: - fix = fixer.Fixer(ds_id) + # if self.search_result: + # ds_ids = self.search_result.files() + # else: + # ds_ids = self.coll + # for ds_id in ds_ids: + # fix = fixer.Fixer(ds_id) - if fix.pre_processor or fix.post_processors: - return True + # if fix.pre_processor or fix.post_processors: + # return True - return False + # return False def request_aligns_with_files(self): """ diff --git a/tests/test_director/test_director.py b/tests/test_director/test_director.py index e48035c..9951eaa 100644 --- a/tests/test_director/test_director.py +++ b/tests/test_director/test_director.py @@ -38,7 +38,7 @@ def test_original_files(self): "/rlds_Amon_INM-CM5-0_ssp245_r1i1p1f1_gr1_201501-210012.nc" ] - @pytest.mark.xfail(reason="not relevant") + @pytest.mark.skip(reason="not relevant") def test_pre_checked_not_characterised(self): # raise exception inputs = {"pre_checked": True}