Skip to content

Conversation

@matt-ramotar
Copy link
Collaborator

This PR is updating our tests to use Turbine.

@codecov
Copy link

codecov bot commented Nov 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.43%. Comparing base (369fd83) to head (5b08906).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #672      +/-   ##
==========================================
+ Coverage   74.89%   75.43%   +0.54%     
==========================================
  Files          38       38              
  Lines         920      920              
  Branches      168      168              
==========================================
+ Hits          689      694       +5     
+ Misses        144      142       -2     
+ Partials       87       84       -3     

see 2 files with indirect coverage changes

@tyvsmith
Copy link
Member

Going to conflict with your latest kover pr

@matt-ramotar matt-ramotar marked this pull request as draft November 25, 2024 14:36
@matt-ramotar matt-ramotar marked this pull request as ready for review November 27, 2024 17:45
@tyvsmith
Copy link
Member

Were the test changes done manually or scripted/automated? If the latter, please provide what was used.

@matt-ramotar
Copy link
Collaborator Author

@tyvsmith I updated them manually

@matt-ramotar matt-ramotar merged commit 120d13b into main Feb 22, 2025
5 checks passed
itsandreramon pushed a commit to itsandreramon/Store that referenced this pull request Feb 26, 2025
* Update Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update FetcherResponseTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update FetcherResponseTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update FlowStoreTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update MapIndexedTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update HotFlowStoreTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update UpdaterTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update SourceOfTruthErrorsTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update ValueFetcherTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update SourceOfTruthWithBarrierTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Update StreamWithoutSourceOfTruthTests to use Turbine

Signed-off-by: matt-ramotar <[email protected]>

* Delete AssertEmitsExactly.kt

Signed-off-by: matt-ramotar <[email protected]>

* Run gradlew ktlintFormat

Signed-off-by: matt-ramotar <[email protected]>

---------

Signed-off-by: matt-ramotar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants