Closed 1abhishekpandey closed 3 months ago
We have decided to use Mockito
wherever possible instead of mockk
(for the time being). So, I reverted all the changes in the CentralPluginChainTest
class and resolved the code issue that was causing the unit test to fail and also used concrete Message class; instead of mocking the sealed Message class.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Description
8.2.2
, we noticed that unit test cases were failing, as the Gradle has a compatibility issue with an older Java version11
(which the project was currently dependent upon). This PR bumps the Java version to17
throughout the project to address that issue.17
in GitHub action, as Gradle8.2.2
is now dependent upon Java version17
.8.2.2
, we observed that the existing test cases, which utilizedMockito
to mock thesealed
Message class, began to fail. So, we have resolved the code issue that was causing the unit test to fail in theCentralPluginChainTest
class and also used the concrete Message class (instead of mocking the sealed Message class).Known Issue: A few unit tests are still failing, but this issue is unrelated to the changes made in the Gradle version. The affected module is
models
. The failing test cases of this module should be addressed in this: https://github.com/rudderlabs/rudder-sdk-android/pull/400/commits/38547b6df0e923e9737f254474f32dbb303f8dfe.Fixes # (issue)
Type of change
Checklist: