Skip to content

Commit

Permalink
Update tests to use TileDB Python 0.23.0. (#461)
Browse files Browse the repository at this point in the history
This allows us to verify the "don't import pandas" change.
  • Loading branch information
thetorpedodog authored Sep 15, 2023
1 parent 0ca6a01 commit 42000ab
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 4 deletions.
2 changes: 1 addition & 1 deletion requirements-py3.7.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ python-dateutil==2.8.2
pytz==2023.3.post1
six==1.16.0
tblib==1.7.0
tiledb==0.22.3
tiledb==0.23.0
typing_extensions==4.7.1
urllib3==2.0.4
xarray==0.20.2
Expand Down
2 changes: 1 addition & 1 deletion requirements-py3.9.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ python-dateutil==2.8.2
pytz==2023.3.post1
six==1.16.0
tblib==1.7.0
tiledb==0.22.3
tiledb==0.23.0
typing_extensions==4.7.1
urllib3==2.0.4
xarray==2023.8.0
Expand Down
8 changes: 6 additions & 2 deletions tests/test_basic.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@

import numpy
import numpy as np
import packaging.version as pkgver
import pandas
import pyarrow
import pytest

import tiledb
import tiledb.cloud
Expand All @@ -21,7 +22,10 @@


class BasicTests(unittest.TestCase):
@pytest.mark.xfail(reason="awaits next release of tiledb python package")
@unittest.skipIf(
pkgver.parse(pandas.__version__) < pkgver.parse("1.5"),
"does not work if we have to patch Pandas",
)
def test_dont_import_pandas(self):
# Get a list of all modules from a completely fresh interpreter.
all_mods_str = subprocess.check_output(
Expand Down

0 comments on commit 42000ab

Please sign in to comment.