Skip to content

Commit

Permalink
Fix unit test failures
Browse files Browse the repository at this point in the history
  • Loading branch information
YasasRangika committed Sep 6, 2024
1 parent bb6e6f4 commit 7fc7f46
Showing 1 changed file with 9 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1436,13 +1436,15 @@ private String updateSwaggerSecurityDefinitionForStore(Swagger swagger, SwaggerD

KeyManagerConfigurationDTO keyManagerConfigurationDTO = null;

Check warning on line 1437 in components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java

View check run for this annotation

Codecov / codecov/patch

components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java#L1437

Added line #L1437 was not covered by tests
try {
String tenantDomain = PrivilegedCarbonContext.getThreadLocalCarbonContext().getTenantDomain();
APIAdmin apiAdmin = new APIAdminImpl();
keyManagerConfigurationDTO = apiAdmin.getKeyManagerConfigurationById(tenantDomain, kmId);
if (keyManagerConfigurationDTO == null || (StringUtils.isEmpty(kmId) && !Objects.equals(
keyManagerConfigurationDTO.getType(), APIConstants.KeyManager.DEFAULT_KEY_MANAGER_TYPE))) {
keyManagerConfigurationDTO = apiAdmin.getKeyManagerConfigurationByName(tenantDomain,
APIConstants.KeyManager.DEFAULT_KEY_MANAGER);
if (!StringUtils.isEmpty(kmId)) {
String tenantDomain = PrivilegedCarbonContext.getThreadLocalCarbonContext().getTenantDomain();
APIAdmin apiAdmin = new APIAdminImpl();
keyManagerConfigurationDTO = apiAdmin.getKeyManagerConfigurationById(tenantDomain, kmId);

Check warning on line 1442 in components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java

View check run for this annotation

Codecov / codecov/patch

components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java#L1440-L1442

Added lines #L1440 - L1442 were not covered by tests
if (keyManagerConfigurationDTO == null || (StringUtils.isEmpty(kmId) && !Objects.equals(
keyManagerConfigurationDTO.getType(), APIConstants.KeyManager.DEFAULT_KEY_MANAGER_TYPE))) {
keyManagerConfigurationDTO = apiAdmin.getKeyManagerConfigurationByName(tenantDomain,

Check warning on line 1445 in components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java

View check run for this annotation

Codecov / codecov/patch

components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java#L1444-L1445

Added lines #L1444 - L1445 were not covered by tests
APIConstants.KeyManager.DEFAULT_KEY_MANAGER);
}
}
} catch (APIManagementException e) {

Check warning on line 1449 in components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java

View check run for this annotation

Codecov / codecov/patch

components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/definitions/OAS2Parser.java#L1449

Added line #L1449 was not covered by tests
if (!StringUtils.isEmpty(kmId)) {
Expand Down

0 comments on commit 7fc7f46

Please sign in to comment.