bkkothari2255 opened a new pull request, #80:
URL: https://github.com/apache/sling-org-apache-sling-models-impl/pull/80
Issue: SLING-13088
- Migrated **8 test classes**:
1. `ModelAdapterFactoryTest`
2. `ModelAdapterFactory_ImplementationPickerOrderTest`
3. `ModelAdapterFactory_InjectorOrderTest`
4. `ModelPackageBundleListenerTest`
5. `MultipleInjectorTest`
6. `OSGiInjectionTest`
7. `OptionalObjectsTest,
8. `OptionalPrimitivesTest`
* Changes
* Replaced `MockitoJUnitRunner` with
`@ExtendWith(MockitoExtension.class)`
* Added JUnit 5 OSGi dependency and updated `OsgiContext` usage to
`@RegisterExtension`
* Converted `@Test(expected)` to `assertThrows()`
* Switched to `doReturn().when()` to resolve generic type inference
issues
* Corrected assertion argument order (expected vs. actual)
* Added `<?>` wildcards to raw `Type` variables to fix compiler warnings
* Renamed shadowed local `log` variables to `logger` to avoid conflicts
* Removed unnecessary `eq()` matchers and redundant `throws` declarations
* Cleaned up `@SuppressWarnings` and resolved related Sonar issues
* Applied `lenient()` strictness to preserve legacy behavior
* Added JUnit 5 OSGi dependency for implementation picker & Injector
order test
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
