Skip to content

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Nov 21, 2018

Adds more idiom to the following:

  • parser/skiprows --> parser/test_skiprows
  • parser/na_values --> parser/test_na_values
  • parser/parse_dates --> parser/test_parse_dates
  • parser/multithread --> parser/test_multi_thread

Also adds new test suite: test_date_converters (was previously hidden in parser/parse_dates)

Builds off of #23712.

@gfyoung gfyoung added Refactor Internal refactoring of code Testing pandas testing functions or related to the test suite IO CSV read_csv, to_csv labels Nov 21, 2018
@gfyoung gfyoung added this to the 0.24.0 milestone Nov 21, 2018
@pep8speaks
Copy link

Hello @gfyoung! Thanks for submitting the PR.

@gfyoung gfyoung force-pushed the pytest-idiom-parsers-more branch from 9c4835b to 719d66d Compare November 21, 2018 08:35
@codecov
Copy link

codecov bot commented Nov 21, 2018

Codecov Report

Merging #23832 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #23832      +/-   ##
==========================================
- Coverage   92.29%   92.28%   -0.01%     
==========================================
  Files         161      161              
  Lines       51500    51500              
==========================================
- Hits        47531    47528       -3     
- Misses       3969     3972       +3
Flag Coverage Δ
#multiple 90.68% <ø> (-0.01%) ⬇️
#single 42.31% <ø> (ø) ⬆️
Impacted Files Coverage Δ
pandas/io/parsers.py 95.36% <0%> (-0.2%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e01c38...2180379. Read the comment docs.

@gfyoung gfyoung force-pushed the pytest-idiom-parsers-more branch from 719d66d to c52fa7b Compare November 21, 2018 09:22
Adds more idiom to the following:

* parser/skiprows --> parser/test_skiprows
* parser/na_values --> parser/test_na_values
* parser/parse_dates --> parser/test_parse_dates
* parser/multithread --> parser/test_multi_thread

Also adds new test suite: test_date_converters

Builds off of pandas-devgh-23712.
@jreback
Copy link
Contributor

jreback commented Nov 21, 2018

thanks @gfyoung

@gfyoung gfyoung deleted the pytest-idiom-parsers-more branch November 21, 2018 19:46
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Adds more idiom to the following:

* parser/skiprows --> parser/test_skiprows
* parser/na_values --> parser/test_na_values
* parser/parse_dates --> parser/test_parse_dates
* parser/multithread --> parser/test_multi_thread

Also adds new test suite: test_date_converters

Builds off of pandas-devgh-23712.
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Adds more idiom to the following:

* parser/skiprows --> parser/test_skiprows
* parser/na_values --> parser/test_na_values
* parser/parse_dates --> parser/test_parse_dates
* parser/multithread --> parser/test_multi_thread

Also adds new test suite: test_date_converters

Builds off of pandas-devgh-23712.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO CSV read_csv, to_csv Refactor Internal refactoring of code Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants