Skip to content

Commit

Permalink
add deps change to CHANGELOG, add reason to skipif
Browse files Browse the repository at this point in the history
  • Loading branch information
scottstanie committed Aug 24, 2023
1 parent 52d4433 commit f1b9245
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@
**Fixed**
- Geometry/`static layers` file creation from new COMPASS changes

**Dependencies**
- Upgraded `pydantic >= 2.1`
- Pinned minimum dolphin version due to mamba weirdness

# [0.1.0](https://github.com/isce-framework/sweets/commits/v0.1.0) - 2023-08-22


Expand Down
8 changes: 4 additions & 4 deletions tests/test_missing_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@


# Fixture for creating temporary files
@pytest.mark.skipif(MISSING_DEPS)
@pytest.mark.skipif(MISSING_DEPS, reason="geopandas/matplotlib not installed")
@pytest.fixture()
def slc_file_list(tmp_path):
shape = (10, 100, 100)
Expand Down Expand Up @@ -45,22 +45,22 @@ def slc_file_list(tmp_path):


# Now we write test cases using the created fixture
@pytest.mark.skipif(MISSING_DEPS)
@pytest.mark.skipif(MISSING_DEPS, reason="geopandas/matplotlib not installed")
def test_get_bad_files(slc_file_list):
bad_files, bad_stats = get_bad_files(str(slc_file_list[0].parent), max_jobs=1)
assert isinstance(bad_files, list)
assert isinstance(bad_stats, list)
get_bad_files(str(slc_file_list[0].parent), max_jobs=5)


@pytest.mark.skipif(MISSING_DEPS)
@pytest.mark.skipif(MISSING_DEPS, reason="geopandas/matplotlib not installed")
def test_is_valid(slc_file_list):
valid, reason = is_valid(str(slc_file_list[0]))
assert valid
assert reason == ""


@pytest.mark.skipif(MISSING_DEPS)
@pytest.mark.skipif(MISSING_DEPS, reason="geopandas/matplotlib not installed")
def test_get_raster_stats(slc_file_list):
stats = get_raster_stats(str(slc_file_list[0]))
assert isinstance(stats, Stats)

0 comments on commit f1b9245

Please sign in to comment.