Skip to content
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

Add kotest-assertions to common test dependencies #445

Merged
merged 4 commits into from
Dec 1, 2022
Merged

Conversation

petertrr
Copy link
Member

@petertrr petertrr commented Aug 30, 2022

  • Add kotest assertions to common test dependencies
  • Start using them in some tests
  • Remove duplicated conversion to sequence

Part of #408

* Add kotest assertions to common test dependencies
* Start using them in some tests
* Remove duplicated conversion to sequence
@codecov
Copy link

codecov bot commented Aug 30, 2022

Codecov Report

Merging #445 (ae29d4f) into main (101ce22) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##               main     #445   +/-   ##
=========================================
  Coverage     47.77%   47.77%           
  Complexity      147      147           
=========================================
  Files            52       52           
  Lines          1779     1779           
  Branches        308      308           
=========================================
  Hits            850      850           
  Misses          803      803           
  Partials        126      126           
Flag Coverage Δ
unittests 47.77% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...aveourtool/save/core/plugin/ExtraFlagsExtractor.kt 85.41% <ø> (ø)
...lin/com/saveourtool/save/plugin/warn/WarnPlugin.kt 60.86% <0.00%> (ø)

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

@github-actions
Copy link

github-actions bot commented Aug 30, 2022

JUnit Tests (Linux, EnricoMi/publish-unit-test-result-action@v1)

239 tests  ±0   215 ✔️ ±0   9m 6s ⏱️ +11s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit 973a2b1. ± Comparison against base commit 09feb36.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 30, 2022

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v1)

239 tests  ±0   215 ✔️ ±0   10m 36s ⏱️ - 4m 30s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit 973a2b1. ± Comparison against base commit 09feb36.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 30, 2022

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v1)

239 tests  ±0   215 ✔️ ±0   10m 3s ⏱️ -38s
  48 suites ±0     24 💤 ±0 
  48 files   ±0       0 ±0 

Results for commit 973a2b1. ± Comparison against base commit 09feb36.

♻️ This comment has been updated with latest results.

@petertrr petertrr marked this pull request as ready for review December 1, 2022 12:32
@petertrr petertrr enabled auto-merge (squash) December 1, 2022 12:33
@petertrr petertrr merged commit e557722 into main Dec 1, 2022
@petertrr petertrr deleted the infra/kotest branch December 1, 2022 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants