Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Moved compute tests to tests/ #388

Merged
merged 6 commits into from
Sep 7, 2021
Merged

Moved compute tests to tests/ #388

merged 6 commits into from
Sep 7, 2021

Conversation

jorgecarleitao
Copy link
Owner

No description provided.

@jorgecarleitao jorgecarleitao added the testing PRs that only increase coverage label Sep 7, 2021
@codecov
Copy link

codecov bot commented Sep 7, 2021

Codecov Report

Merging #388 (22f38df) into main (d7bd957) will not change coverage.
The diff coverage is 98.98%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #388   +/-   ##
=======================================
  Coverage   81.13%   81.13%           
=======================================
  Files         331      337    +6     
  Lines       21995    21995           
=======================================
  Hits        17846    17846           
  Misses       4149     4149           
Impacted Files Coverage Δ
src/compute/boolean.rs 92.85% <ø> (-6.29%) ⬇️
src/compute/concat.rs 100.00% <ø> (ø)
src/compute/filter.rs 49.23% <ø> (-25.96%) ⬇️
src/compute/hash.rs 94.00% <ø> (-2.56%) ⬇️
src/compute/if_then_else.rs 70.96% <ø> (-12.67%) ⬇️
src/compute/sort/mod.rs 35.00% <ø> (-22.53%) ⬇️
tests/it/compute/sort.rs 94.18% <94.18%> (ø)
tests/it/compute/boolean.rs 100.00% <100.00%> (ø)
tests/it/compute/concat.rs 100.00% <100.00%> (ø)
tests/it/compute/filter.rs 100.00% <100.00%> (ø)
... and 8 more

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 d7bd957...22f38df. Read the comment docs.

@jorgecarleitao jorgecarleitao changed the title Modes compute tests to tests/ Moved compute tests to tests/ Sep 7, 2021
@jorgecarleitao jorgecarleitao merged commit f28427d into main Sep 7, 2021
@jorgecarleitao jorgecarleitao deleted the tests_compute branch September 7, 2021 20:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing PRs that only increase coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant