Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

Add unit tests for code coverage #88

Merged
merged 9 commits into from
Oct 11, 2023
Merged

Conversation

jahid1209
Copy link
Collaborator

  • Added unit tests to improve code coverage from 54% to 72%.

globalConfigValues.put(ApplicationConstants.BLACKDUCK_TOKEN_KEY, "fake-blackduck-token");
globalConfigValues.put(ApplicationConstants.SYNOPSYS_BRIDGE_INSTALL_DIRECTORY, "/fake/path");

Map<String, Object> result = scanParametersFactory.preparePipelineParametersMap(securityScanStep,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here, preparePipelineParametersMap static method is accessed by the instance reference. I think static method should be accessed by the class name.

@jahid1209 jahid1209 merged commit 596983c into main Oct 11, 2023
1 check passed
@jahid1209 jahid1209 deleted the add-unit-tests-for-code-coverage branch October 11, 2023 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants