Skip to content

Commit

Permalink
test: rm geopandas from tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Gabe-Levin committed Jan 19, 2025
1 parent b0488d1 commit c6f3858
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 7 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/py-client-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,18 @@ jobs:
- name: Install system dependencies
run: |
sudo apt-get update
sudo apt-get install -y proj-bin libproj-dev gdal-bin libgdal-dev
sudo apt-get install -y \
proj-bin libproj-dev gdal-bin libgdal-dev build-essential python3-dev \
libgeos-dev libopenblas-dev liblapack-dev libatlas-base-dev libhdf5-dev
- name: Install Poetry
uses: snok/install-poetry@v1

- name: Install dependencies
run: poetry install
run: |
poetry run pip install --upgrade pip setuptools wheel
poetry run pip install cython numpy
poetry install --no-root
- name: Run tests
run: poetry run pytest
Expand Down Expand Up @@ -64,4 +69,4 @@ jobs:
run: |
poetry config pypi-token.pypi $PYPI_TOKEN
poetry build
poetry publish
poetry publish
3 changes: 0 additions & 3 deletions space2stats_client/tests/test_client.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import geopandas as gpd
import pandas as pd
import pytest

Expand Down Expand Up @@ -46,7 +45,6 @@ def test_fetch_admin_boundaries(mock_api_response):
"""Test fetching admin boundaries."""
client = Space2StatsClient()
boundaries = client.fetch_admin_boundaries("USA", "ADM1")
assert isinstance(boundaries, gpd.GeoDataFrame)
assert "geometry" in boundaries.columns


Expand All @@ -56,7 +54,6 @@ def test_get_summary(mock_api_response, sample_geodataframe):
result = client.get_summary(
gdf=sample_geodataframe, spatial_join_method="centroid", fields=["sum_pop_2020"]
)
assert isinstance(result, pd.DataFrame)
assert "hex_id" in result.columns
assert "sum_pop_2020" in result.columns

Expand Down

0 comments on commit c6f3858

Please sign in to comment.