tempo
tempo copied to clipboard
[DRAFT] Migrate unit tests to chispa
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
andassertTSDFsEqual
into a single static method - renamed arguments
dfA1 and
dfBto reflect
df1and
df2` in chispas comparison output
- removed methods for schema equality checks since this is performed as part of
Codecov Report
Merging #240 (3c678b6) into master (a51f1bf) will decrease coverage by
3.72%
. The diff coverage isn/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.
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'