-
-
Notifications
You must be signed in to change notification settings - Fork 19.3k
TST: coverage for skipped tests in io/formats/test_to_html.py #22888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Hello @simonjayhawkins! Thanks for submitting the PR.
|
|
|
|
changing: pandas/pandas/io/formats/html.py Line 235 in e45a6c1
to: sentinel = Falsewould pass all tests. |
|
I have reduced the scope of the skipped tests to the truncation part and removed the there are other tests that check that the style is applied. |
|
can you rebase / tests are failng |
Codecov Report
@@ Coverage Diff @@
## master #22888 +/- ##
==========================================
+ Coverage 92.25% 92.27% +0.01%
==========================================
Files 161 161
Lines 51186 51186
==========================================
+ Hits 47222 47230 +8
+ Misses 3964 3956 -8
Continue to review full report at Codecov.
|
|
thanks! |
|
xref #11060 |
git diff upstream/master -u -- "*.py" | flake8 --diffThere are currently 3 skipped tests in
io/formats/test_to_html.py;test_to_html_truncate,test_to_html_truncate_multi_indexandtest_to_html_truncate_multi_index_sparse_offOnly
test_to_html_truncateis required to provide regression coverage for #22786However, it is probably appropriate for any changes to
test_to_html_truncateto also be applied to the other two skipped tests within the scope of this PR.cc @jreback