Skip to content

feat(core): add jit principal generation - #105

Open
RVANDO12 wants to merge 23 commits into
mainfrom
feat/right-management
Open

feat(core): add jit principal generation#105
RVANDO12 wants to merge 23 commits into
mainfrom
feat/right-management

feat(core): copilot review 1 and soner qube coverage

624e10f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 30, 2026 in 37s

Quality Gate passed

Issues
16 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 77 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoV&open=AZ-ya7A3eoeKKH6oBtoV&pullRequest=105

Check warning on line 120 in src/test/java/com/decathlon/idp_core/TestSecurityConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to local variable "filterChain". 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7BUeoeKKH6oBtob&open=AZ-ya7BUeoeKKH6oBtob&pullRequest=105

Check warning on line 12 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.util.Optional'. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoZ&open=AZ-ya7A3eoeKKH6oBtoZ&pullRequest=105

Check warning on line 13 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.util.UUID'. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoa&open=AZ-ya7A3eoeKKH6oBtoa&pullRequest=105

Check warning on line 17 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/api/controller/PrincipalControllerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'com.decathlon.idp_core.domain.port.EntityTemplateRepositoryPort'. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7AoeoeKKH6oBtoT&open=AZ-ya7AoeoeKKH6oBtoT&pullRequest=105

Check warning on line 36 in src/test/java/com/decathlon/idp_core/domain/service/principal/PrincipalProvisioningServiceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "@InjectMocks" to inject these mock fields instead of manually constructing the object. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-yBboPx3kqwR67Y5J0&open=AZ-yBboPx3kqwR67Y5J0&pullRequest=105

Check warning on line 4 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.mockito.ArgumentMatchers.any'. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoX&open=AZ-ya7A3eoeKKH6oBtoX&pullRequest=105

Check failure on line 87 in src/main/java/com/decathlon/idp_core/infrastructure/adapters/api/principal/PrincipalExtractor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use already-defined constant 'CLAIM_EMAIL' instead of duplicating its value here. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-yBbkzx3kqwR67Y5Jv&open=AZ-yBbkzx3kqwR67Y5Jv&pullRequest=105

Check warning on line 125 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoW&open=AZ-ya7A3eoeKKH6oBtoW&pullRequest=105

Check warning on line 5 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.mockito.ArgumentMatchers.eq'. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoY&open=AZ-ya7A3eoeKKH6oBtoY&pullRequest=105

Check warning on line 12 in src/main/java/com/decathlon/idp_core/domain/exception/principal/PrincipalCreationException.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "principalIdentifier" private field. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7AZeoeKKH6oBtoS&open=AZ-ya7AZeoeKKH6oBtoS&pullRequest=105

Check warning on line 120 in src/test/java/com/decathlon/idp_core/TestSecurityConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "filterChain" local variable. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7BUeoeKKH6oBtod&open=AZ-ya7BUeoeKKH6oBtod&pullRequest=105

Check warning on line 43 in src/test/java/com/decathlon/idp_core/infrastructure/adapters/persistence/PostgresEntityAdapterTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "@InjectMocks" to inject these mock fields instead of manually constructing the object. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7A3eoeKKH6oBtoU&open=AZ-ya7A3eoeKKH6oBtoU&pullRequest=105

Check warning on line 113 in src/test/java/com/decathlon/idp_core/TestSecurityConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya7BUeoeKKH6oBtoc&open=AZ-ya7BUeoeKKH6oBtoc&pullRequest=105

Check failure on line 103 in src/main/java/com/decathlon/idp_core/infrastructure/adapters/api/principal/PrincipalExtractor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use already-defined constant 'CLAIM_CLIENT_ID' instead of duplicating its value here. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-yBbkzx3kqwR67Y5Ju&open=AZ-yBbkzx3kqwR67Y5Ju&pullRequest=105

Check warning on line 95 in src/main/java/com/decathlon/idp_core/domain/service/principal/PrincipalProvisioningService.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "e" with an unnamed pattern. 

See more on https://sonarcloud.io/project/issues?id=Decathlon_internal-developer-platform&issues=AZ-ya69xeoeKKH6oBtoR&open=AZ-ya69xeoeKKH6oBtoR&pullRequest=105
View more details on SonarQubeCloud