Skip to content

Commit

Permalink
Fix up tests by signing all planetary computer urls (#39)
Browse files Browse the repository at this point in the history
Also pin testing env under zarr 3 and pystac 1.12:
* pystac 1.12 migrates to projection extension v2 which is not
  supported by stackstac (gjoseph92/stackstac#262)
* zarr 3 has as-yet unresolved issues with fsspec
  • Loading branch information
jsignell authored Jan 31, 2025
1 parent 7b77ecc commit 466152f
Show file tree
Hide file tree
Showing 9 changed files with 91 additions and 725 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest]
python-version: ["3.8", "3.9", "3.10", "3.11"]
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
environment-file: ["environment.yaml"]
defaults:
run:
Expand All @@ -46,4 +46,4 @@ jobs:

- name: run tests
id: status
run: pytest -v --cov xpystac --cov-report term-missing --vcr-record=none .
run: pytest -v --cov xpystac --cov-report term-missing .
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ ref: https://planetarycomputer.microsoft.com/docs/quickstarts/reading-zarr-data/
## Install

```bash
pip install git+https://github.com/stac-utils/xpystac
pip install xpystac
```

## How it works
Expand Down
7 changes: 3 additions & 4 deletions environment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ channels:
dependencies:
- python
# required
- pystac
- pystac<1.12
- xarray
# optional
- adlfs
Expand All @@ -17,9 +17,8 @@ dependencies:
- requests
- rioxarray
- stackstac
- urllib3<2 # temporary pin https://github.com/stac-utils/pystac-client/issues/509
- zarr
- urllib3
- zarr<3
# testing
- pytest
- pytest-cov
- pytest-vcr
322 changes: 0 additions & 322 deletions tests/cassettes/fixtures/simple_reference_file.yaml

This file was deleted.

53 changes: 0 additions & 53 deletions tests/cassettes/fixtures/simple_search.yaml

This file was deleted.

Loading

0 comments on commit 466152f

Please sign in to comment.