Skip to content

Commit

Permalink
CLN: pandas-dev#28926 Fix mypy errors in pandas/tests/io/parser/conft…
Browse files Browse the repository at this point in the history
…est.py (pandas-dev#28973)
  • Loading branch information
aaditya-panik authored and proost committed Dec 19, 2019
1 parent 0a8461f commit 1aae8f8
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
9 changes: 5 additions & 4 deletions pandas/tests/io/parser/conftest.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,15 @@
import os
from typing import List, Optional

import pytest

from pandas import read_csv, read_table


class BaseParser:
engine = None
engine = None # type: Optional[str]
low_memory = True
float_precision_choices = []
float_precision_choices = [] # type: List[Optional[str]]

def update_kwargs(self, kwargs):
kwargs = kwargs.copy()
Expand Down Expand Up @@ -59,11 +60,11 @@ def csv1(csv_dir_path):

_py_parsers_only = [_pythonParser]
_c_parsers_only = [_cParserHighMemory, _cParserLowMemory]
_all_parsers = _c_parsers_only + _py_parsers_only
_all_parsers = [*_c_parsers_only, *_py_parsers_only]

_py_parser_ids = ["python"]
_c_parser_ids = ["c_high", "c_low"]
_all_parser_ids = _c_parser_ids + _py_parser_ids
_all_parser_ids = [*_c_parser_ids, *_py_parser_ids]


@pytest.fixture(params=_all_parsers, ids=_all_parser_ids)
Expand Down
3 changes: 0 additions & 3 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -205,9 +205,6 @@ ignore_errors=True
[mypy-pandas.tests.io.json.test_ujson]
ignore_errors=True

[mypy-pandas.tests.io.parser.conftest]
ignore_errors=True

[mypy-pandas.tests.io.test_sql]
ignore_errors=True

Expand Down

0 comments on commit 1aae8f8

Please sign in to comment.