Skip to content

Commit

Permalink
Test also computed values
Browse files Browse the repository at this point in the history
  • Loading branch information
pnuu committed Nov 11, 2024
1 parent cbc872e commit 85f85af
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions satpy/tests/reader_tests/test_mersi_l1b.py
Original file line number Diff line number Diff line change
Expand Up @@ -525,6 +525,7 @@ def test_all_resolutions(self):
res = reader.load(self.bands_1000 + self.bands_250)
for i in res:
assert res[i].dtype == np.float32
assert res[i].values.dtype == np.float32

Check warning on line 529 in satpy/tests/reader_tests/test_mersi_l1b.py

View check run for this annotation

CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Complex Method

MERSI12llL1BTester.test_all_resolutions has a cyclomatic complexity of 9, threshold = 9. This function has many conditional statements (e.g. if, for, while), leading to lower code health. Avoid adding more conditionals and code to it without refactoring.
if resolution != "250":
assert len(res) == len(self.bands_1000 + self.bands_250)
Expand Down

0 comments on commit 85f85af

Please sign in to comment.