Thulasi Dass Subramanian requested to merge az/td-jdk-17-migration into master Jul 12, 2023

Description:

Migration from Java 8 to Java 17.
Unit test fixes with build run args and dependency upgrades.
maven:3.9.3-eclipse-temurin-17 image is used as recommended by David osdu/platform/ci-cd-pipelines!957 (comment 234215)
For Integration Test - Refactored with Apache's CloseableHttpClient which supports all HTTP methods (include PATCH).

How to test:

Via Unit and Integration tests.

Changes include:

Breaking change (a change that is not backward-compatible and/or changes current functionality).

Changes in:

Common code

Dev Checklist:

Added Unit Tests, wherever applicable.
Updated the Readme, if applicable.
Existing Tests pass
Verified functionality locally
Self Reviewed my code for formatting and complex business logic.

Other comments:

Upgraded 'lombok' & 'jacoco' plugin version
Changed TestRunner 'PowerMockRunner' to 'MockitoJUnitRunner'
Removed 'powermock-api-mockito2' & 'powermock-module-junit4' dependencies
Integration test: Jersey HttpClient to support PATCH method, uses reflection logic which will not work in JDK 17. Hence replaced with Apache Apache's CloseableHttpClient.

Edited Jul 19, 2023 by Thulasi Dass Subramanian