Skip to content

Commit

Permalink
TEST-#2291: use class scope fixture
Browse files Browse the repository at this point in the history
Signed-off-by: Alexander Myskov <[email protected]>
  • Loading branch information
amyskov committed Dec 1, 2020
1 parent 6ea7711 commit 1851494
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions modin/pandas/test/test_io.py
Original file line number Diff line number Diff line change
Expand Up @@ -313,6 +313,7 @@ def TestReadCSVFixture():
"test_read_csv_regular",
"test_read_csv_blank_lines",
"test_read_csv_yes_no",
"test_read_csv_nans",
]
# each xdist worker spawned in separate process with separate namespace and dataset
pytest.csvs_names = {file_id: get_unique_filename() for file_id in files_ids}
Expand All @@ -330,6 +331,12 @@ def TestReadCSVFixture():
filename=pytest.csvs_names["test_read_csv_blank_lines"],
add_blank_lines=True,
)
# test_read_csv_nans_handling
_make_csv_file(filenames)(
filename=pytest.csvs_names["test_read_csv_nans"],
add_blank_lines=True,
additional_col_values=["<NA>", "N/A", "NA", "NULL", "custom_nan", "73"],
)

yield
# Delete csv files that were created
Expand Down Expand Up @@ -752,9 +759,8 @@ def test_read_csv_mangle_dupe_cols(self):
@pytest.mark.parametrize("na_filter", [True, False])
@pytest.mark.parametrize("verbose", [True, False])
@pytest.mark.parametrize("skip_blank_lines", [True, False])
def test_read_csv_na_handling(
def test_read_csv_nans_handling(
self,
make_csv_file,
na_values,
keep_default_na,
na_filter,
Expand All @@ -768,14 +774,8 @@ def test_read_csv_na_handling(
"verbose": verbose,
"skip_blank_lines": skip_blank_lines,
}
unique_name = get_unique_filename("test_read_csv_na_handling", kwargs)
make_csv_file(
filename=unique_name,
add_blank_lines=True,
additional_col_values=["<NA>", "N/A", "NA", "NULL", "custom_nan", "73"],
)
eval_io(
filepath_or_buffer=unique_name,
filepath_or_buffer=pytest.csvs_names["test_read_csv_nans"],
fn_name="read_csv",
**kwargs,
)
Expand Down

0 comments on commit 1851494

Please sign in to comment.