Skip to content

Commit

Permalink
SQRP-279 Don't say SCA analysis
Browse files Browse the repository at this point in the history
  • Loading branch information
wenottingham authored and lukasz-jarocki-sonarsource committed Feb 28, 2025
1 parent 4f7051b commit d82c566
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@

/**
* Represents a single release of a package, such as an npm or maven package,
* as found in a single SCA analysis run (so it's attached to a branch component,
* as found in a single dependency analysis run (so it's attached to a branch component,
* and there's a separate copy of each release per branch it appears in).
*
* @param uuid Primary key
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,14 +50,14 @@ public ScaExecutor(CliCacheService cliCacheService, CliService cliService, Repor

public void execute(DefaultInputModule root) {
if (!featureFlagsRepository.isEnabled(SCA_FEATURE_NAME)) {
LOG.debug("SCA analysis skipped");
LOG.debug("Dependency analysis skipped");
return;
}

LOG.info("Checking for latest CLI");
File cliFile = cliCacheService.cacheCli();

LOG.info("Collecting manifests for the SCA analysis...");
LOG.info("Collecting manifests for the dependency analysis...");
if (cliFile.exists()) {
try {
File generatedZip = cliService.generateManifestsZip(root, cliFile);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ void execute_shouldSkipAnalysisWhenFeatureFlagDisabled() {

underTest.execute(root);

assertThat(logTester.logs()).contains("SCA analysis skipped");
assertThat(logTester.logs()).contains("Dependency analysis skipped");
verifyNoInteractions(cliService, cliCacheService);
}

Expand Down

0 comments on commit d82c566

Please sign in to comment.