tempo icon indicating copy to clipboard operation
tempo copied to clipboard

[DRAFT] Migrate unit tests to chispa

Open R7L208 opened this issue 2 years ago • 1 comments

fixes #233

  • migrate unit tests for dataframe equality to chispa
    • removed methods for schema equality checks since this is performed as part of assert_df_equality
    • combined assertDataFramesEqual and assertTSDFsEqual into a single static method
    • renamed arguments dfA1 and dfBto reflectdf1anddf2` in chispas comparison output

R7L208 avatar Aug 10 '22 23:08 R7L208

Codecov Report

Merging #240 (3c678b6) into master (a51f1bf) will decrease coverage by 3.72%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
- Coverage   86.88%   83.15%   -3.73%     
==========================================
  Files          12        5       -7     
  Lines        1273      653     -620     
  Branches        0      143     +143     
==========================================
- Hits         1106      543     -563     
+ Misses        167       92      -75     
- Partials        0       18      +18     
Impacted Files Coverage Δ
tests/delta_writer_tests.py
tests/utils_tests.py
tests/base.py
tests/interpol_tests.py
tempo/__init__.py
tests/tsdf_tests.py
tests/as_of_join_tests.py
tempo/tsdf.py 77.25% <0.00%> (+1.43%) :arrow_up:
tempo/interpol.py 100.00% <0.00%> (+2.27%) :arrow_up:
tempo/utils.py 62.74% <0.00%> (+4.93%) :arrow_up:
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Aug 10 '22 23:08 codecov[bot]

This pull request introduces 1 alert when merging ea880e8225d2e8735f568e477a80da2e2e68ec2e into a51f1bfadda602926ff5ff4b477df8e15fe8a294 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

lgtm-com[bot] avatar Aug 15 '22 22:08 lgtm-com[bot]