issues/32: moving tests out of package

gage-russell / pandas-lineage

Apache License 2.0

13 stars 6 forks source link

issues/32: moving tests out of package #33

Open gage-russell opened 1 year ago

gage-russell commented 1 year ago

Moving tests up one directory level outside of the package.

Update CI step to make better usage of poetry.

gage-russell commented 1 year ago

Currently a little confused as to how tests are passing.

I install --with tests, but that doesn't include pyarrow ... the read_parquet tests should fail without pyarrow.

I should need to install --with tests optional to include pyarrow .. I think that maybe needs an update such that test packages include pyarrow, but again a little confused as to why its currently working.

cc @dlstadther

dlstadther commented 1 year ago

Will try to look at this tomorrow

gage-russell commented 1 year ago

Will try to look at this tomorrow

Found that all dependency groups were being installed. I've updated pyproject.toml to:

use the proper dev group for testing/linting packages
properly include pyarrow as an optional dependency with an extras group for parquet usage