SpineEventEngine / core-java

The Java implementation of the framework core
https://spine.io
Apache License 2.0
36 stars 12 forks source link

Make some of `CatchUpTest` tests overridable again (again!) #1532

Closed armiol closed 1 year ago

armiol commented 1 year ago

This is another PR on the same matter as #1531, as the previous one got stuck due to some GH Actions failure to report the status of the builds.

Previously, some of CatchUpTest methods were grouped into @Nested tests. It made impossible to override/disable them in our "descendant" libraries, such as gcloud-java.

This PR restores the original behavior.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1532 (f9ff03f) into master (866df5f) will not change coverage. The diff coverage is n/a.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1532 +/- ## ========================================= Coverage 89.76% 89.76% Complexity 5000 5000 ========================================= Files 648 648 Lines 15689 15689 Branches 906 906 ========================================= Hits 14083 14083 Misses 1290 1290 Partials 316 316 ```