Skip to content

Commit 35c9eba

Browse files
rebecca-palmerjreback
authored andcommitted
Check for pyarrow not feather before pyarrow tests (#31144)
1 parent daef69c commit 35c9eba

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

pandas/tests/io/test_common.py

+11-6
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def test_iterator(self):
129129
(pd.read_csv, "os", FileNotFoundError, "csv"),
130130
(pd.read_fwf, "os", FileNotFoundError, "txt"),
131131
(pd.read_excel, "xlrd", FileNotFoundError, "xlsx"),
132-
(pd.read_feather, "feather", Exception, "feather"),
132+
(pd.read_feather, "pyarrow", IOError, "feather"),
133133
(pd.read_hdf, "tables", FileNotFoundError, "h5"),
134134
(pd.read_stata, "os", FileNotFoundError, "dta"),
135135
(pd.read_sas, "os", FileNotFoundError, "sas7bdat"),
@@ -153,8 +153,11 @@ def test_read_non_existant(self, reader, module, error_class, fn_ext):
153153
msg7 = (
154154
fr"\[Errno 2\] File o directory non esistente: '.+does_not_exist\.{fn_ext}'"
155155
)
156+
msg8 = fr"Failed to open local file.+does_not_exist\.{fn_ext}.?, error: .*"
157+
156158
with pytest.raises(
157-
error_class, match=fr"({msg1}|{msg2}|{msg3}|{msg4}|{msg5}|{msg6}|{msg7})"
159+
error_class,
160+
match=fr"({msg1}|{msg2}|{msg3}|{msg4}|{msg5}|{msg6}|{msg7}|{msg8})",
158161
):
159162
reader(path)
160163

@@ -165,7 +168,7 @@ def test_read_non_existant(self, reader, module, error_class, fn_ext):
165168
(pd.read_table, "os", FileNotFoundError, "csv"),
166169
(pd.read_fwf, "os", FileNotFoundError, "txt"),
167170
(pd.read_excel, "xlrd", FileNotFoundError, "xlsx"),
168-
(pd.read_feather, "feather", Exception, "feather"),
171+
(pd.read_feather, "pyarrow", IOError, "feather"),
169172
(pd.read_hdf, "tables", FileNotFoundError, "h5"),
170173
(pd.read_stata, "os", FileNotFoundError, "dta"),
171174
(pd.read_sas, "os", FileNotFoundError, "sas7bdat"),
@@ -193,9 +196,11 @@ def test_read_expands_user_home_dir(
193196
msg7 = (
194197
fr"\[Errno 2\] File o directory non esistente: '.+does_not_exist\.{fn_ext}'"
195198
)
199+
msg8 = fr"Failed to open local file.+does_not_exist\.{fn_ext}.?, error: .*"
196200

197201
with pytest.raises(
198-
error_class, match=fr"({msg1}|{msg2}|{msg3}|{msg4}|{msg5}|{msg6}|{msg7})"
202+
error_class,
203+
match=fr"({msg1}|{msg2}|{msg3}|{msg4}|{msg5}|{msg6}|{msg7}|{msg8})",
199204
):
200205
reader(path)
201206

@@ -212,7 +217,7 @@ def test_read_expands_user_home_dir(
212217
(pd.read_excel, "xlrd", ("io", "data", "excel", "test1.xlsx")),
213218
(
214219
pd.read_feather,
215-
"feather",
220+
"pyarrow",
216221
("io", "data", "feather", "feather-0_3_1.feather"),
217222
),
218223
(
@@ -249,7 +254,7 @@ def test_read_fspath_all(self, reader, module, path, datapath):
249254
[
250255
("to_csv", {}, "os"),
251256
("to_excel", {"engine": "xlwt"}, "xlwt"),
252-
("to_feather", {}, "feather"),
257+
("to_feather", {}, "pyarrow"),
253258
("to_html", {}, "os"),
254259
("to_json", {}, "os"),
255260
("to_latex", {}, "os"),

0 commit comments

Comments
 (0)