From c6a817468bcd71129f9da2b453a5a91af62df570 Mon Sep 17 00:00:00 2001 From: Zipeng WU Date: Tue, 10 Jan 2023 11:51:59 +0100 Subject: [PATCH] [NO-JIRA] Use record for data classes --- .../github/GithubApplicationClient.java | 63 -------------- .../auth/ldap/DefaultLdapGroupsProvider.java | 2 +- .../auth/ldap/DefaultLdapUsersProvider.java | 2 +- .../sonar/auth/ldap/LdapGroupsProvider.java | 23 +---- .../sonar/auth/ldap/LdapUsersProvider.java | 25 +----- .../org/sonar/ce/queue/CeQueueImplTest.java | 8 +- .../component/DequeBasedPath.java | 6 +- .../component/PathAwareCrawler.java | 2 +- .../component/PathAwareVisitor.java | 4 +- .../component/PathElementImpl.java | 22 +---- .../filemove/FileMoveDetectionStep.java | 57 ++++--------- .../task/projectanalysis/filemove/Match.java | 40 +-------- .../filemove/MovedFilesRepository.java | 19 +---- .../PullRequestFileMoveDetectionStep.java | 6 +- ...inesAndConditionsWithUncoveredCounter.java | 8 +- ...sAndConditionsWithUncoveredMetricKeys.java | 26 +----- .../coverage/SingleWithUncoveredCounter.java | 4 +- .../SingleWithUncoveredMetricKeys.java | 16 +--- .../issue/BaseInputFactory.java | 2 +- .../issue/MovedIssueVisitor.java | 5 +- .../RemoveProcessedComponentsVisitor.java | 2 +- .../TrackerTargetBranchInputFactory.java | 2 +- ...ortAnalysisFailureNotificationBuilder.java | 60 +------------ ...lysisFailureNotificationEmailTemplate.java | 24 +++--- ...ortAnalysisFailureNotificationHandler.java | 2 +- ...ysisFailureNotificationSerializerImpl.java | 16 ++-- .../qualitygate/EvaluationResult.java | 23 +---- .../EvaluationResultTextConverterImpl.java | 2 +- .../projectanalysis/scm/ScmInfoDbLoader.java | 2 +- .../source/FileSourceDataWarnings.java | 2 +- .../source/SourceLinesDiffImpl.java | 2 +- .../source/linereader/LineReader.java | 25 +----- .../step/PersistComponentsStep.java | 4 +- .../step/QualityGateMeasuresStep.java | 8 +- .../SmallChangesetQualityGateSpecialCase.java | 4 +- .../projectexport/issue/ExportIssuesStep.java | 2 +- .../projectexport/rule/ExportRuleStep.java | 6 +- .../ce/task/projectexport/rule/Rule.java | 20 +---- .../rule/RuleRepositoryImpl.java | 4 +- .../CallRecorderPathAwareVisitor.java | 2 +- .../filemove/FileMoveDetectionStepTest.java | 18 ++-- .../projectanalysis/filemove/MatchTest.java | 4 +- .../PullRequestFileMoveDetectionStepTest.java | 4 +- .../issue/MovedIssueVisitorTest.java | 2 +- ...sFailureNotificationEmailTemplateTest.java | 68 +++++++++++++++ ...nalysisFailureNotificationHandlerTest.java | 6 +- .../qualitygate/EvaluationResultAssert.java | 8 +- .../qualitygate/EvaluationResultTest.java | 4 +- .../source/FileSourceDataComputerTest.java | 2 +- .../step/NewCoverageMeasuresStepTest.java | 18 ++-- .../step/QualityGateMeasuresStepTest.java | 2 +- .../step/ReportCoverageMeasuresStepTest.java | 18 ++-- ...llChangesetQualityGateSpecialCaseTest.java | 4 +- .../step/ViewsCoverageMeasuresStepTest.java | 68 +++++++-------- .../issue/ExportIssuesStepTest.java | 2 +- .../rule/RuleRepositoryImplTest.java | 18 ++-- .../main/java/org/sonar/ce/task/CeTask.java | 15 +--- .../java/org/sonar/ce/task/CeTaskTest.java | 6 +- .../sonar/ce/taskprocessor/CeWorkerImpl.java | 4 +- ...lureNotificationExecutionListenerTest.java | 24 +++--- .../ce/queue/InternalCeQueueImplTest.java | 8 +- .../ce/taskprocessor/CeWorkerImplTest.java | 24 +++--- .../org/sonar/db/event/EventPurgeData.java | 13 +-- .../db/event/EventComponentChangeDaoTest.java | 2 +- .../version/v00/PopulateInitialSchema.java | 25 ++---- .../sonar/application/command/JvmOptions.java | 19 +---- .../ChangesOnMyIssueNotificationHandler.java | 8 +- .../FPOrWontFixNotificationHandler.java | 4 +- .../MyNewIssuesNotificationHandler.java | 4 +- .../notification/NewIssuesNotification.java | 35 +------- .../NewIssuesNotificationHandler.java | 2 +- .../DefaultNotificationManager.java | 29 ++----- .../notification/NotificationManager.java | 19 +---- .../email/EmailNotificationChannel.java | 26 +----- .../QGChangeNotificationHandler.java | 2 +- .../sonar/server/telemetry/TelemetryData.java | 84 +------------------ .../telemetry/TelemetryDataJsonWriter.java | 26 +++--- .../java/org/sonar/server/webhook/CeTask.java | 32 +------ .../org/sonar/server/webhook/WebHooks.java | 41 +-------- .../sonar/server/webhook/WebHooksImpl.java | 6 +- .../webhook/WebhookPayloadFactoryImpl.java | 4 +- ...angesOnMyIssueNotificationHandlerTest.java | 20 ++--- .../FPOrWontFixNotificationHandlerTest.java | 4 +- .../NewIssuesNotificationHandlerTest.java | 6 +- .../QGChangeNotificationHandlerTest.java | 6 +- .../org/sonar/server/webhook/CeTaskTest.java | 4 +- .../sonar/server/project/RekeyedProject.java | 33 +------- .../server/project/RekeyedProjectTest.java | 4 +- .../org/sonar/server/user/UserSession.java | 33 +------- .../TelemetryDataLoaderImplTest.java | 18 ++-- .../org/sonar/server/es/RecoveryIndexer.java | 17 +--- .../QProfileRestoreSummary.java | 13 +-- .../builtin/DescendantProfilesSupplier.java | 23 +---- .../builtin/RuleActivationContext.java | 4 +- .../qualityprofile/ws/RestoreAction.java | 4 +- .../QProfileBackuperImplTest.java | 12 +-- .../qualityprofile/ws/RestoreActionTest.java | 2 +- 97 files changed, 395 insertions(+), 1068 deletions(-) create mode 100644 server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplateTest.java diff --git a/server/sonar-alm-client/src/main/java/org/sonar/alm/client/github/GithubApplicationClient.java b/server/sonar-alm-client/src/main/java/org/sonar/alm/client/github/GithubApplicationClient.java index 4cd90cfefba..7446d304ec6 100644 --- a/server/sonar-alm-client/src/main/java/org/sonar/alm/client/github/GithubApplicationClient.java +++ b/server/sonar-alm-client/src/main/java/org/sonar/alm/client/github/GithubApplicationClient.java @@ -21,12 +21,10 @@ package org.sonar.alm.client.github; import com.google.gson.annotations.SerializedName; import java.util.List; -import java.util.Objects; import java.util.Optional; import javax.annotation.CheckForNull; import javax.annotation.Nullable; import javax.annotation.concurrent.Immutable; - import org.sonar.alm.client.github.config.GithubAppConfiguration; import org.sonar.alm.client.github.security.AccessToken; import org.sonar.alm.client.github.security.UserAccessToken; @@ -166,67 +164,6 @@ public interface GithubApplicationClient { } } - @Immutable - final class RepositoryDetails { - private final Repository repository; - private final String description; - private final String mainBranchName; - private final String url; - - public RepositoryDetails(Repository repository, String description, String mainBranchName, String url) { - this.repository = repository; - this.description = description; - this.mainBranchName = mainBranchName; - this.url = url; - } - - public Repository getRepository() { - return repository; - } - - public String getDescription() { - return description; - } - - public String getMainBranchName() { - return mainBranchName; - } - - public String getUrl() { - return url; - } - - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - RepositoryDetails that = (RepositoryDetails) o; - return Objects.equals(repository, that.repository) && - Objects.equals(description, that.description) && - Objects.equals(mainBranchName, that.mainBranchName) && - Objects.equals(url, that.url); - } - - @Override - public int hashCode() { - return Objects.hash(repository, description, mainBranchName, url); - } - - @Override - public String toString() { - return "RepositoryDetails{" + - "repository=" + repository + - ", description='" + description + '\'' + - ", mainBranchName='" + mainBranchName + '\'' + - ", url='" + url + '\'' + - '}'; - } - } - class Organizations { private int total; private List organizations; diff --git a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapGroupsProvider.java b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapGroupsProvider.java index 42bef16f57d..6f7515c1138 100644 --- a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapGroupsProvider.java +++ b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapGroupsProvider.java @@ -56,7 +56,7 @@ public class DefaultLdapGroupsProvider implements LdapGroupsProvider { */ @Override public Collection doGetGroups(Context context) { - return getGroups(context.getServerKey(), context.getUsername()); + return getGroups(context.serverKey(), context.username()); } private Collection getGroups(String serverKey, String username) { diff --git a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapUsersProvider.java b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapUsersProvider.java index ae18f897b73..d9af9e48357 100644 --- a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapUsersProvider.java +++ b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/DefaultLdapUsersProvider.java @@ -55,7 +55,7 @@ public class DefaultLdapUsersProvider implements LdapUsersProvider { @Override public LdapUserDetails doGetUserDetails(Context context) { - return getUserDetails(context.getServerKey(), context.getUsername()); + return getUserDetails(context.serverKey(), context.username()); } /** diff --git a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapGroupsProvider.java b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapGroupsProvider.java index 337e39fa3b9..629ead08265 100644 --- a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapGroupsProvider.java +++ b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapGroupsProvider.java @@ -26,27 +26,6 @@ public interface LdapGroupsProvider { Collection doGetGroups(Context context); - final class Context { - private final String serverKey; - private final String username; - private final HttpServletRequest request; - - public Context(String serverKey, String username, HttpServletRequest request) { - this.serverKey = serverKey; - this.username = username; - this.request = request; - } - - public String getServerKey() { - return serverKey; - } - - public String getUsername() { - return username; - } - - public HttpServletRequest getRequest() { - return request; - } + record Context(String serverKey, String username, HttpServletRequest request) { } } diff --git a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapUsersProvider.java b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapUsersProvider.java index d7a2e4ea50b..7c3ba105863 100644 --- a/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapUsersProvider.java +++ b/server/sonar-auth-ldap/src/main/java/org/sonar/auth/ldap/LdapUsersProvider.java @@ -25,30 +25,7 @@ public interface LdapUsersProvider { LdapUserDetails doGetUserDetails(Context context); - final class Context { - private final String username; - - private final String serverKey; - - private final HttpServletRequest request; - - public Context(String serverKey, String username, HttpServletRequest request) { - this.username = username; - this.serverKey = serverKey; - this.request = request; - } - - public String getUsername() { - return username; - } - - public String getServerKey() { - return serverKey; - } - - public HttpServletRequest getRequest() { - return request; - } + record Context(String serverKey, String username, HttpServletRequest request) { } } diff --git a/server/sonar-ce-common/src/test/java/org/sonar/ce/queue/CeQueueImplTest.java b/server/sonar-ce-common/src/test/java/org/sonar/ce/queue/CeQueueImplTest.java index 4541acf10b7..d17937e7234 100644 --- a/server/sonar-ce-common/src/test/java/org/sonar/ce/queue/CeQueueImplTest.java +++ b/server/sonar-ce-common/src/test/java/org/sonar/ce/queue/CeQueueImplTest.java @@ -577,11 +577,11 @@ public class CeQueueImplTest { assertThat(task.getType()).isEqualTo(taskSubmit.getType()); if (taskSubmit.getSubmitterUuid() != null) { if (userDto == null) { - assertThat(task.getSubmitter().getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); - assertThat(task.getSubmitter().getLogin()).isNull(); + assertThat(task.getSubmitter().uuid()).isEqualTo(taskSubmit.getSubmitterUuid()); + assertThat(task.getSubmitter().login()).isNull(); } else { - assertThat(task.getSubmitter().getUuid()).isEqualTo(userDto.getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); - assertThat(task.getSubmitter().getLogin()).isEqualTo(userDto.getLogin()); + assertThat(task.getSubmitter().uuid()).isEqualTo(userDto.getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); + assertThat(task.getSubmitter().login()).isEqualTo(userDto.getLogin()); } } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/DequeBasedPath.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/DequeBasedPath.java index fc6f4a49df1..b012ebf795c 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/DequeBasedPath.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/DequeBasedPath.java @@ -29,7 +29,7 @@ final class DequeBasedPath implements PathAwareVisitor.Path, Iterable implements PathAwareVisitor.Path, Iterable implements PathAwareVisitor.Path, Iterable implements ComponentCrawler { @Override @Nonnull public String apply(@Nonnull PathAwareVisitor.PathElement input) { - return format("%s(type=%s)", input.getComponent().getKey(), input.getComponent().getType()); + return format("%s(type=%s)", input.component().getKey(), input.component().getType()); } } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathAwareVisitor.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathAwareVisitor.java index af54d6e85be..c85471be855 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathAwareVisitor.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathAwareVisitor.java @@ -116,11 +116,11 @@ public interface PathAwareVisitor extends ComponentVisitor { /** * The Component on the path. */ - Component getComponent(); + Component component(); /** * The stacked element for the Component of this PathElement. */ - T getElement(); + T element(); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathElementImpl.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathElementImpl.java index 95ca65ba9de..a621f45331c 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathElementImpl.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/component/PathElementImpl.java @@ -19,25 +19,5 @@ */ package org.sonar.ce.task.projectanalysis.component; -import javax.annotation.concurrent.Immutable; - -@Immutable -final class PathElementImpl implements PathAwareVisitor.PathElement { - private final Component component; - private final T element; - - public PathElementImpl(Component component, T element) { - this.component = component; - this.element = element; - } - - @Override - public Component getComponent() { - return component; - } - - @Override - public T getElement() { - return element; - } +record PathElementImpl(Component component, T element) implements PathAwareVisitor.PathElement { } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStep.java index 14fe9cebfa1..1013fe56de4 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStep.java @@ -36,7 +36,6 @@ import java.util.Map; import java.util.Set; import java.util.stream.Collectors; import javax.annotation.Nullable; -import javax.annotation.concurrent.Immutable; import org.apache.ibatis.session.ResultContext; import org.apache.ibatis.session.ResultHandler; import org.sonar.api.utils.log.Logger; @@ -178,8 +177,8 @@ public class FileMoveDetectionStep implements ComputationStep { LOG.debug("{} files moves found", electedMatches.size()); for (Match validatedMatch : electedMatches) { movedFilesRepository.setOriginalFile( - reportFilesByUuid.get(validatedMatch.getReportUuid()), - toOriginalFile(dbFilesByUuid.get(validatedMatch.getDbUuid()))); + reportFilesByUuid.get(validatedMatch.reportUuid()), + toOriginalFile(dbFilesByUuid.get(validatedMatch.dbUuid()))); LOG.trace("File move found: {}", validatedMatch); } } @@ -192,7 +191,7 @@ public class FileMoveDetectionStep implements ComputationStep { } else { Set reallyAddedFileUuids = new HashSet<>(addedFileUuids); for (Match electedMatch : electedMatches) { - reallyAddedFileUuids.remove(electedMatch.getReportUuid()); + reallyAddedFileUuids.remove(electedMatch.reportUuid()); } reallyAddedFileUuids.stream() .map(reportFilesByUuid::get) @@ -209,7 +208,7 @@ public class FileMoveDetectionStep implements ComputationStep { builder.add(new DbComponent(row.getKey(), row.getUuid(), row.getPath(), row.getLineCount())); }); return builder.build().stream() - .collect(MoreCollectors.uniqueIndex(DbComponent::getUuid)); + .collect(MoreCollectors.uniqueIndex(DbComponent::uuid)); } } @@ -245,7 +244,7 @@ public class FileMoveDetectionStep implements ComputationStep { ScoreMatrix.ScoreFile[] removedFiles = removedFileUuids.stream() .map(key -> { DbComponent dbComponent = dtosByUuid.get(key); - return new ScoreMatrix.ScoreFile(dbComponent.getUuid(), dbComponent.getLineCount()); + return new ScoreMatrix.ScoreFile(dbComponent.uuid(), dbComponent.lineCount()); }) .toArray(ScoreMatrix.ScoreFile[]::new); @@ -353,13 +352,13 @@ public class FileMoveDetectionStep implements ComputationStep { } else { matchesPerFileForScore.clear(); for (Match match : matchesToValidate) { - matchesPerFileForScore.put(match.getDbUuid(), match); - matchesPerFileForScore.put(match.getReportUuid(), match); + matchesPerFileForScore.put(match.dbUuid(), match); + matchesPerFileForScore.put(match.reportUuid(), match); } // validate non-ambiguous matches (i.e. the match is the only match of either the db file and the report file) for (Match match : matchesToValidate) { - int dbFileMatchesCount = matchesPerFileForScore.get(match.getDbUuid()).size(); - int reportFileMatchesCount = matchesPerFileForScore.get(match.getReportUuid()).size(); + int dbFileMatchesCount = matchesPerFileForScore.get(match.dbUuid()).size(); + int reportFileMatchesCount = matchesPerFileForScore.get(match.reportUuid()).size(); if (dbFileMatchesCount == 1 && reportFileMatchesCount == 1) { electedMatches.add(match); } @@ -368,38 +367,10 @@ public class FileMoveDetectionStep implements ComputationStep { } private static MovedFilesRepository.OriginalFile toOriginalFile(DbComponent dbComponent) { - return new MovedFilesRepository.OriginalFile(dbComponent.getUuid(), dbComponent.getKey()); + return new MovedFilesRepository.OriginalFile(dbComponent.uuid(), dbComponent.key()); } - @Immutable - public static final class DbComponent { - private final String key; - private final String uuid; - private final String path; - private final int lineCount; - - public DbComponent(String key, String uuid, String path, int lineCount) { - this.key = key; - this.uuid = uuid; - this.path = path; - this.lineCount = lineCount; - } - - public String getKey() { - return key; - } - - public String getUuid() { - return uuid; - } - - public String getPath() { - return path; - } - - public int getLineCount() { - return lineCount; - } + public record DbComponent(String key, String uuid, String path, int lineCount) { } private static class ElectedMatches implements Iterable { @@ -413,8 +384,8 @@ public class FileMoveDetectionStep implements ComputationStep { public void add(Match match) { matches.add(match); - matchedFileUuids.add(match.getDbUuid()); - matchedFileUuids.add(match.getReportUuid()); + matchedFileUuids.add(match.dbUuid()); + matchedFileUuids.add(match.reportUuid()); } public List filter(Collection matches) { @@ -422,7 +393,7 @@ public class FileMoveDetectionStep implements ComputationStep { } private boolean notAlreadyMatched(Match input) { - return !(matchedFileUuids.contains(input.getDbUuid()) || matchedFileUuids.contains(input.getReportUuid())); + return !(matchedFileUuids.contains(input.dbUuid()) || matchedFileUuids.contains(input.reportUuid())); } @Override diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/Match.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/Match.java index e6d57c2e96f..19bf92ec1a7 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/Match.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/Match.java @@ -19,45 +19,7 @@ */ package org.sonar.ce.task.projectanalysis.filemove; -import java.util.Objects; -import javax.annotation.Nullable; -import javax.annotation.concurrent.Immutable; - -@Immutable -final class Match { - private final String dbUuid; - private final String reportUuid; - - Match(String dbUuid, String reportUuid) { - this.dbUuid = dbUuid; - this.reportUuid = reportUuid; - } - - public String getDbUuid() { - return dbUuid; - } - - public String getReportUuid() { - return reportUuid; - } - - @Override - public boolean equals(@Nullable Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - Match match = (Match) o; - return dbUuid.equals(match.dbUuid) && reportUuid.equals(match.reportUuid); - } - - @Override - public int hashCode() { - return Objects.hash(dbUuid, reportUuid); - } - +record Match(String dbUuid, String reportUuid) { @Override public String toString() { return '{' + dbUuid + "=>" + reportUuid + '}'; diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/MovedFilesRepository.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/MovedFilesRepository.java index ab2c188c009..2117ff104d9 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/MovedFilesRepository.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/MovedFilesRepository.java @@ -42,23 +42,12 @@ public interface MovedFilesRepository { */ Optional getOriginalPullRequestFile(Component file); - final class OriginalFile { - private final String uuid; - private final String key; - + record OriginalFile(String uuid, String key) { public OriginalFile(String uuid, String key) { this.uuid = requireNonNull(uuid, "uuid can not be null"); this.key = requireNonNull(key, "key can not be null"); } - public String getUuid() { - return uuid; - } - - public String getKey() { - return key; - } - @Override public boolean equals(@Nullable Object o) { if (this == o) { @@ -79,9 +68,9 @@ public interface MovedFilesRepository { @Override public String toString() { return "OriginalFile{" + - "uuid='" + uuid + '\'' + - ", key='" + key + '\'' + - '}'; + "uuid='" + uuid + '\'' + + ", key='" + key + '\'' + + '}'; } } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStep.java index f1c19139fca..fa4fa79b888 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStep.java @@ -161,7 +161,7 @@ public class PullRequestFileMoveDetectionStep implements ComputationStep { private static ResultHandler accumulateFilesForFileMove(Map accumulator) { return resultContext -> { DbComponent component = rowToDbComponent(resultContext.getResultObject()); - accumulator.put(component.getUuid(), component); + accumulator.put(component.uuid(), component); }; } @@ -177,7 +177,7 @@ public class PullRequestFileMoveDetectionStep implements ComputationStep { private static Map toDbFilesByPathReferenceMap(Collection dbFiles) { return dbFiles .stream() - .collect(toMap(DbComponent::getPath, identity())); + .collect(toMap(DbComponent::path, identity())); } private static Map getReportFilesByUuid(Component root) { @@ -195,6 +195,6 @@ public class PullRequestFileMoveDetectionStep implements ComputationStep { } private static OriginalFile toOriginalFile(DbComponent dbComponent) { - return new OriginalFile(dbComponent.getUuid(), dbComponent.getKey()); + return new OriginalFile(dbComponent.uuid(), dbComponent.key()); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredCounter.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredCounter.java index ce41c15d659..6075e89be87 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredCounter.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredCounter.java @@ -32,10 +32,10 @@ public final class LinesAndConditionsWithUncoveredCounter extends ElementsAndCov @Override protected void initializeForSupportedLeaf(CounterInitializationContext counterContext) { - this.elements = getLongMeasureValue(counterContext, metricKeys.getLines()) - + getLongMeasureValue(counterContext, metricKeys.getConditions()); + this.elements = getLongMeasureValue(counterContext, metricKeys.lines()) + + getLongMeasureValue(counterContext, metricKeys.conditions()); this.coveredElements = this.elements - - getLongMeasureValue(counterContext, metricKeys.getUncoveredLines()) - - getLongMeasureValue(counterContext, metricKeys.getUncoveredConditions()); + - getLongMeasureValue(counterContext, metricKeys.uncoveredLines()) + - getLongMeasureValue(counterContext, metricKeys.uncoveredConditions()); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredMetricKeys.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredMetricKeys.java index acc4c044a08..6d3e88ce62a 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredMetricKeys.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/LinesAndConditionsWithUncoveredMetricKeys.java @@ -19,37 +19,13 @@ */ package org.sonar.ce.task.projectanalysis.formula.coverage; -import javax.annotation.concurrent.Immutable; - import static java.util.Objects.requireNonNull; -@Immutable -public final class LinesAndConditionsWithUncoveredMetricKeys { - private final String lines; - private final String conditions; - private final String uncoveredLines; - private final String uncoveredConditions; - +public record LinesAndConditionsWithUncoveredMetricKeys(String lines, String conditions, String uncoveredLines, String uncoveredConditions) { public LinesAndConditionsWithUncoveredMetricKeys(String lines, String conditions, String uncoveredLines, String uncoveredConditions) { this.lines = requireNonNull(lines); this.conditions = requireNonNull(conditions); this.uncoveredLines = requireNonNull(uncoveredLines); this.uncoveredConditions = requireNonNull(uncoveredConditions); } - - public String getLines() { - return lines; - } - - public String getConditions() { - return conditions; - } - - public String getUncoveredLines() { - return uncoveredLines; - } - - public String getUncoveredConditions() { - return uncoveredConditions; - } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredCounter.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredCounter.java index e6004c1d8d7..f3cd8ad6892 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredCounter.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredCounter.java @@ -32,7 +32,7 @@ public final class SingleWithUncoveredCounter extends ElementsAndCoveredElements @Override protected void initializeForSupportedLeaf(CounterInitializationContext counterContext) { - this.elements = getLongMeasureValue(counterContext, metricKeys.getCovered()); - this.coveredElements = this.elements - getLongMeasureValue(counterContext, metricKeys.getUncovered()); + this.elements = getLongMeasureValue(counterContext, metricKeys.covered()); + this.coveredElements = this.elements - getLongMeasureValue(counterContext, metricKeys.uncovered()); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredMetricKeys.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredMetricKeys.java index 6ff205d1975..1294812a75a 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredMetricKeys.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/formula/coverage/SingleWithUncoveredMetricKeys.java @@ -19,25 +19,11 @@ */ package org.sonar.ce.task.projectanalysis.formula.coverage; -import javax.annotation.concurrent.Immutable; - import static java.util.Objects.requireNonNull; -@Immutable -public final class SingleWithUncoveredMetricKeys { - private final String covered; - private final String uncovered; - +public record SingleWithUncoveredMetricKeys(String covered, String uncovered) { public SingleWithUncoveredMetricKeys(String covered, String uncovered) { this.covered = requireNonNull(covered); this.uncovered = requireNonNull(uncovered); } - - public String getCovered() { - return covered; - } - - public String getUncovered() { - return uncovered; - } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/BaseInputFactory.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/BaseInputFactory.java index b3f8e60c995..c608cef3f7c 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/BaseInputFactory.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/BaseInputFactory.java @@ -44,7 +44,7 @@ public abstract class BaseInputFactory { BaseLazyInput(DbClient dbClient, Component component, @Nullable MovedFilesRepository.OriginalFile originalFile) { this.dbClient = dbClient; this.component = component; - this.effectiveUuid = originalFile == null ? component.getUuid() : originalFile.getUuid(); + this.effectiveUuid = originalFile == null ? component.getUuid() : originalFile.uuid(); } @Override diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitor.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitor.java index 5bc67db8a6e..f72d3d1b500 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitor.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitor.java @@ -51,9 +51,10 @@ public class MovedIssueVisitor extends IssueVisitor { "Issue %s for component %s has a different component key but no original file exist in MovedFilesRepository", issue, component); OriginalFile originalFile = originalFileOptional.get(); - checkState(originalFile.getUuid().equals(issue.componentUuid()), + String fileUuid = originalFile.uuid(); + checkState(fileUuid.equals(issue.componentUuid()), "Issue %s doesn't belong to file %s registered as original file of current file %s", - issue, originalFile.getUuid(), component); + issue, fileUuid, component); // changes the issue's component uuid, and set issue as changed, to enforce it is persisted to DB issueUpdater.setIssueComponent(issue, component.getUuid(), component.getKey(), new Date(analysisMetadataHolder.getAnalysisDate())); diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/RemoveProcessedComponentsVisitor.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/RemoveProcessedComponentsVisitor.java index 1a2656e46c7..a0a165a0022 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/RemoveProcessedComponentsVisitor.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/RemoveProcessedComponentsVisitor.java @@ -37,7 +37,7 @@ public class RemoveProcessedComponentsVisitor extends IssueVisitor { componentsWithUnprocessedIssues.remove(component.getUuid()); Optional originalFile = movedFilesRepository.getOriginalFile(component); if (originalFile.isPresent()) { - componentsWithUnprocessedIssues.remove(originalFile.get().getUuid()); + componentsWithUnprocessedIssues.remove(originalFile.get().uuid()); } } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/TrackerTargetBranchInputFactory.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/TrackerTargetBranchInputFactory.java index 313127cfb41..0caec78277c 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/TrackerTargetBranchInputFactory.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/issue/TrackerTargetBranchInputFactory.java @@ -71,7 +71,7 @@ public class TrackerTargetBranchInputFactory { private Optional getOriginalComponentKey(Component component) { return movedFilesRepository .getOriginalPullRequestFile(component) - .map(OriginalFile::getKey); + .map(OriginalFile::key); } private class TargetLazyInput extends LazyInput { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationBuilder.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationBuilder.java index 7bfb0dafc41..6ab18a43ce0 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationBuilder.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationBuilder.java @@ -19,41 +19,19 @@ */ package org.sonar.ce.task.projectanalysis.notification; -import javax.annotation.CheckForNull; import javax.annotation.Nullable; import static java.util.Objects.requireNonNull; -public class ReportAnalysisFailureNotificationBuilder { - private final Project project; - private final Task task; - private final String errorMessage; - +public record ReportAnalysisFailureNotificationBuilder(Project project, Task task, String errorMessage) { public ReportAnalysisFailureNotificationBuilder(Project project, Task task, @Nullable String errorMessage) { this.project = requireNonNull(project, "project can't be null"); this.task = requireNonNull(task, "task can't be null"); this.errorMessage = errorMessage; } - public Project getProject() { - return project; - } - - public Task getTask() { - return task; - } - - @CheckForNull - public String getErrorMessage() { - return errorMessage; - } - - public static final class Project { - private final String uuid; - private final String key; - private final String name; - private final String branchName; + public record Project(String uuid, String key, String name, String branchName) { public Project(String uuid, String key, String name, @Nullable String branchName) { this.uuid = requireNonNull(uuid, "uuid can't be null"); this.key = requireNonNull(key, "key can't be null"); @@ -61,45 +39,13 @@ public class ReportAnalysisFailureNotificationBuilder { this.branchName = branchName; } - public String getUuid() { - return uuid; - } - - public String getKey() { - return key; - } - - public String getName() { - return name; - } - - @CheckForNull - public String getBranchName() { - return branchName; - } } - public static final class Task { - private final String uuid; - private final long createdAt; - private final long failedAt; - + public record Task(String uuid, long createdAt, long failedAt) { public Task(String uuid, long createdAt, long failedAt) { this.uuid = requireNonNull(uuid, "uuid can't be null"); this.createdAt = createdAt; this.failedAt = failedAt; } - - public String getUuid() { - return uuid; - } - - public long getCreatedAt() { - return createdAt; - } - - public long getFailedAt() { - return failedAt; - } } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplate.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplate.java index 280cdb29102..9f157e355e7 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplate.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplate.java @@ -50,8 +50,8 @@ public class ReportAnalysisFailureNotificationEmailTemplate implements EmailTemp } ReportAnalysisFailureNotificationBuilder taskFailureNotification = serializer.fromNotification((ReportAnalysisFailureNotification) notification); - String projectUuid = taskFailureNotification.getProject().getUuid(); - String projectFullName = computeProjectFullName(taskFailureNotification.getProject()); + String projectUuid = taskFailureNotification.project().uuid(); + String projectFullName = computeProjectFullName(taskFailureNotification.project()); return new EmailMessage() .setMessageId(notification.getType() + "/" + projectUuid) @@ -60,11 +60,11 @@ public class ReportAnalysisFailureNotificationEmailTemplate implements EmailTemp } private static String computeProjectFullName(ReportAnalysisFailureNotificationBuilder.Project project) { - String branchName = project.getBranchName(); + String branchName = project.branchName(); if (branchName != null) { - return String.format("%s (%s)", project.getName(), branchName); + return String.format("%s (%s)", project.name(), branchName); } - return project.getName(); + return project.name(); } private static String subject(String projectFullName) { @@ -72,23 +72,23 @@ public class ReportAnalysisFailureNotificationEmailTemplate implements EmailTemp } private String message(String projectFullName, ReportAnalysisFailureNotificationBuilder taskFailureNotification) { - ReportAnalysisFailureNotificationBuilder.Project project = taskFailureNotification.getProject(); - ReportAnalysisFailureNotificationBuilder.Task task = taskFailureNotification.getTask(); + ReportAnalysisFailureNotificationBuilder.Project project = taskFailureNotification.project(); + ReportAnalysisFailureNotificationBuilder.Task task = taskFailureNotification.task(); StringBuilder res = new StringBuilder(); res.append("Project:").append(TAB).append(projectFullName).append(LINE_RETURN); - res.append("Background task:").append(TAB).append(task.getUuid()).append(LINE_RETURN); - res.append("Submission time:").append(TAB).append(formatDateTime(task.getCreatedAt())).append(LINE_RETURN); - res.append("Failure time:").append(TAB).append(formatDateTime(task.getFailedAt())).append(LINE_RETURN); + res.append("Background task:").append(TAB).append(task.uuid()).append(LINE_RETURN); + res.append("Submission time:").append(TAB).append(formatDateTime(task.createdAt())).append(LINE_RETURN); + res.append("Failure time:").append(TAB).append(formatDateTime(task.failedAt())).append(LINE_RETURN); - String errorMessage = taskFailureNotification.getErrorMessage(); + String errorMessage = taskFailureNotification.errorMessage(); if (errorMessage != null) { res.append(LINE_RETURN); res.append("Error message:").append(TAB).append(errorMessage).append(LINE_RETURN); } res.append(LINE_RETURN); - res.append("More details at: ").append(String.format("%s/project/background_tasks?id=%s", settings.getServerBaseURL(), encode(project.getKey()))); + res.append("More details at: ").append(String.format("%s/project/background_tasks?id=%s", settings.getServerBaseURL(), encode(project.key()))); return res.toString(); } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandler.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandler.java index c964ca50565..0387dd532f1 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandler.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandler.java @@ -83,6 +83,6 @@ public class ReportAnalysisFailureNotificationHandler extends EmailNotificationH return notificationManager.findSubscribedEmailRecipients(KEY, projectKey, REQUIRED_SUBSCRIBER_PERMISSIONS) .stream() .flatMap(emailRecipient -> notifications.stream() - .map(notification -> new EmailDeliveryRequest(emailRecipient.getEmail(), notification))); + .map(notification -> new EmailDeliveryRequest(emailRecipient.email(), notification))); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationSerializerImpl.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationSerializerImpl.java index 78db255ec8f..53a058ae928 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationSerializerImpl.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationSerializerImpl.java @@ -35,14 +35,14 @@ public class ReportAnalysisFailureNotificationSerializerImpl implements ReportAn public ReportAnalysisFailureNotification toNotification(ReportAnalysisFailureNotificationBuilder reportAnalysisFailureNotificationBuilder) { ReportAnalysisFailureNotification notification = new ReportAnalysisFailureNotification(); notification - .setFieldValue(FIELD_PROJECT_UUID, reportAnalysisFailureNotificationBuilder.getProject().getUuid()) - .setFieldValue(FIELD_PROJECT_KEY, reportAnalysisFailureNotificationBuilder.getProject().getKey()) - .setFieldValue(FIELD_PROJECT_NAME, reportAnalysisFailureNotificationBuilder.getProject().getName()) - .setFieldValue(FIELD_PROJECT_BRANCH, reportAnalysisFailureNotificationBuilder.getProject().getBranchName()) - .setFieldValue(FIELD_TASK_UUID, reportAnalysisFailureNotificationBuilder.getTask().getUuid()) - .setFieldValue(FIELD_TASK_CREATED_AT, valueOf(reportAnalysisFailureNotificationBuilder.getTask().getCreatedAt())) - .setFieldValue(FIELD_TASK_FAILED_AT, valueOf(reportAnalysisFailureNotificationBuilder.getTask().getFailedAt())) - .setFieldValue(FIELD_ERROR_MESSAGE, reportAnalysisFailureNotificationBuilder.getErrorMessage()); + .setFieldValue(FIELD_PROJECT_UUID, reportAnalysisFailureNotificationBuilder.project().uuid()) + .setFieldValue(FIELD_PROJECT_KEY, reportAnalysisFailureNotificationBuilder.project().key()) + .setFieldValue(FIELD_PROJECT_NAME, reportAnalysisFailureNotificationBuilder.project().name()) + .setFieldValue(FIELD_PROJECT_BRANCH, reportAnalysisFailureNotificationBuilder.project().branchName()) + .setFieldValue(FIELD_TASK_UUID, reportAnalysisFailureNotificationBuilder.task().uuid()) + .setFieldValue(FIELD_TASK_CREATED_AT, valueOf(reportAnalysisFailureNotificationBuilder.task().createdAt())) + .setFieldValue(FIELD_TASK_FAILED_AT, valueOf(reportAnalysisFailureNotificationBuilder.task().failedAt())) + .setFieldValue(FIELD_ERROR_MESSAGE, reportAnalysisFailureNotificationBuilder.errorMessage()); return notification; } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResult.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResult.java index 021b6d963ed..7f4e27ed103 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResult.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResult.java @@ -22,36 +22,21 @@ package org.sonar.ce.task.projectanalysis.qualitygate; import com.google.common.base.MoreObjects; import javax.annotation.CheckForNull; import javax.annotation.Nullable; -import javax.annotation.concurrent.Immutable; import org.sonar.ce.task.projectanalysis.measure.Measure; import static java.util.Objects.requireNonNull; -@Immutable -public final class EvaluationResult { - private final Measure.Level level; - @CheckForNull - private final Comparable value; - +public record EvaluationResult(Measure.Level level, @CheckForNull Comparable value) { public EvaluationResult(Measure.Level level, @Nullable Comparable value) { this.level = requireNonNull(level); this.value = value; } - public Measure.Level getLevel() { - return level; - } - - @CheckForNull - public Comparable getValue() { - return value; - } - @Override public String toString() { return MoreObjects.toStringHelper(this) - .add("level", level) - .add("value", value) - .toString(); + .add("level", level) + .add("value", value) + .toString(); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTextConverterImpl.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTextConverterImpl.java index 59f2393d6d7..27cf617fe51 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTextConverterImpl.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTextConverterImpl.java @@ -47,7 +47,7 @@ public final class EvaluationResultTextConverterImpl implements EvaluationResult @CheckForNull public String asText(Condition condition, EvaluationResult evaluationResult) { requireNonNull(condition); - if (evaluationResult.getLevel() == Measure.Level.OK) { + if (evaluationResult.level() == Measure.Level.OK) { return null; } return getAlertLabel(condition); diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/scm/ScmInfoDbLoader.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/scm/ScmInfoDbLoader.java index 8eccf00a8e8..c4b9eb9d677 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/scm/ScmInfoDbLoader.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/scm/ScmInfoDbLoader.java @@ -77,7 +77,7 @@ public class ScmInfoDbLoader { if (!analysisMetadataHolder.isFirstAnalysis() && !analysisMetadataHolder.isPullRequest() && !isReferenceBranch()) { Optional originalFile = movedFilesRepository.getOriginalFile(file); if (originalFile.isPresent()) { - return originalFile.map(MovedFilesRepository.OriginalFile::getUuid); + return originalFile.map(MovedFilesRepository.OriginalFile::uuid); } return Optional.of(file.getUuid()); } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataWarnings.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataWarnings.java index c64452cc1b3..68b2a4f67cb 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataWarnings.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataWarnings.java @@ -53,7 +53,7 @@ public class FileSourceDataWarnings { requireNonNull(file, "file can't be null"); requireNonNull(readError, "readError can't be null"); - fileErrorsPerData.compute(readError.getData(), (data, existingList) -> { + fileErrorsPerData.compute(readError.data(), (data, existingList) -> { Set res = existingList == null ? new HashSet<>() : existingList; res.add(file); return res; diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/SourceLinesDiffImpl.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/SourceLinesDiffImpl.java index 78446ac4069..d645d174f06 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/SourceLinesDiffImpl.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/SourceLinesDiffImpl.java @@ -74,7 +74,7 @@ public class SourceLinesDiffImpl implements SourceLinesDiff { uuid = newCodeReferenceBranchComponentUuids.getComponentUuid(component.getKey()); } else { Optional originalFile = movedFilesRepository.getOriginalFile(component); - uuid = originalFile.map(MovedFilesRepository.OriginalFile::getUuid).orElse(component.getUuid()); + uuid = originalFile.map(MovedFilesRepository.OriginalFile::uuid).orElse(component.getUuid()); } if (uuid == null) { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/linereader/LineReader.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/linereader/LineReader.java index 9c11fbf2061..34b326d9d37 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/linereader/LineReader.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/source/linereader/LineReader.java @@ -19,9 +19,7 @@ */ package org.sonar.ce.task.projectanalysis.source.linereader; -import java.util.Objects; import java.util.Optional; -import javax.annotation.concurrent.Immutable; import org.sonar.db.protobuf.DbFileSources; import static com.google.common.base.Preconditions.checkArgument; @@ -36,24 +34,10 @@ public interface LineReader { COVERAGE, DUPLICATION, HIGHLIGHTING, SCM, SYMBOLS } - @Immutable - final class ReadError { - private final Data data; - private final int line; - - public ReadError(Data data, int line) { + record ReadError(Data data, int line) { + public ReadError { requireNonNull(data); checkArgument(line >= 0); - this.data = data; - this.line = line; - } - - public Data getData() { - return data; - } - - public int getLine() { - return line; } @Override @@ -69,11 +53,6 @@ public interface LineReader { data == readError.data; } - @Override - public int hashCode() { - return Objects.hash(data, line); - } - @Override public String toString() { return "ReadError{" + diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/PersistComponentsStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/PersistComponentsStep.java index 3435a9c4c7c..b61f462d9f9 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/PersistComponentsStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/PersistComponentsStep.java @@ -383,10 +383,10 @@ public class PersistComponentsStep implements ComputationStep { componentDto.setBranchUuid(path.root().getDto().uuid()); ComponentDto parentModule = StreamSupport.stream(path.getCurrentPath().spliterator(), false) - .filter(p -> p.getComponent().getType() == Component.Type.PROJECT) + .filter(p -> p.component().getType() == Component.Type.PROJECT) .findFirst() .get() - .getElement().getDto(); + .element().getDto(); componentDto.setUuidPath(formatUuidPathFromParent(path.parent().getDto())); componentDto.setRootUuid(parentModule.uuid()); componentDto.setModuleUuid(parentModule.uuid()); diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStep.java index ddb392959e1..48f54ac7c7c 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStep.java @@ -184,7 +184,7 @@ public class QualityGateMeasuresStep implements ComputationStep { builder.addLabel(text); Measure updatedMeasure = Measure.updatedMeasureBuilder(measure.get()) - .setQualityGateStatus(new QualityGateStatus(finalMetricEvaluationResult.evaluationResult.getLevel(), text)) + .setQualityGateStatus(new QualityGateStatus(finalMetricEvaluationResult.evaluationResult.level(), text)) .create(); measureRepository.update(project, metric, updatedMeasure); @@ -198,7 +198,7 @@ public class QualityGateMeasuresStep implements ComputationStep { MetricEvaluationResult metricEvaluationResult = null; for (Condition newCondition : conditions) { EvaluationResult newEvaluationResult = conditionEvaluator.evaluate(newCondition, measure); - if (metricEvaluationResult == null || newEvaluationResult.getLevel().ordinal() > metricEvaluationResult.evaluationResult.getLevel().ordinal()) { + if (metricEvaluationResult == null || newEvaluationResult.level().ordinal() > metricEvaluationResult.evaluationResult.level().ordinal()) { metricEvaluationResult = new MetricEvaluationResult(newEvaluationResult, newCondition); } } @@ -244,12 +244,12 @@ public class QualityGateMeasuresStep implements ComputationStep { } public void addEvaluatedCondition(MetricEvaluationResult metricEvaluationResult) { - Measure.Level level = metricEvaluationResult.evaluationResult.getLevel(); + Measure.Level level = metricEvaluationResult.evaluationResult.level(); if (Measure.Level.ERROR == level) { globalLevel = Measure.Level.ERROR; } evaluatedConditions.add( - new EvaluatedCondition(metricEvaluationResult.condition, level, metricEvaluationResult.evaluationResult.getValue())); + new EvaluatedCondition(metricEvaluationResult.condition, level, metricEvaluationResult.evaluationResult.value())); } public List getEvaluatedConditions() { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCase.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCase.java index f0323d61897..0dfa53f96ae 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCase.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCase.java @@ -46,7 +46,7 @@ public class SmallChangesetQualityGateSpecialCase { public boolean appliesTo(Component project, @Nullable MetricEvaluationResult metricEvaluationResult) { return metricEvaluationResult != null - && metricEvaluationResult.evaluationResult.getLevel() != Measure.Level.OK + && metricEvaluationResult.evaluationResult.level() != Measure.Level.OK && METRICS_TO_IGNORE_ON_SMALL_CHANGESETS.contains(metricEvaluationResult.condition.getMetric().getKey()) && config.getConfiguration().getBoolean(CoreProperties.QUALITY_GATE_IGNORE_SMALL_CHANGES).orElse(true) && isSmallChangeset(project); @@ -54,7 +54,7 @@ public class SmallChangesetQualityGateSpecialCase { MetricEvaluationResult apply(MetricEvaluationResult metricEvaluationResult) { return new MetricEvaluationResult( - new EvaluationResult(Measure.Level.OK, metricEvaluationResult.evaluationResult.getValue()), metricEvaluationResult.condition); + new EvaluationResult(Measure.Level.OK, metricEvaluationResult.evaluationResult.value()), metricEvaluationResult.condition); } private boolean isSmallChangeset(Component project) { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStep.java index b8aeebc7cfd..4331ea36598 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStep.java @@ -158,7 +158,7 @@ public class ExportIssuesStep implements ComputationStep { String ruleUuid = rs.getString(2); String ruleKey = rs.getString(3); String repositoryKey = rs.getString(4); - builder.setRuleRef(ruleRegistrar.register(ruleUuid, repositoryKey, ruleKey).getRef()); + builder.setRuleRef(ruleRegistrar.register(ruleUuid, repositoryKey, ruleKey).ref()); } private static void setLocations(ProjectDump.Issue.Builder builder, ResultSet rs, String issueUuid) throws SQLException { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/ExportRuleStep.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/ExportRuleStep.java index bcc2ea53a2e..6367ddbcbc9 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/ExportRuleStep.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/ExportRuleStep.java @@ -61,9 +61,9 @@ public class ExportRuleStep implements ComputationStep { private static ProjectDump.Rule toRuleMessage(ProjectDump.Rule.Builder ruleBuilder, Rule rule) { ruleBuilder.clear(); return ruleBuilder - .setRef(rule.getRef()) - .setKey(rule.getKey()) - .setRepository(rule.getRepository()) + .setRef(rule.ref()) + .setKey(rule.key()) + .setRepository(rule.repository()) .build(); } } diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/Rule.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/Rule.java index 884a23bb751..b41d9971cd0 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/Rule.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/Rule.java @@ -21,34 +21,16 @@ package org.sonar.ce.task.projectexport.rule; import java.util.Objects; import javax.annotation.Nullable; -import javax.annotation.concurrent.Immutable; import static java.util.Objects.requireNonNull; -@Immutable -public final class Rule { - private final String ref; - private final String repository; - private final String key; - +public record Rule(String ref, String repository, String key) { public Rule(String ref, String repository, String key) { this.ref = ref; this.repository = requireNonNull(repository, "repository can not be null"); this.key = requireNonNull(key, "key can not be null"); } - public String getRef() { - return ref; - } - - public String getRepository() { - return repository; - } - - public String getKey() { - return key; - } - @Override public boolean equals(@Nullable Object o) { if (this == o) { diff --git a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImpl.java b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImpl.java index c3bb5b05936..fb6fc337cf8 100644 --- a/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImpl.java +++ b/server/sonar-ce-task-projectanalysis/src/main/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImpl.java @@ -37,10 +37,10 @@ public class RuleRepositoryImpl implements RuleRepository { Rule rule = rulesByUuid.get(ref); if (rule != null) { - if (!ruleKey.repository().equals(rule.getRepository()) || !ruleKey.rule().equals(rule.getKey())) { + if (!ruleKey.repository().equals(rule.repository()) || !ruleKey.rule().equals(rule.key())) { throw new IllegalArgumentException(format( "Specified RuleKey '%s' is not equal to the one already registered in repository for ref %s: '%s'", - ruleKey, ref, RuleKey.of(rule.getRepository(), rule.getKey()))); + ruleKey, ref, RuleKey.of(rule.repository(), rule.key()))); } return rule; } diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/component/CallRecorderPathAwareVisitor.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/component/CallRecorderPathAwareVisitor.java index 555d3847dcc..6a32c5aaef4 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/component/CallRecorderPathAwareVisitor.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/component/CallRecorderPathAwareVisitor.java @@ -85,7 +85,7 @@ class CallRecorderPathAwareVisitor extends PathAwareVisitorAdapter { } private static List toValueList(Path path) { - return StreamSupport.stream(path.getCurrentPath().spliterator(), false).map(PathElement::getElement).collect(Collectors.toList()); + return StreamSupport.stream(path.getCurrentPath().spliterator(), false).map(PathElement::element).collect(Collectors.toList()); } private static Integer getParent(Path path) { diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStepTest.java index a8ac83af9eb..c3116579358 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/FileMoveDetectionStepTest.java @@ -340,8 +340,8 @@ public class FileMoveDetectionStepTest { assertThat(movedFilesRepository.getComponentsWithOriginal()).containsExactly(file2); MovedFilesRepository.OriginalFile originalFile = movedFilesRepository.getOriginalFile(file2).get(); - assertThat(originalFile.getKey()).isEqualTo(dtos[0].getKey()); - assertThat(originalFile.getUuid()).isEqualTo(dtos[0].uuid()); + assertThat(originalFile.key()).isEqualTo(dtos[0].getKey()); + assertThat(originalFile.uuid()).isEqualTo(dtos[0].uuid()); assertThat(addedFileRepository.getComponents()).isEmpty(); verifyStatistics(context, 1, 1, 1, 1); } @@ -505,11 +505,11 @@ public class FileMoveDetectionStepTest { assertThat(movedFilesRepository.getComponentsWithOriginal()).containsOnly(file3, file6); MovedFilesRepository.OriginalFile originalFile2 = movedFilesRepository.getOriginalFile(file3).get(); - assertThat(originalFile2.getKey()).isEqualTo(dtos[0].getKey()); - assertThat(originalFile2.getUuid()).isEqualTo(dtos[0].uuid()); + assertThat(originalFile2.key()).isEqualTo(dtos[0].getKey()); + assertThat(originalFile2.uuid()).isEqualTo(dtos[0].uuid()); MovedFilesRepository.OriginalFile originalFile5 = movedFilesRepository.getOriginalFile(file6).get(); - assertThat(originalFile5.getKey()).isEqualTo(dtos[3].getKey()); - assertThat(originalFile5.getUuid()).isEqualTo(dtos[3].uuid()); + assertThat(originalFile5.key()).isEqualTo(dtos[3].getKey()); + assertThat(originalFile5.uuid()).isEqualTo(dtos[3].uuid()); assertThat(scoreMatrixDumper.scoreMatrix.getMaxScore()).isGreaterThan(MIN_REQUIRED_SCORE); assertThat(addedFileRepository.getComponents()).isEmpty(); verifyStatistics(context, 3, 4, 2, 2); @@ -581,11 +581,11 @@ public class FileMoveDetectionStepTest { migrationRb1238, addComponentUuidAndAnalysisUuidColumnToDuplicationsIndex); - assertThat(movedFilesRepository.getOriginalFile(makeComponentUuidAndAnalysisUuidNotNullOnDuplicationsIndex).get().getUuid()) + assertThat(movedFilesRepository.getOriginalFile(makeComponentUuidAndAnalysisUuidNotNullOnDuplicationsIndex).get().uuid()) .isEqualTo("uuid_" + "MakeComponentUuidNotNullOnDuplicationsIndex.java".hashCode()); - assertThat(movedFilesRepository.getOriginalFile(migrationRb1238).get().getUuid()) + assertThat(movedFilesRepository.getOriginalFile(migrationRb1238).get().uuid()) .isEqualTo("uuid_" + "1242_make_analysis_uuid_not_null_on_duplications_index.rb".hashCode()); - assertThat(movedFilesRepository.getOriginalFile(addComponentUuidAndAnalysisUuidColumnToDuplicationsIndex).get().getUuid()) + assertThat(movedFilesRepository.getOriginalFile(addComponentUuidAndAnalysisUuidColumnToDuplicationsIndex).get().uuid()) .isEqualTo("uuid_" + "AddComponentUuidColumnToDuplicationsIndex.java".hashCode()); verifyStatistics(context, comps.values().size(), 12, 6, 3); } diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/MatchTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/MatchTest.java index 3e53abe8b6b..43e4ae16397 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/MatchTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/MatchTest.java @@ -42,12 +42,12 @@ public class MatchTest { @Test public void getDbKey_returns_first_constructor_argument() { - assertThat(underTest.getDbUuid()).isEqualTo(SOME_KEY); + assertThat(underTest.dbUuid()).isEqualTo(SOME_KEY); } @Test public void getDbKey_returns_second_constructor_argument() { - assertThat(underTest.getReportUuid()).isEqualTo(SOME_REPORT_KEY); + assertThat(underTest.reportUuid()).isEqualTo(SOME_REPORT_KEY); } @Test diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStepTest.java index 1028485f4fd..5c734d08786 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/filemove/PullRequestFileMoveDetectionStepTest.java @@ -245,8 +245,8 @@ public class PullRequestFileMoveDetectionStepTest { assertThat(movedFilesRepository.getOriginalPullRequestFile(fileInReport)).isPresent(); OriginalFile detectedOriginalFile = movedFilesRepository.getOriginalPullRequestFile(fileInReport).get(); - assertThat(detectedOriginalFile.getKey()).isEqualTo(originalFileInDatabase.getKey()); - assertThat(detectedOriginalFile.getUuid()).isEqualTo(originalFileInDatabase.getUuid()); + assertThat(detectedOriginalFile.key()).isEqualTo(originalFileInDatabase.getKey()); + assertThat(detectedOriginalFile.uuid()).isEqualTo(originalFileInDatabase.getUuid()); } private Map initializeTargetBranchDatabaseComponents(Set references) { diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitorTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitorTest.java index 5ed5f0dd959..6905fc30a31 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitorTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/issue/MovedIssueVisitorTest.java @@ -103,7 +103,7 @@ public class MovedIssueVisitorTest { @Test public void onIssue_update_component_and_module_fields_to_component_and_flag_issue_has_changed() { MovedFilesRepository.OriginalFile originalFile = new MovedFilesRepository.OriginalFile("original uuid", "original key"); - DefaultIssue issue = mockIssue(originalFile.getUuid()); + DefaultIssue issue = mockIssue(originalFile.uuid()); when(movedFilesRepository.getOriginalFile(FILE)) .thenReturn(Optional.of(originalFile)); diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplateTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplateTest.java new file mode 100644 index 00000000000..49ff9de5b51 --- /dev/null +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationEmailTemplateTest.java @@ -0,0 +1,68 @@ +/* + * SonarQube + * Copyright (C) 2009-2023 SonarSource SA + * mailto:info AT sonarsource DOT com + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 3 of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with this program; if not, write to the Free Software Foundation, + * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + */ +package org.sonar.ce.task.projectanalysis.notification; + +import org.junit.Test; +import org.sonar.api.config.EmailSettings; +import org.sonar.server.qualitygate.notification.QGChangeNotification; + +import static org.assertj.core.api.Assertions.assertThat; +import static org.mockito.Mockito.mock; +import static org.mockito.Mockito.when; + +public class ReportAnalysisFailureNotificationEmailTemplateTest { + ReportAnalysisFailureNotificationSerializer serializer = new ReportAnalysisFailureNotificationSerializerImpl(); + EmailSettings settings = mock(EmailSettings.class); + + ReportAnalysisFailureNotificationEmailTemplate underTest = new ReportAnalysisFailureNotificationEmailTemplate(serializer, settings); + + @Test + public void should_not_format_other_than_analysis_failure() { + assertThat(underTest.format(new QGChangeNotification())).isNull(); + } + + @Test + public void check_formatting() { + ReportAnalysisFailureNotification notification = mock(ReportAnalysisFailureNotification.class); + + when(notification.getFieldValue("project.uuid")).thenReturn("uuid"); + when(notification.getFieldValue("project.name")).thenReturn("name"); + when(notification.getFieldValue("project.key")).thenReturn("key"); + when(notification.getFieldValue("project.branch")).thenReturn("branch"); + when(notification.getFieldValue("task.uuid")).thenReturn("task_uuid"); + when(notification.getFieldValue("task.createdAt")).thenReturn("1673449576159"); + when(notification.getFieldValue("task.failedAt")).thenReturn("1673449576159"); + when(notification.getFieldValue("error.message")).thenReturn("error"); + + when(settings.getServerBaseURL()).thenReturn("sonarsource.com"); + + var result = underTest.format(notification); + + assertThat(result.getSubject()).isEqualTo("name: Background task in failure"); + assertThat(result.getMessage()) + .contains(""" + Project: name + Background task: task_uuid""") + .contains(""" + Error message: error + + More details at: sonarsource.com/project/background_tasks?id=key"""); + } +} diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandlerTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandlerTest.java index 289f04fbb30..3ac67b2f538 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandlerTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/notification/ReportAnalysisFailureNotificationHandlerTest.java @@ -161,7 +161,7 @@ public class ReportAnalysisFailureNotificationHandlerTest { .map(login -> new EmailRecipient(login, emailOf(login))) .collect(toSet()); Set expectedRequests = emailRecipients.stream() - .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif))) + .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif))) .collect(toSet()); when(emailNotificationChannel.isActivated()).thenReturn(true); when(notificationManager.findSubscribedEmailRecipients(REPORT_FAILURE_DISPATCHER_KEY, projectKey, REQUIRED_SUBSCRIBER_PERMISSIONS)) @@ -202,9 +202,9 @@ public class ReportAnalysisFailureNotificationHandlerTest { .thenReturn(emailRecipients2); Set expectedRequests = Stream.concat( emailRecipients1.stream() - .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif))), + .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif))), emailRecipients2.stream() - .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif)))) + .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif)))) .collect(toSet()); int deliver = underTest.deliver(Stream.concat(notifications1.stream(), notifications2.stream()).collect(toSet())); diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultAssert.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultAssert.java index ca3bf6b8162..95fc43f7b6b 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultAssert.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultAssert.java @@ -37,8 +37,8 @@ class EvaluationResultAssert extends AbstractAssert but was <%s>", expected, actual.getLevel()); + if (actual.level() != expected) { + failWithMessage("Expected Level to be <%s> but was <%s>", expected, actual.level()); } return this; @@ -47,8 +47,8 @@ class EvaluationResultAssert extends AbstractAssert expected) { isNotNull(); - if (!Objects.equals(actual.getValue(), expected)) { - failWithMessage("Expected Value to be <%s> but was <%s>", expected, actual.getValue()); + if (!Objects.equals(actual.value(), expected)) { + failWithMessage("Expected Value to be <%s> but was <%s>", expected, actual.value()); } return this; diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTest.java index 7c1205f5d8a..86cb51ce34d 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/qualitygate/EvaluationResultTest.java @@ -38,8 +38,8 @@ public class EvaluationResultTest { Measure.Level level = Measure.Level.OK; EvaluationResult evaluationResult = new EvaluationResult(level, value); - assertThat(evaluationResult.getLevel()).isEqualTo(level); - assertThat(evaluationResult.getValue()).isEqualTo(value); + assertThat(evaluationResult.level()).isEqualTo(level); + assertThat(evaluationResult.value()).isEqualTo(value); } @Test diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataComputerTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataComputerTest.java index ecdb48f1a4d..62ba991acf9 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataComputerTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/source/FileSourceDataComputerTest.java @@ -102,7 +102,7 @@ public class FileSourceDataComputerTest { ArgumentCaptor readErrorCaptor = ArgumentCaptor.forClass(LineReader.ReadError.class); verify(fileSourceDataWarnings, times(lineCount)).addWarning(same(FILE), readErrorCaptor.capture()); assertThat(readErrorCaptor.getAllValues()) - .extracting(LineReader.ReadError::getLine) + .extracting(LineReader.ReadError::line) .containsExactly(IntStream.range(randomStartPoint, randomStartPoint + lineCount).boxed().toArray(Integer[]::new)); } diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/NewCoverageMeasuresStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/NewCoverageMeasuresStepTest.java index 537f184b19c..f1cce237ef7 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/NewCoverageMeasuresStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/NewCoverageMeasuresStepTest.java @@ -266,15 +266,15 @@ public class NewCoverageMeasuresStepTest { private void verify_aggregates_variations(LinesAndConditionsWithUncoveredMetricKeys metricKeys, String codeCoverageKey, String lineCoverageKey, String branchCoverageKey) { treeRootHolder.setRoot(MULTIPLE_FILES_TREE); measureRepository - .addRawMeasure(FILE_1_REF, metricKeys.getLines(), createMeasure(3000)) - .addRawMeasure(FILE_1_REF, metricKeys.getConditions(), createMeasure(300)) - .addRawMeasure(FILE_1_REF, metricKeys.getUncoveredLines(), createMeasure(30)) - .addRawMeasure(FILE_1_REF, metricKeys.getUncoveredConditions(), createMeasure(9)) - - .addRawMeasure(FILE_2_REF, metricKeys.getLines(), createMeasure(2000)) - .addRawMeasure(FILE_2_REF, metricKeys.getConditions(), createMeasure(400)) - .addRawMeasure(FILE_2_REF, metricKeys.getUncoveredLines(), createMeasure(200)) - .addRawMeasure(FILE_2_REF, metricKeys.getUncoveredConditions(), createMeasure(16)); + .addRawMeasure(FILE_1_REF, metricKeys.lines(), createMeasure(3000)) + .addRawMeasure(FILE_1_REF, metricKeys.conditions(), createMeasure(300)) + .addRawMeasure(FILE_1_REF, metricKeys.uncoveredLines(), createMeasure(30)) + .addRawMeasure(FILE_1_REF, metricKeys.uncoveredConditions(), createMeasure(9)) + + .addRawMeasure(FILE_2_REF, metricKeys.lines(), createMeasure(2000)) + .addRawMeasure(FILE_2_REF, metricKeys.conditions(), createMeasure(400)) + .addRawMeasure(FILE_2_REF, metricKeys.uncoveredLines(), createMeasure(200)) + .addRawMeasure(FILE_2_REF, metricKeys.uncoveredConditions(), createMeasure(16)); underTest.execute(new TestComputationStepContext()); diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStepTest.java index c1ab8c58c1f..52c75e9c326 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/QualityGateMeasuresStepTest.java @@ -108,7 +108,7 @@ public class QualityGateMeasuresStepTest { public String answer(InvocationOnMock invocation) { Condition condition = (Condition) invocation.getArguments()[0]; EvaluationResult evaluationResult = (EvaluationResult) invocation.getArguments()[1]; - return dumbResultTextAnswer(condition, evaluationResult.getLevel(), evaluationResult.getValue()); + return dumbResultTextAnswer(condition, evaluationResult.level(), evaluationResult.value()); } }); } diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ReportCoverageMeasuresStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ReportCoverageMeasuresStepTest.java index 01eb8b8cc83..71c83a0488d 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ReportCoverageMeasuresStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ReportCoverageMeasuresStepTest.java @@ -136,15 +136,15 @@ public class ReportCoverageMeasuresStepTest { private void verify_coverage_aggregates_values(LinesAndConditionsWithUncoveredMetricKeys metricKeys, String codeCoverageKey, String lineCoverageKey, String branchCoverageKey) { measureRepository - .addRawMeasure(FILE_1_REF, metricKeys.getLines(), newMeasureBuilder().create(3000)) - .addRawMeasure(FILE_1_REF, metricKeys.getConditions(), newMeasureBuilder().create(300)) - .addRawMeasure(FILE_1_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(30)) - .addRawMeasure(FILE_1_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(9)) - - .addRawMeasure(FILE_2_REF, metricKeys.getLines(), newMeasureBuilder().create(2000)) - .addRawMeasure(FILE_2_REF, metricKeys.getConditions(), newMeasureBuilder().create(400)) - .addRawMeasure(FILE_2_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(200)) - .addRawMeasure(FILE_2_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(16)); + .addRawMeasure(FILE_1_REF, metricKeys.lines(), newMeasureBuilder().create(3000)) + .addRawMeasure(FILE_1_REF, metricKeys.conditions(), newMeasureBuilder().create(300)) + .addRawMeasure(FILE_1_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(30)) + .addRawMeasure(FILE_1_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(9)) + + .addRawMeasure(FILE_2_REF, metricKeys.lines(), newMeasureBuilder().create(2000)) + .addRawMeasure(FILE_2_REF, metricKeys.conditions(), newMeasureBuilder().create(400)) + .addRawMeasure(FILE_2_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(200)) + .addRawMeasure(FILE_2_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(16)); underTest.execute(new TestComputationStepContext()); diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCaseTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCaseTest.java index 855c4be4973..90dcbb5e1eb 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCaseTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/SmallChangesetQualityGateSpecialCaseTest.java @@ -145,8 +145,8 @@ public class SmallChangesetQualityGateSpecialCaseTest { QualityGateMeasuresStep.MetricEvaluationResult modified = underTest.apply(original); - assertThat(modified.evaluationResult.getLevel()).isSameAs(OK); - assertThat(modified.evaluationResult.getValue()).isSameAs(value); + assertThat(modified.evaluationResult.level()).isSameAs(OK); + assertThat(modified.evaluationResult.value()).isSameAs(value); assertThat(modified.condition).isSameAs(condition); } diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ViewsCoverageMeasuresStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ViewsCoverageMeasuresStepTest.java index db47b230941..d16cf6b0bd9 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ViewsCoverageMeasuresStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectanalysis/step/ViewsCoverageMeasuresStepTest.java @@ -91,37 +91,37 @@ public class ViewsCoverageMeasuresStepTest { private void verify_lines_and_conditions_aggregates_values(LinesAndConditionsWithUncoveredMetricKeys metricKeys) { measureRepository - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getLines(), newMeasureBuilder().create(3000)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getConditions(), newMeasureBuilder().create(300)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(30)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(9)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.lines(), newMeasureBuilder().create(3000)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.conditions(), newMeasureBuilder().create(300)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(30)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(9)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getLines(), newMeasureBuilder().create(2000)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getConditions(), newMeasureBuilder().create(400)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(200)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(16)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.lines(), newMeasureBuilder().create(2000)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.conditions(), newMeasureBuilder().create(400)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(200)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(16)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getLines(), newMeasureBuilder().create(1000)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getConditions(), newMeasureBuilder().create(500)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(300)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(19)); + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.lines(), newMeasureBuilder().create(1000)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.conditions(), newMeasureBuilder().create(500)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(300)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(19)); underTest.execute(new TestComputationStepContext()); MeasureRepoEntry[] subViewRepoEntries = { - entryOf(metricKeys.getLines(), newMeasureBuilder().create(5000)), - entryOf(metricKeys.getConditions(), newMeasureBuilder().create(700)), - entryOf(metricKeys.getUncoveredLines(), newMeasureBuilder().create(230)), - entryOf(metricKeys.getUncoveredConditions(), newMeasureBuilder().create(25)) + entryOf(metricKeys.lines(), newMeasureBuilder().create(5000)), + entryOf(metricKeys.conditions(), newMeasureBuilder().create(700)), + entryOf(metricKeys.uncoveredLines(), newMeasureBuilder().create(230)), + entryOf(metricKeys.uncoveredConditions(), newMeasureBuilder().create(25)) }; assertThat(toEntries(measureRepository.getAddedRawMeasures(SUB_SUBVIEW_REF))).contains(subViewRepoEntries); assertThat(toEntries(measureRepository.getAddedRawMeasures(SUBVIEW_REF))).contains(subViewRepoEntries); assertThat(toEntries(measureRepository.getAddedRawMeasures(ROOT_REF))).contains( - entryOf(metricKeys.getLines(), newMeasureBuilder().create(6000)), - entryOf(metricKeys.getConditions(), newMeasureBuilder().create(1200)), - entryOf(metricKeys.getUncoveredLines(), newMeasureBuilder().create(530)), - entryOf(metricKeys.getUncoveredConditions(), newMeasureBuilder().create(44))); + entryOf(metricKeys.lines(), newMeasureBuilder().create(6000)), + entryOf(metricKeys.conditions(), newMeasureBuilder().create(1200)), + entryOf(metricKeys.uncoveredLines(), newMeasureBuilder().create(530)), + entryOf(metricKeys.uncoveredConditions(), newMeasureBuilder().create(44))); } @Test @@ -138,20 +138,20 @@ public class ViewsCoverageMeasuresStepTest { private void verify_coverage_aggregates_values(LinesAndConditionsWithUncoveredMetricKeys metricKeys, String codeCoverageKey, String lineCoverageKey, String branchCoverageKey) { measureRepository - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getLines(), newMeasureBuilder().create(3000)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getConditions(), newMeasureBuilder().create(300)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(30)) - .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(9)) - - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getLines(), newMeasureBuilder().create(2000)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getConditions(), newMeasureBuilder().create(400)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(200)) - .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(16)) - - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getLines(), newMeasureBuilder().create(1000)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getConditions(), newMeasureBuilder().create(500)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getUncoveredLines(), newMeasureBuilder().create(300)) - .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.getUncoveredConditions(), newMeasureBuilder().create(19)); + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.lines(), newMeasureBuilder().create(3000)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.conditions(), newMeasureBuilder().create(300)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(30)) + .addRawMeasure(PROJECTVIEW_1_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(9)) + + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.lines(), newMeasureBuilder().create(2000)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.conditions(), newMeasureBuilder().create(400)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(200)) + .addRawMeasure(PROJECTVIEW_2_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(16)) + + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.lines(), newMeasureBuilder().create(1000)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.conditions(), newMeasureBuilder().create(500)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.uncoveredLines(), newMeasureBuilder().create(300)) + .addRawMeasure(PROJECTVIEW_3_REF, metricKeys.uncoveredConditions(), newMeasureBuilder().create(19)); underTest.execute(new TestComputationStepContext()); diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStepTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStepTest.java index 82890f6028f..7d9544b0a83 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStepTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/issue/ExportIssuesStepTest.java @@ -292,7 +292,7 @@ public class ExportIssuesStepTest { ProjectDump.Issue issue = getWrittenIssue(); assertThat(issue.getRuleRef()) - .isEqualTo(ruleRepository.register(issueDto.getRuleUuid(), readyRuleDto.getKey()).getRef()); + .isEqualTo(ruleRepository.register(issueDto.getRuleUuid(), readyRuleDto.getKey()).ref()); } @Test diff --git a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImplTest.java b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImplTest.java index ca7a36d04ac..f50016cfb0e 100644 --- a/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImplTest.java +++ b/server/sonar-ce-task-projectanalysis/src/test/java/org/sonar/ce/task/projectexport/rule/RuleRepositoryImplTest.java @@ -51,9 +51,9 @@ public class RuleRepositoryImplTest { underTest.register(SOME_UUID, RuleKey.of(SOME_REPOSITORY, SOME_RULE_KEY)); for (int i = 0; i < someRandomInt(); i++) { Rule otherRule = underTest.register(Integer.toString(i), RuleKey.of(SOME_REPOSITORY, SOME_RULE_KEY)); - assertThat(otherRule.getRef()).isEqualTo(Integer.toString(i)); - assertThat(otherRule.getRepository()).isEqualTo(SOME_REPOSITORY); - assertThat(otherRule.getKey()).isEqualTo(SOME_RULE_KEY); + assertThat(otherRule.ref()).isEqualTo(Integer.toString(i)); + assertThat(otherRule.repository()).isEqualTo(SOME_REPOSITORY); + assertThat(otherRule.key()).isEqualTo(SOME_RULE_KEY); } } @@ -79,7 +79,7 @@ public class RuleRepositoryImplTest { public void register_returns_the_same_object_for_every_call_with_equals_RuleKey_objects() { Rule rule = underTest.register(SOME_UUID, RuleKey.of(SOME_REPOSITORY, SOME_RULE_KEY)); for (int i = 0; i < someRandomInt(); i++) { - assertThat(underTest.register(Uuids.createFast(), RuleKey.of(SOME_REPOSITORY, SOME_RULE_KEY)).getRef()).isNotEqualTo(rule.getRef()); + assertThat(underTest.register(Uuids.createFast(), RuleKey.of(SOME_REPOSITORY, SOME_RULE_KEY)).ref()).isNotEqualTo(rule.ref()); } } @@ -89,9 +89,9 @@ public class RuleRepositoryImplTest { String repository = SOME_REPOSITORY + i; String ruleKey = String.valueOf(i); Rule rule = underTest.register(Integer.toString(i), RuleKey.of(repository, ruleKey)); - assertThat(rule.getRef()).isEqualTo(Integer.toString(i)); - assertThat(rule.getRepository()).isEqualTo(repository); - assertThat(rule.getKey()).isEqualTo(ruleKey); + assertThat(rule.ref()).isEqualTo(Integer.toString(i)); + assertThat(rule.repository()).isEqualTo(repository); + assertThat(rule.key()).isEqualTo(ruleKey); } } @@ -119,8 +119,8 @@ public class RuleRepositoryImplTest { Collection all = underTest.getAll(); - assertThat(all).extracting(Rule::getRepository).containsOnly(repositories); - assertThat(all).extracting(Rule::getKey).containsOnly(keys); + assertThat(all).extracting(Rule::repository).containsOnly(repositories); + assertThat(all).extracting(Rule::key).containsOnly(keys); assertThatThrownBy(() -> all.add(SOME_RULE)) .isInstanceOf(UnsupportedOperationException.class); diff --git a/server/sonar-ce-task/src/main/java/org/sonar/ce/task/CeTask.java b/server/sonar-ce-task/src/main/java/org/sonar/ce/task/CeTask.java index 0c33e8afa42..82aed72779c 100644 --- a/server/sonar-ce-task/src/main/java/org/sonar/ce/task/CeTask.java +++ b/server/sonar-ce-task/src/main/java/org/sonar/ce/task/CeTask.java @@ -59,25 +59,12 @@ public class CeTask { } } - @Immutable - public static final class User { - private final String uuid; - private final String login; - + public record User(String uuid, String login) { public User(String uuid, @Nullable String login) { this.uuid = requireNonNull(uuid); this.login = emptyToNull(login); } - public String getUuid() { - return uuid; - } - - @CheckForNull - public String getLogin() { - return login; - } - @Override public boolean equals(Object o) { if (this == o) { diff --git a/server/sonar-ce-task/src/test/java/org/sonar/ce/task/CeTaskTest.java b/server/sonar-ce-task/src/test/java/org/sonar/ce/task/CeTaskTest.java index 65fcdc9f9ee..c773ae1c5ce 100644 --- a/server/sonar-ce-task/src/test/java/org/sonar/ce/task/CeTaskTest.java +++ b/server/sonar-ce-task/src/test/java/org/sonar/ce/task/CeTaskTest.java @@ -110,7 +110,7 @@ public class CeTaskTest { .setSubmitter(new CeTask.User("USER_ID", "")) .build(); - assertThat(ceTask.getSubmitter().getLogin()).isNull(); + assertThat(ceTask.getSubmitter().login()).isNull(); } @Test @@ -140,8 +140,8 @@ public class CeTaskTest { public void verify_submitter_getters() { CeTask.User user = new CeTask.User("UUID", "LOGIN"); - assertThat(user.getUuid()).isEqualTo("UUID"); - assertThat(user.getLogin()).isEqualTo("LOGIN"); + assertThat(user.uuid()).isEqualTo("UUID"); + assertThat(user.login()).isEqualTo("LOGIN"); } @Test diff --git a/server/sonar-ce/src/main/java/org/sonar/ce/taskprocessor/CeWorkerImpl.java b/server/sonar-ce/src/main/java/org/sonar/ce/taskprocessor/CeWorkerImpl.java index 327f4317a0f..917a6fe582b 100644 --- a/server/sonar-ce/src/main/java/org/sonar/ce/taskprocessor/CeWorkerImpl.java +++ b/server/sonar-ce/src/main/java/org/sonar/ce/taskprocessor/CeWorkerImpl.java @@ -287,11 +287,11 @@ public class CeWorkerImpl implements CeWorker { if (submitter == null) { return null; } - String submitterLogin = submitter.getLogin(); + String submitterLogin = submitter.login(); if (submitterLogin != null) { return submitterLogin; } else { - return submitter.getUuid(); + return submitter.uuid(); } } diff --git a/server/sonar-ce/src/test/java/org/sonar/ce/notification/ReportAnalysisFailureNotificationExecutionListenerTest.java b/server/sonar-ce/src/test/java/org/sonar/ce/notification/ReportAnalysisFailureNotificationExecutionListenerTest.java index d018cb6ecde..fc223dc7f2a 100644 --- a/server/sonar-ce/src/test/java/org/sonar/ce/notification/ReportAnalysisFailureNotificationExecutionListenerTest.java +++ b/server/sonar-ce/src/test/java/org/sonar/ce/notification/ReportAnalysisFailureNotificationExecutionListenerTest.java @@ -221,15 +221,15 @@ public class ReportAnalysisFailureNotificationExecutionListenerTest { ReportAnalysisFailureNotificationBuilder reportAnalysisFailureNotificationBuilder = notificationCaptor.getValue(); - ReportAnalysisFailureNotificationBuilder.Project notificationProject = reportAnalysisFailureNotificationBuilder.getProject(); - assertThat(notificationProject.getName()).isEqualTo(project.name()); - assertThat(notificationProject.getKey()).isEqualTo(project.getKey()); - assertThat(notificationProject.getUuid()).isEqualTo(project.uuid()); - assertThat(notificationProject.getBranchName()).isNull(); - ReportAnalysisFailureNotificationBuilder.Task notificationTask = reportAnalysisFailureNotificationBuilder.getTask(); - assertThat(notificationTask.getUuid()).isEqualTo(taskUuid); - assertThat(notificationTask.getCreatedAt()).isEqualTo(createdAt); - assertThat(notificationTask.getFailedAt()).isEqualTo(executedAt); + ReportAnalysisFailureNotificationBuilder.Project notificationProject = reportAnalysisFailureNotificationBuilder.project(); + assertThat(notificationProject.name()).isEqualTo(project.name()); + assertThat(notificationProject.key()).isEqualTo(project.getKey()); + assertThat(notificationProject.uuid()).isEqualTo(project.uuid()); + assertThat(notificationProject.branchName()).isNull(); + ReportAnalysisFailureNotificationBuilder.Task notificationTask = reportAnalysisFailureNotificationBuilder.task(); + assertThat(notificationTask.uuid()).isEqualTo(taskUuid); + assertThat(notificationTask.createdAt()).isEqualTo(createdAt); + assertThat(notificationTask.failedAt()).isEqualTo(executedAt); } @Test @@ -243,7 +243,7 @@ public class ReportAnalysisFailureNotificationExecutionListenerTest { ArgumentCaptor notificationCaptor = verifyAndCaptureSerializedNotification(); ReportAnalysisFailureNotificationBuilder reportAnalysisFailureNotificationBuilder = notificationCaptor.getValue(); - assertThat(reportAnalysisFailureNotificationBuilder.getErrorMessage()).isEqualTo(message); + assertThat(reportAnalysisFailureNotificationBuilder.errorMessage()).isEqualTo(message); } @Test @@ -258,7 +258,7 @@ public class ReportAnalysisFailureNotificationExecutionListenerTest { ArgumentCaptor notificationCaptor = verifyAndCaptureSerializedNotification(); ReportAnalysisFailureNotificationBuilder reportAnalysisFailureNotificationBuilder = notificationCaptor.getValue(); - assertThat(reportAnalysisFailureNotificationBuilder.getErrorMessage()).isNull(); + assertThat(reportAnalysisFailureNotificationBuilder.errorMessage()).isNull(); } @Test @@ -296,7 +296,7 @@ public class ReportAnalysisFailureNotificationExecutionListenerTest { verify(notificationService).deliver(same(notificationMock)); ArgumentCaptor notificationCaptor = verifyAndCaptureSerializedNotification(); - assertThat(notificationCaptor.getValue().getTask().getFailedAt()).isEqualTo(now); + assertThat(notificationCaptor.getValue().task().failedAt()).isEqualTo(now); } private ReportAnalysisFailureNotification mockSerializer() { diff --git a/server/sonar-ce/src/test/java/org/sonar/ce/queue/InternalCeQueueImplTest.java b/server/sonar-ce/src/test/java/org/sonar/ce/queue/InternalCeQueueImplTest.java index f1ab19d7927..4a537ed3120 100644 --- a/server/sonar-ce/src/test/java/org/sonar/ce/queue/InternalCeQueueImplTest.java +++ b/server/sonar-ce/src/test/java/org/sonar/ce/queue/InternalCeQueueImplTest.java @@ -656,11 +656,11 @@ public class InternalCeQueueImplTest { } if (taskSubmit.getSubmitterUuid() != null) { if (userDto == null) { - assertThat(task.getSubmitter().getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); - assertThat(task.getSubmitter().getLogin()).isNull(); + assertThat(task.getSubmitter().uuid()).isEqualTo(taskSubmit.getSubmitterUuid()); + assertThat(task.getSubmitter().login()).isNull(); } else { - assertThat(task.getSubmitter().getUuid()).isEqualTo(userDto.getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); - assertThat(task.getSubmitter().getUuid()).isEqualTo(userDto.getLogin()); + assertThat(task.getSubmitter().uuid()).isEqualTo(userDto.getUuid()).isEqualTo(taskSubmit.getSubmitterUuid()); + assertThat(task.getSubmitter().uuid()).isEqualTo(userDto.getLogin()); } } } diff --git a/server/sonar-ce/src/test/java/org/sonar/ce/taskprocessor/CeWorkerImplTest.java b/server/sonar-ce/src/test/java/org/sonar/ce/taskprocessor/CeWorkerImplTest.java index 685e74e891f..9b55622906f 100644 --- a/server/sonar-ce/src/test/java/org/sonar/ce/taskprocessor/CeWorkerImplTest.java +++ b/server/sonar-ce/src/test/java/org/sonar/ce/taskprocessor/CeWorkerImplTest.java @@ -367,8 +367,8 @@ public class CeWorkerImplTest { verifyWorkerUuid(); List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(0)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=SUCCESS | time=", submitter.getLogin())); + assertThat(logs.get(0)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=SUCCESS | time=", submitter.login())); assertThat(logTester.logs(LoggerLevel.ERROR)).isEmpty(); assertThat(logTester.logs(LoggerLevel.DEBUG)).isEmpty(); } @@ -387,8 +387,8 @@ public class CeWorkerImplTest { verifyWorkerUuid(); List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(0)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.getLogin())); + assertThat(logs.get(0)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.login())); logs = logTester.logs(LoggerLevel.ERROR); assertThat(logs).hasSize(1); assertThat(logs.iterator().next()).isEqualTo("Failed to execute task " + ceTask.getUuid()); @@ -464,8 +464,8 @@ public class CeWorkerImplTest { List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(0)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.getLogin())); + assertThat(logs.get(0)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.login())); logs = logTester.logs(LoggerLevel.ERROR); assertThat(logs).hasSize(1); assertThat(logs.iterator().next()).isEqualTo("Failed to execute task " + ceTask.getUuid()); @@ -482,8 +482,8 @@ public class CeWorkerImplTest { List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(1)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.getLogin())); + assertThat(logs.get(1)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.login())); assertThat(logTester.logs(LoggerLevel.ERROR)).isEmpty(); } @@ -512,8 +512,8 @@ public class CeWorkerImplTest { List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(0)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.getLogin())); + assertThat(logs.get(0)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.login())); List logAndArguments = logTester.getLogs(LoggerLevel.ERROR); assertThat(logAndArguments).hasSize(2); @@ -541,8 +541,8 @@ public class CeWorkerImplTest { List logs = logTester.logs(LoggerLevel.INFO); assertThat(logs).hasSize(2); - assertThat(logs.get(0)).contains(" | submitter=" + submitter.getLogin()); - assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.getLogin())); + assertThat(logs.get(0)).contains(" | submitter=" + submitter.login()); + assertThat(logs.get(1)).contains(String.format(" | submitter=%s | status=FAILED | time=", submitter.login())); List logAndArguments = logTester.getLogs(LoggerLevel.ERROR); assertThat(logAndArguments).hasSize(1); assertThat(logAndArguments.get(0).getFormattedMsg()).isEqualTo("Failed to finalize task with uuid '" + ceTask.getUuid() + "' and persist its state to db"); diff --git a/server/sonar-db-dao/src/main/java/org/sonar/db/event/EventPurgeData.java b/server/sonar-db-dao/src/main/java/org/sonar/db/event/EventPurgeData.java index 1c448a4b49f..29988891dec 100644 --- a/server/sonar-db-dao/src/main/java/org/sonar/db/event/EventPurgeData.java +++ b/server/sonar-db-dao/src/main/java/org/sonar/db/event/EventPurgeData.java @@ -21,20 +21,9 @@ package org.sonar.db.event; import static java.util.Objects.requireNonNull; -public final class EventPurgeData { - private final String componentUuid; - private final String analysisUuid; - +public record EventPurgeData(String componentUuid, String analysisUuid) { public EventPurgeData(String componentUuid, String analysisUuid) { this.componentUuid = requireNonNull(componentUuid); this.analysisUuid = requireNonNull(analysisUuid); } - - public String getComponentUuid() { - return componentUuid; - } - - public String getAnalysisUuid() { - return analysisUuid; - } } diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/event/EventComponentChangeDaoTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/event/EventComponentChangeDaoTest.java index 161735abc46..97a521409b5 100644 --- a/server/sonar-db-dao/src/test/java/org/sonar/db/event/EventComponentChangeDaoTest.java +++ b/server/sonar-db-dao/src/test/java/org/sonar/db/event/EventComponentChangeDaoTest.java @@ -276,7 +276,7 @@ public class EventComponentChangeDaoTest { Arrays.stream(event1Dtos).forEach(dto -> underTest.insert(dbSession, dto, doesNotMatter)); Arrays.stream(event2Dtos).forEach(dto -> underTest.insert(dbSession, dto, doesNotMatter)); - assertThat(underTest.selectByAnalysisUuids(dbSession, singletonList(doesNotMatter.getAnalysisUuid()))) + assertThat(underTest.selectByAnalysisUuids(dbSession, singletonList(doesNotMatter.analysisUuid()))) .extracting( EventComponentChangeDto::getUuid, EventComponentChangeDto::getEventUuid, diff --git a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v00/PopulateInitialSchema.java b/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v00/PopulateInitialSchema.java index 7deeee35168..4f90c1d2ee1 100644 --- a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v00/PopulateInitialSchema.java +++ b/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v00/PopulateInitialSchema.java @@ -203,22 +203,7 @@ public class PopulateInitialSchema extends DataChange { return uuid; } - private static final class Groups { - private final String adminGroupUuid; - private final String userGroupUuid; - - private Groups(String adminGroupUuid, String userGroupUuid) { - this.adminGroupUuid = adminGroupUuid; - this.userGroupUuid = userGroupUuid; - } - - public String getAdminGroupUuid() { - return adminGroupUuid; - } - - public String getUserGroupUuid() { - return userGroupUuid; - } + private record Groups(String adminGroupUuid, String userGroupUuid) { } private void insertGroupRoles(Context context, Groups groups) throws SQLException { @@ -228,14 +213,14 @@ public class PopulateInitialSchema extends DataChange { for (String adminRole : ADMIN_ROLES) { upsert .setString(1, uuidFactory.create()) - .setString(2, groups.getAdminGroupUuid()) + .setString(2, groups.adminGroupUuid()) .setString(3, adminRole) .addBatch(); } for (String anyoneRole : Arrays.asList("scan", "provisioning")) { upsert .setString(1, uuidFactory.create()) - .setString(2, groups.getUserGroupUuid()) + .setString(2, groups.userGroupUuid()) .setString(3, anyoneRole) .addBatch(); } @@ -250,11 +235,11 @@ public class PopulateInitialSchema extends DataChange { Upsert upsert = context.prepareUpsert(createInsertStatement("groups_users", "user_uuid", "group_uuid")); upsert .setString(1, adminUserUuid) - .setString(2, groups.getUserGroupUuid()) + .setString(2, groups.userGroupUuid()) .addBatch(); upsert .setString(1, adminUserUuid) - .setString(2, groups.getAdminGroupUuid()) + .setString(2, groups.adminGroupUuid()) .addBatch(); upsert .execute() diff --git a/server/sonar-main/src/main/java/org/sonar/application/command/JvmOptions.java b/server/sonar-main/src/main/java/org/sonar/application/command/JvmOptions.java index ea0a4ded5f9..f5637492a2c 100644 --- a/server/sonar-main/src/main/java/org/sonar/application/command/JvmOptions.java +++ b/server/sonar-main/src/main/java/org/sonar/application/command/JvmOptions.java @@ -92,14 +92,14 @@ public class JvmOptions { private void checkMandatoryOptionOverwrite(String propertyName, List jvmOptionsFromProperty) { List matches = jvmOptionsFromProperty.stream() .map(jvmOption -> new Match(jvmOption, mandatoryOptionFor(jvmOption))) - .filter(match -> match.getMandatoryOption() != null) + .filter(match -> match.mandatoryOption() != null) .toList(); if (!matches.isEmpty()) { throw new MessageException(format( "a JVM option can't overwrite mandatory JVM options. The following JVM options defined by property '%s' are invalid: %s", propertyName, matches.stream() - .map(m -> m.getOption() + " overwrites " + m.mandatoryOption.getKey() + m.mandatoryOption.getValue()) + .map(m -> m.option() + " overwrites " + m.mandatoryOption.getKey() + m.mandatoryOption.getValue()) .collect(joining(", ")))); } } @@ -158,24 +158,11 @@ public class JvmOptions { return options.toString(); } - private static final class Match { - private final String option; - - private final Map.Entry mandatoryOption; - + private record Match(String option, Map.Entry mandatoryOption) { private Match(String option, @Nullable Map.Entry mandatoryOption) { this.option = option; this.mandatoryOption = mandatoryOption; } - String getOption() { - return option; - } - - @CheckForNull - Map.Entry getMandatoryOption() { - return mandatoryOption; - } - } } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandler.java b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandler.java index ffd237e7390..4be0cd5a127 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandler.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandler.java @@ -108,7 +108,7 @@ public class ChangesOnMyIssueNotificationHandler extends EmailNotificationHandle return subscribedAssignees.stream() .flatMap(recipient -> notificationsWithPeerChangedIssues.stream() // do not notify users of the changes they made themselves - .filter(notification -> !notification.getChange().isAuthorLogin(recipient.getLogin())) + .filter(notification -> !notification.getChange().isAuthorLogin(recipient.login())) .map(notification -> toEmailDeliveryRequest(notification, recipient, projectKeys))) .filter(Objects::nonNull) .collect(toSet(notificationsWithPeerChangedIssues.size())); @@ -140,7 +140,7 @@ public class ChangesOnMyIssueNotificationHandler extends EmailNotificationHandle Set subscribedProjectKeys = (Set) entry.getValue(); return notificationsWithPeerChangedIssues.stream() // do not notify users of the changes they made themselves - .filter(notification -> !notification.getChange().isAuthorLogin(recipient.getLogin())) + .filter(notification -> !notification.getChange().isAuthorLogin(recipient.login())) .map(notification -> toEmailDeliveryRequest(notification, recipient, subscribedProjectKeys)) .filter(Objects::nonNull); }) @@ -156,14 +156,14 @@ public class ChangesOnMyIssueNotificationHandler extends EmailNotificationHandle @CheckForNull private static EmailDeliveryRequest toEmailDeliveryRequest(NotificationWithProjectKeys notification, EmailRecipient recipient, Set subscribedProjectKeys) { Set recipientIssuesByProject = notification.getIssues().stream() - .filter(issue -> issue.getAssignee().filter(assignee -> recipient.getLogin().equals(assignee.getLogin())).isPresent()) + .filter(issue -> issue.getAssignee().filter(assignee -> recipient.login().equals(assignee.getLogin())).isPresent()) .filter(issue -> subscribedProjectKeys.contains(issue.getProject().getKey())) .collect(toSet(notification.getIssues().size())); if (recipientIssuesByProject.isEmpty()) { return null; } return new EmailDeliveryRequest( - recipient.getEmail(), + recipient.email(), new ChangesOnMyIssuesNotification(notification.getChange(), recipientIssuesByProject)); } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandler.java b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandler.java index 9244ad20780..d0f463123e7 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandler.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandler.java @@ -136,7 +136,7 @@ public class FPOrWontFixNotificationHandler extends EmailNotificationHandler toRequests(NotificationWithProjectKeys notification, Set projectKeys, Collection recipients) { return recipients.stream() // do not notify author of the change - .filter(recipient -> !notification.getChange().isAuthorLogin(recipient.getLogin())) + .filter(recipient -> !notification.getChange().isAuthorLogin(recipient.login())) .flatMap(recipient -> { SetMultimap issuesByNewResolution = notification.getIssues().stream() // ignore issues not changed to a FP or Won't Fix resolution @@ -155,7 +155,7 @@ public class FPOrWontFixNotificationHandler extends EmailNotificationHandler new FPOrWontFixNotification(notification.getChange(), wontFixIssues, WONT_FIX)) .orElse(null)) .filter(Objects::nonNull) - .map(fpOrWontFixNotification -> new EmailDeliveryRequest(recipient.getEmail(), fpOrWontFixNotification)); + .map(fpOrWontFixNotification -> new EmailDeliveryRequest(recipient.email(), fpOrWontFixNotification)); }); } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/MyNewIssuesNotificationHandler.java b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/MyNewIssuesNotificationHandler.java index 4a4342db7a6..14d3d6b0e4e 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/MyNewIssuesNotificationHandler.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/MyNewIssuesNotificationHandler.java @@ -91,7 +91,7 @@ public class MyNewIssuesNotificationHandler extends EmailNotificationHandler recipientsByLogin = notificationManager .findSubscribedEmailRecipients(KEY, projectKey, assignees, ALL_MUST_HAVE_ROLE_USER) .stream() - .collect(MoreCollectors.uniqueIndex(NotificationManager.EmailRecipient::getLogin)); + .collect(MoreCollectors.uniqueIndex(NotificationManager.EmailRecipient::login)); return notifications.stream() .map(notification -> toEmailDeliveryRequest(recipientsByLogin, notification)) .filter(Objects::nonNull); @@ -104,7 +104,7 @@ public class MyNewIssuesNotificationHandler extends EmailNotificationHandler getUserNameByUuid(String uuid); } - @Immutable - public static final class RuleDefinition { - private final String name; - private final String language; - + public record RuleDefinition(String name, String language) { public RuleDefinition(String name, @Nullable String language) { this.name = requireNonNull(name, "name can't be null"); this.language = language; } - public String getName() { - return name; - } - - @CheckForNull - public String getLanguage() { - return language; - } - - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - RuleDefinition that = (RuleDefinition) o; - return name.equals(that.name) && Objects.equals(language, that.language); - } - - @Override - public int hashCode() { - return Objects.hash(name, language); - } - @Override public String toString() { return "RuleDefinition{" + name + " (" + language + ')' + '}'; @@ -177,7 +146,7 @@ public class NewIssuesNotification extends Notification { String ruleKey = ruleStats.getKey(); RuleDefinition rule = detailsSupplier.getRuleDefinitionByRuleKey(RuleKey.parse(ruleKey)) .orElseThrow(() -> new IllegalStateException(String.format("Rule with key '%s' does not exist", ruleKey))); - String name = rule.getName() + " (" + rule.getLanguage() + ")"; + String name = rule.name() + " (" + rule.language() + ")"; setFieldValue(metric + DOT + i + LABEL, name); setFieldValue(metric + DOT + i + COUNT, String.valueOf(ruleStats.getValue().getOnCurrentAnalysis())); i++; diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/NewIssuesNotificationHandler.java b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/NewIssuesNotificationHandler.java index 5dea353a119..d1de9b0465f 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/NewIssuesNotificationHandler.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/issue/notification/NewIssuesNotificationHandler.java @@ -82,7 +82,7 @@ public class NewIssuesNotificationHandler extends EmailNotificationHandler notifications.stream() - .map(notification -> new EmailDeliveryRequest(emailRecipient.getEmail(), notification))); + .map(notification -> new EmailDeliveryRequest(emailRecipient.email(), notification))); } } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/notification/DefaultNotificationManager.java b/server/sonar-server-common/src/main/java/org/sonar/server/notification/DefaultNotificationManager.java index a9d52fe27cc..8b6800ae195 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/notification/DefaultNotificationManager.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/notification/DefaultNotificationManager.java @@ -134,8 +134,8 @@ public class DefaultNotificationManager implements NotificationManager { try (DbSession dbSession = dbClient.openSession(false)) { Set authorizedLogins = keepAuthorizedLogins(dbSession, projectKey, subscriberAndChannels, subscriberPermissionsOnProject); subscriberAndChannels.stream() - .filter(subscriberAndChannel -> authorizedLogins.contains(subscriberAndChannel.getSubscriber().getLogin())) - .forEach(subscriberAndChannel -> builder.put(subscriberAndChannel.getSubscriber().getLogin(), subscriberAndChannel.getChannel())); + .filter(subscriberAndChannel -> authorizedLogins.contains(subscriberAndChannel.subscriber().getLogin())) + .forEach(subscriberAndChannel -> builder.put(subscriberAndChannel.subscriber().getLogin(), subscriberAndChannel.channel())); } return builder.build(); } @@ -167,8 +167,8 @@ public class DefaultNotificationManager implements NotificationManager { private Set keepAuthorizedLogins(DbSession dbSession, String projectKey, Set subscriberAndChannels, @Nullable Boolean global, String permission) { Set logins = subscriberAndChannels.stream() - .filter(s -> global == null || s.getSubscriber().isGlobal() == global) - .map(s -> s.getSubscriber().getLogin()) + .filter(s -> global == null || s.subscriber().isGlobal() == global) + .map(s -> s.subscriber().getLogin()) .collect(Collectors.toSet()); if (logins.isEmpty()) { return Collections.emptySet(); @@ -244,22 +244,7 @@ public class DefaultNotificationManager implements NotificationManager { .filter(s -> authorizedLogins.contains(s.getLogin())); } - private static final class SubscriberAndChannel { - private final Subscriber subscriber; - private final NotificationChannel channel; - - private SubscriberAndChannel(Subscriber subscriber, NotificationChannel channel) { - this.subscriber = subscriber; - this.channel = channel; - } - - Subscriber getSubscriber() { - return subscriber; - } - - NotificationChannel getChannel() { - return channel; - } + private record SubscriberAndChannel(Subscriber subscriber, NotificationChannel channel) { @Override public boolean equals(Object o) { @@ -274,10 +259,6 @@ public class DefaultNotificationManager implements NotificationManager { Objects.equals(channel, that.channel); } - @Override - public int hashCode() { - return Objects.hash(subscriber, channel); - } } @VisibleForTesting diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/notification/NotificationManager.java b/server/sonar-server-common/src/main/java/org/sonar/server/notification/NotificationManager.java index c6fadc9658e..6517c4ee9a1 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/notification/NotificationManager.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/notification/NotificationManager.java @@ -22,7 +22,6 @@ package org.sonar.server.notification; import com.google.common.collect.Multimap; import java.util.Objects; import java.util.Set; -import javax.annotation.concurrent.Immutable; import org.sonar.api.notifications.Notification; import org.sonar.api.notifications.NotificationChannel; import org.sonar.api.web.UserRole; @@ -57,29 +56,17 @@ public interface NotificationManager { * @param dispatcher the dispatcher for which this list of users is requested * @param projectKey key of the project * @param subscriberPermissionsOnProject the required permission for global and project subscribers - * * @return the list of user login along with the subscribed channels */ Multimap findSubscribedRecipientsForDispatcher(NotificationDispatcher dispatcher, String projectKey, SubscriberPermissionsOnProject subscriberPermissionsOnProject); - @Immutable - final class EmailRecipient { - private final String login; - private final String email; + record EmailRecipient(String login, String email) { public EmailRecipient(String login, String email) { this.login = requireNonNull(login, "login can't be null"); this.email = requireNonNull(email, "email can't be null"); } - public String getLogin() { - return login; - } - - public String getEmail() { - return email; - } - @Override public String toString() { return "EmailRecipient{" + "'" + login + '\'' + ":'" + email + '\'' + '}'; @@ -97,10 +84,6 @@ public interface NotificationManager { return login.equals(that.login) && email.equals(that.email); } - @Override - public int hashCode() { - return Objects.hash(login, email); - } } /** diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/notification/email/EmailNotificationChannel.java b/server/sonar-server-common/src/main/java/org/sonar/server/notification/email/EmailNotificationChannel.java index 2bc3179ae1c..5e9f60b5668 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/notification/email/EmailNotificationChannel.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/notification/email/EmailNotificationChannel.java @@ -24,7 +24,6 @@ import java.net.URL; import java.util.Objects; import java.util.Set; import javax.annotation.CheckForNull; -import javax.annotation.concurrent.Immutable; import org.apache.commons.lang.StringUtils; import org.apache.commons.mail.Email; import org.apache.commons.mail.EmailException; @@ -128,24 +127,12 @@ public class EmailNotificationChannel extends NotificationChannel { return false; } - @Immutable - public static final class EmailDeliveryRequest { - private final String recipientEmail; - private final Notification notification; - + public record EmailDeliveryRequest(String recipientEmail, Notification notification) { public EmailDeliveryRequest(String recipientEmail, Notification notification) { this.recipientEmail = requireNonNull(recipientEmail, "recipientEmail can't be null"); this.notification = requireNonNull(notification, "notification can't be null"); } - public String getRecipientEmail() { - return recipientEmail; - } - - public Notification getNotification() { - return notification; - } - @Override public boolean equals(Object o) { if (this == o) { @@ -159,11 +146,6 @@ public class EmailNotificationChannel extends NotificationChannel { Objects.equals(notification, that.notification); } - @Override - public int hashCode() { - return Objects.hash(recipientEmail, notification); - } - @Override public String toString() { return "EmailDeliveryRequest{" + "'" + recipientEmail + '\'' + " : " + notification + '}'; @@ -177,11 +159,11 @@ public class EmailNotificationChannel extends NotificationChannel { } return (int) deliveries.stream() - .filter(t -> !t.getRecipientEmail().isBlank()) + .filter(t -> !t.recipientEmail().isBlank()) .map(t -> { - EmailMessage emailMessage = format(t.getNotification()); + EmailMessage emailMessage = format(t.notification()); if (emailMessage != null) { - emailMessage.setTo(t.getRecipientEmail()); + emailMessage.setTo(t.recipientEmail()); return deliver(emailMessage); } return false; diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandler.java b/server/sonar-server-common/src/main/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandler.java index 7b479100b61..a47bb5b95ff 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandler.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandler.java @@ -82,6 +82,6 @@ public class QGChangeNotificationHandler extends EmailNotificationHandler notifications.stream() - .map(notification -> new EmailDeliveryRequest(emailRecipient.getEmail(), notification))); + .map(notification -> new EmailDeliveryRequest(emailRecipient.email(), notification))); } } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryData.java b/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryData.java index d1976434651..61f4dc47e43 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryData.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryData.java @@ -25,7 +25,6 @@ import java.util.Map; import java.util.Objects; import java.util.Optional; import java.util.Set; -import javax.annotation.CheckForNull; import javax.annotation.Nullable; import org.sonar.core.platform.EditionProvider; import org.sonar.core.platform.EditionProvider.Edition; @@ -251,62 +250,13 @@ public class TelemetryData { } - static class Database { - private final String name; - private final String version; - - Database(String name, String version) { - this.name = name; - this.version = version; - } - - public String getName() { - return name; - } - - public String getVersion() { - return version; - } + record Database(String name, String version) { } - static class Project { - private final String projectUuid; - private final String language; - private final Long loc; - private final Long lastAnalysis; - - public Project(String projectUuid, Long lastAnalysis, String language, Long loc) { - this.projectUuid = projectUuid; - this.lastAnalysis = lastAnalysis; - this.language = language; - this.loc = loc; - } - - public String getProjectUuid() { - return projectUuid; - } - - public String getLanguage() { - return language; - } - - public Long getLoc() { - return loc; - } - - public Long getLastAnalysis() { - return lastAnalysis; - } + record Project(String projectUuid, Long lastAnalysis, String language, Long loc) { } - static class ProjectStatistics { - private final String projectUuid; - private final Long branchCount; - private final Long pullRequestCount; - private final String scm; - private final String ci; - private final String devopsPlatform; - + record ProjectStatistics(String projectUuid, Long branchCount, Long pullRequestCount, String scm, String ci, String devopsPlatform) { ProjectStatistics(String projectUuid, Long branchCount, Long pullRequestCount, @Nullable String scm, @Nullable String ci, @Nullable String devopsPlatform) { this.projectUuid = projectUuid; @@ -316,33 +266,5 @@ public class TelemetryData { this.ci = ci; this.devopsPlatform = devopsPlatform; } - - public String getProjectUuid() { - return projectUuid; - } - - public Long getBranchCount() { - return branchCount; - } - - public Long getPullRequestCount() { - return pullRequestCount; - } - - @CheckForNull - public String getScm() { - return scm; - } - - @CheckForNull - public String getCi() { - return ci; - } - - @CheckForNull - public String getDevopsPlatform() { - return devopsPlatform; - } - } } diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryDataJsonWriter.java b/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryDataJsonWriter.java index fc399f1c23c..50a163f4062 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryDataJsonWriter.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/telemetry/TelemetryDataJsonWriter.java @@ -57,8 +57,8 @@ public class TelemetryDataJsonWriter { statistics.getEdition().ifPresent(e -> json.prop("edition", e.name().toLowerCase(Locale.ENGLISH))); json.name("database"); json.beginObject(); - json.prop("name", statistics.getDatabase().getName()); - json.prop("version", statistics.getDatabase().getVersion()); + json.prop("name", statistics.getDatabase().name()); + json.prop("version", statistics.getDatabase().version()); json.endObject(); json.name("plugins"); json.beginArray(); @@ -128,12 +128,12 @@ public class TelemetryDataJsonWriter { json.beginArray(); statistics.getProjects().forEach(project -> { json.beginObject(); - json.prop("projectUuid", project.getProjectUuid()); - if (project.getLastAnalysis() != null) { - json.prop("lastAnalysis", toUtc(project.getLastAnalysis())); + json.prop("projectUuid", project.projectUuid()); + if (project.lastAnalysis() != null) { + json.prop("lastAnalysis", toUtc(project.lastAnalysis())); } - json.prop(LANGUAGE_PROPERTY, project.getLanguage()); - json.prop("loc", project.getLoc()); + json.prop(LANGUAGE_PROPERTY, project.language()); + json.prop("loc", project.loc()); json.endObject(); }); json.endArray(); @@ -146,12 +146,12 @@ public class TelemetryDataJsonWriter { json.beginArray(); statistics.getProjectStatistics().forEach(project -> { json.beginObject(); - json.prop("projectUuid", project.getProjectUuid()); - json.prop("branchCount", project.getBranchCount()); - json.prop("pullRequestCount", project.getPullRequestCount()); - json.prop("scm", project.getScm()); - json.prop("ci", project.getCi()); - json.prop("devopsPlatform", project.getDevopsPlatform()); + json.prop("projectUuid", project.projectUuid()); + json.prop("branchCount", project.branchCount()); + json.prop("pullRequestCount", project.pullRequestCount()); + json.prop("scm", project.scm()); + json.prop("ci", project.ci()); + json.prop("devopsPlatform", project.devopsPlatform()); json.endObject(); }); json.endArray(); diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/CeTask.java b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/CeTask.java index 8568a6fa57d..8dd9b60ea06 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/CeTask.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/CeTask.java @@ -19,48 +19,18 @@ */ package org.sonar.server.webhook; -import java.util.Objects; - import static java.util.Objects.requireNonNull; -public final class CeTask { - private final String id; - private final Status status; - +public record CeTask(String id, Status status) { public CeTask(String id, Status status) { this.id = requireNonNull(id, "id can't be null"); this.status = requireNonNull(status, "status can't be null"); } - public String getId() { - return id; - } - - public Status getStatus() { - return status; - } - public enum Status { SUCCESS, FAILED } - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - CeTask task = (CeTask) o; - return Objects.equals(id, task.id) && - status == task.status; - } - - @Override - public int hashCode() { - return Objects.hash(id, status); - } @Override public String toString() { diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooks.java b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooks.java index 1de667f023a..50ffa197f86 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooks.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooks.java @@ -19,9 +19,7 @@ */ package org.sonar.server.webhook; -import java.util.Objects; import java.util.function.Supplier; -import javax.annotation.CheckForNull; import javax.annotation.Nullable; import org.sonar.api.ce.posttask.PostProjectAnalysisTask; import org.sonar.api.config.Configuration; @@ -51,50 +49,13 @@ public interface WebHooks { */ void sendProjectAnalysisUpdate(Analysis analysis, Supplier payloadSupplier, PostProjectAnalysisTask.LogStatistics taskLogStatistics); - final class Analysis { - private final String projectUuid; - private final String ceTaskUuid; - private final String analysisUuid; - + record Analysis(String projectUuid, String analysisUuid, String ceTaskUuid) { public Analysis(String projectUuid, @Nullable String analysisUuid, @Nullable String ceTaskUuid) { this.projectUuid = requireNonNull(projectUuid, "projectUuid can't be null"); this.analysisUuid = analysisUuid; this.ceTaskUuid = ceTaskUuid; } - public String getProjectUuid() { - return projectUuid; - } - - @CheckForNull - public String getCeTaskUuid() { - return ceTaskUuid; - } - - @CheckForNull - public String getAnalysisUuid() { - return analysisUuid; - } - - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - Analysis analysis = (Analysis) o; - return Objects.equals(projectUuid, analysis.projectUuid) && - Objects.equals(ceTaskUuid, analysis.ceTaskUuid) && - Objects.equals(analysisUuid, analysis.analysisUuid); - } - - @Override - public int hashCode() { - return Objects.hash(projectUuid, ceTaskUuid, analysisUuid); - } - @Override public String toString() { return "Analysis{" + diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooksImpl.java b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooksImpl.java index c2c3f781616..4a8239802b2 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooksImpl.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebHooksImpl.java @@ -103,8 +103,8 @@ public class WebHooksImpl implements WebHooks { private void sendProjectAnalysisUpdateImpl(Analysis analysis, Supplier payloadSupplier, @Nullable PostProjectAnalysisTask.LogStatistics taskLogStatistics) { - List webhooks = readWebHooksFrom(analysis.getProjectUuid(), taskLogStatistics) - .map(dto -> new Webhook(dto.getUuid(), analysis.getProjectUuid(), analysis.getCeTaskUuid(), analysis.getAnalysisUuid(), + List webhooks = readWebHooksFrom(analysis.projectUuid(), taskLogStatistics) + .map(dto -> new Webhook(dto.getUuid(), analysis.projectUuid(), analysis.ceTaskUuid(), analysis.analysisUuid(), dto.getName(), dto.getUrl(), dto.getSecret())) .collect(MoreCollectors.toList()); if (webhooks.isEmpty()) { @@ -117,7 +117,7 @@ public class WebHooksImpl implements WebHooks { log(delivery); deliveryStorage.persist(delivery); })); - asyncExecution.addToQueue(() -> deliveryStorage.purge(analysis.getProjectUuid())); + asyncExecution.addToQueue(() -> deliveryStorage.purge(analysis.projectUuid())); } private static void log(WebhookDelivery delivery) { diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebhookPayloadFactoryImpl.java b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebhookPayloadFactoryImpl.java index 7a7e333357b..cb48722bda6 100644 --- a/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebhookPayloadFactoryImpl.java +++ b/server/sonar-server-common/src/main/java/org/sonar/server/webhook/WebhookPayloadFactoryImpl.java @@ -103,8 +103,8 @@ public class WebhookPayloadFactoryImpl implements WebhookPayloadFactory { } private static void writeTask(JsonWriter writer, Optional ceTask) { - ceTask.ifPresent(ceTask1 -> writer.prop("taskId", ceTask1.getId())); - writer.prop(PROPERTY_STATUS, ceTask.map(CeTask::getStatus).orElse(CeTask.Status.SUCCESS).toString()); + ceTask.ifPresent(ceTask1 -> writer.prop("taskId", ceTask1.id())); + writer.prop(PROPERTY_STATUS, ceTask.map(CeTask::status).orElse(CeTask.Status.SUCCESS).toString()); } private void writeBranch(JsonWriter writer, Project project, Branch branch) { diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandlerTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandlerTest.java index 0705d421bc1..d1fbb495c9e 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandlerTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/ChangesOnMyIssueNotificationHandlerTest.java @@ -283,15 +283,15 @@ public class ChangesOnMyIssueNotificationHandlerTest { Set emailDeliveryRequests = emailDeliveryRequestSetCaptor.getValue(); assertThat(emailDeliveryRequests).hasSize(4); ListMultimap emailDeliveryRequestByEmail = emailDeliveryRequests.stream() - .collect(index(EmailDeliveryRequest::getRecipientEmail)); + .collect(index(EmailDeliveryRequest::recipientEmail)); List assignee1Requests = emailDeliveryRequestByEmail.get(emailOf(assignee1.getLogin())); assertThat(assignee1Requests) .hasSize(2) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChange) .containsOnly(userOrAnalysisChange); assertThat(assignee1Requests) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChangedIssues) .containsOnly( assignee1Issues.stream().limit(5).collect(unorderedIndex(t -> project, t -> t)), @@ -300,11 +300,11 @@ public class ChangesOnMyIssueNotificationHandlerTest { List assignee2Requests = emailDeliveryRequestByEmail.get(emailOf(assignee2.getLogin())); assertThat(assignee2Requests) .hasSize(2) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChange) .containsOnly(userOrAnalysisChange); assertThat(assignee2Requests) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChangedIssues) .containsOnly( assignee2Issues.stream().limit(6).collect(unorderedIndex(t -> project, t -> t)), @@ -376,15 +376,15 @@ public class ChangesOnMyIssueNotificationHandlerTest { Set emailDeliveryRequests = emailDeliveryRequestSetCaptor.getValue(); assertThat(emailDeliveryRequests).hasSize(3); ListMultimap emailDeliveryRequestByEmail = emailDeliveryRequests.stream() - .collect(index(EmailDeliveryRequest::getRecipientEmail)); + .collect(index(EmailDeliveryRequest::recipientEmail)); List assignee1Requests = emailDeliveryRequestByEmail.get(emailOf(assignee1.getLogin())); assertThat(assignee1Requests) .hasSize(2) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChange) .containsOnly(userOrAnalysisChange, assignee2Change1); assertThat(assignee1Requests) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChangedIssues) .containsOnly( assignee1Issues.stream().skip(4).limit(2).collect(unorderedIndex(t -> project1, t -> t)), @@ -393,11 +393,11 @@ public class ChangesOnMyIssueNotificationHandlerTest { List assignee2Requests = emailDeliveryRequestByEmail.get(emailOf(assignee2.getLogin())); assertThat(assignee2Requests) .hasSize(1) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChange) .containsOnly(userOrAnalysisChange); assertThat(assignee2Requests) - .extracting(t -> (ChangesOnMyIssuesNotification) t.getNotification()) + .extracting(t -> (ChangesOnMyIssuesNotification) t.notification()) .extracting(ChangesOnMyIssuesNotification::getChangedIssues) .containsOnly(assignee2Issues.stream().skip(7).collect(unorderedIndex(t -> project2, t -> t))); } diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandlerTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandlerTest.java index 2b3ef1af0eb..9e4e19d3e22 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandlerTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/FPOrWontFixNotificationHandlerTest.java @@ -337,7 +337,7 @@ public class FPOrWontFixNotificationHandlerTest { verify(emailNotificationChannel).deliverAll(captor.capture()); verifyNoMoreInteractions(emailNotificationChannel); ListMultimap requestsByRecipientEmail = captor.getValue().stream() - .collect(index(EmailDeliveryRequest::getRecipientEmail)); + .collect(index(EmailDeliveryRequest::recipientEmail)); assertThat(requestsByRecipientEmail.get(emailOf(subscriber1.getLogin()))) .containsOnly( Stream.of( @@ -418,7 +418,7 @@ public class FPOrWontFixNotificationHandlerTest { verify(emailNotificationChannel).deliverAll(captor.capture()); verifyNoMoreInteractions(emailNotificationChannel); ListMultimap requestsByRecipientEmail = captor.getValue().stream() - .collect(index(EmailDeliveryRequest::getRecipientEmail)); + .collect(index(EmailDeliveryRequest::recipientEmail)); assertThat(requestsByRecipientEmail.get(emailOf(subscriber1.getLogin()))) .containsOnly( new EmailDeliveryRequest(emailOf(subscriber1.getLogin()), new FPOrWontFixNotification( diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/NewIssuesNotificationHandlerTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/NewIssuesNotificationHandlerTest.java index 9b7eb191c3c..1650a77004a 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/NewIssuesNotificationHandlerTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/issue/notification/NewIssuesNotificationHandlerTest.java @@ -159,7 +159,7 @@ public class NewIssuesNotificationHandlerTest { .map(login -> new EmailRecipient(login, emailOf(login))) .collect(toSet()); Set expectedRequests = emailRecipients.stream() - .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif))) + .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif))) .collect(toSet()); when(emailNotificationChannel.isActivated()).thenReturn(true); when(notificationManager.findSubscribedEmailRecipients(NEW_ISSUES_DISPATCHER_KEY, projectKey, ALL_MUST_HAVE_ROLE_USER)) @@ -200,9 +200,9 @@ public class NewIssuesNotificationHandlerTest { .thenReturn(emailRecipients2); Set expectedRequests = Stream.concat( emailRecipients1.stream() - .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif))), + .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif))), emailRecipients2.stream() - .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.getEmail(), notif)))) + .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailDeliveryRequest(emailRecipient.email(), notif)))) .collect(toSet()); int deliver = underTest.deliver(Stream.concat(notifications1.stream(), notifications2.stream()).collect(toSet())); diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandlerTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandlerTest.java index eeb04668a3e..941dd76ba43 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandlerTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/qualitygate/notification/QGChangeNotificationHandlerTest.java @@ -157,7 +157,7 @@ public class QGChangeNotificationHandlerTest { .map(login -> new NotificationManager.EmailRecipient(login, emailOf(login))) .collect(toSet()); Set expectedRequests = emailRecipients.stream() - .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.getEmail(), notif))) + .flatMap(emailRecipient -> withProjectKey.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.email(), notif))) .collect(toSet()); when(emailNotificationChannel.isActivated()).thenReturn(true); when(notificationManager.findSubscribedEmailRecipients(QG_CHANGE_DISPATCHER_KEY, projectKey, ALL_MUST_HAVE_ROLE_USER)) @@ -198,9 +198,9 @@ public class QGChangeNotificationHandlerTest { .thenReturn(emailRecipients2); Set expectedRequests = Stream.concat( emailRecipients1.stream() - .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.getEmail(), notif))), + .flatMap(emailRecipient -> notifications1.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.email(), notif))), emailRecipients2.stream() - .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.getEmail(), notif)))) + .flatMap(emailRecipient -> notifications2.stream().map(notif -> new EmailNotificationChannel.EmailDeliveryRequest(emailRecipient.email(), notif)))) .collect(toSet()); int deliver = underTest.deliver(Stream.concat(notifications1.stream(), notifications2.stream()).collect(toSet())); diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/webhook/CeTaskTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/webhook/CeTaskTest.java index a70211066ce..533706cc7a3 100644 --- a/server/sonar-server-common/src/test/java/org/sonar/server/webhook/CeTaskTest.java +++ b/server/sonar-server-common/src/test/java/org/sonar/server/webhook/CeTaskTest.java @@ -44,7 +44,7 @@ public class CeTaskTest { @Test public void verify_getters() { - assertThat(underTest.getId()).isEqualTo("A"); - assertThat(underTest.getStatus()).isEqualTo(CeTask.Status.SUCCESS); + assertThat(underTest.id()).isEqualTo("A"); + assertThat(underTest.status()).isEqualTo(CeTask.Status.SUCCESS); } } diff --git a/server/sonar-webserver-api/src/main/java/org/sonar/server/project/RekeyedProject.java b/server/sonar-webserver-api/src/main/java/org/sonar/server/project/RekeyedProject.java index 90d261f601a..184657c7f7d 100644 --- a/server/sonar-webserver-api/src/main/java/org/sonar/server/project/RekeyedProject.java +++ b/server/sonar-webserver-api/src/main/java/org/sonar/server/project/RekeyedProject.java @@ -19,45 +19,14 @@ */ package org.sonar.server.project; -import java.util.Objects; - import static com.google.common.base.Preconditions.checkNotNull; -public final class RekeyedProject { - private final Project project; - private final String previousKey; - +public record RekeyedProject(Project project, String previousKey) { public RekeyedProject(Project project, String previousKey) { this.project = checkNotNull(project, "project can't be null"); this.previousKey = checkNotNull(previousKey, "previousKey can't be null"); } - public Project getProject() { - return project; - } - - public String getPreviousKey() { - return previousKey; - } - - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - RekeyedProject that = (RekeyedProject) o; - return Objects.equals(project, that.project) && - Objects.equals(previousKey, that.previousKey); - } - - @Override - public int hashCode() { - return Objects.hash(project, previousKey); - } - @Override public String toString() { return "RekeyedProject{" + diff --git a/server/sonar-webserver-api/src/test/java/org/sonar/server/project/RekeyedProjectTest.java b/server/sonar-webserver-api/src/test/java/org/sonar/server/project/RekeyedProjectTest.java index 7bc8bd2b628..e175bed9b10 100644 --- a/server/sonar-webserver-api/src/test/java/org/sonar/server/project/RekeyedProjectTest.java +++ b/server/sonar-webserver-api/src/test/java/org/sonar/server/project/RekeyedProjectTest.java @@ -49,8 +49,8 @@ public class RekeyedProjectTest { String previousKey = randomAlphanumeric(6); RekeyedProject underTest = new RekeyedProject(project, previousKey); - assertThat(underTest.getProject()).isSameAs(project); - assertThat(underTest.getPreviousKey()).isEqualTo(previousKey); + assertThat(underTest.project()).isSameAs(project); + assertThat(underTest.previousKey()).isEqualTo(previousKey); } @Test diff --git a/server/sonar-webserver-auth/src/main/java/org/sonar/server/user/UserSession.java b/server/sonar-webserver-auth/src/main/java/org/sonar/server/user/UserSession.java index c9f49ffc7b5..bebb0229394 100644 --- a/server/sonar-webserver-auth/src/main/java/org/sonar/server/user/UserSession.java +++ b/server/sonar-webserver-auth/src/main/java/org/sonar/server/user/UserSession.java @@ -25,7 +25,6 @@ import java.util.List; import java.util.Objects; import java.util.Optional; import javax.annotation.CheckForNull; -import javax.annotation.concurrent.Immutable; import org.sonar.db.component.ComponentDto; import org.sonar.db.permission.GlobalPermission; import org.sonar.db.project.ProjectDto; @@ -96,23 +95,12 @@ public interface UserSession { */ Optional getIdentityProvider(); - @Immutable final class ExternalIdentity { - private final String id; - private final String login; - + record ExternalIdentity(String id, String login) { public ExternalIdentity(String id, String login) { this.id = requireNonNull(id, "id can't be null"); this.login = requireNonNull(login, "login can't be null"); } - public String getId() { - return id; - } - - public String getLogin() { - return login; - } - @Override public String toString() { return "ExternalIdentity{" + @@ -121,22 +109,6 @@ public interface UserSession { '}'; } - @Override - public boolean equals(Object o) { - if (this == o) { - return true; - } - if (o == null || getClass() != o.getClass()) { - return false; - } - ExternalIdentity that = (ExternalIdentity) o; - return Objects.equals(id, that.id) && Objects.equals(login, that.login); - } - - @Override - public int hashCode() { - return Objects.hash(id, login); - } } /** @@ -156,7 +128,6 @@ public interface UserSession { /** * Returns {@code true} if the permission is granted, otherwise {@code false}. - * */ boolean hasPermission(GlobalPermission permission); @@ -172,7 +143,7 @@ public interface UserSession { * * If the component does not exist, then returns {@code false}. * - * @param component non-null component. + * @param component non-null component. * @param permission project permission as defined by {@link org.sonar.server.permission.PermissionService} */ boolean hasComponentPermission(String permission, ComponentDto component); diff --git a/server/sonar-webserver-core/src/test/java/org/sonar/server/telemetry/TelemetryDataLoaderImplTest.java b/server/sonar-webserver-core/src/test/java/org/sonar/server/telemetry/TelemetryDataLoaderImplTest.java index 2622b60178e..92c5bb10da6 100644 --- a/server/sonar-webserver-core/src/test/java/org/sonar/server/telemetry/TelemetryDataLoaderImplTest.java +++ b/server/sonar-webserver-core/src/test/java/org/sonar/server/telemetry/TelemetryDataLoaderImplTest.java @@ -169,7 +169,7 @@ public class TelemetryDataLoaderImplTest { tuple(activeUsers.get(2).getUuid(), lastConnectionDate, activeUsers.get(2).getLastSonarlintConnectionDate(), true), tuple(inactiveUser.getUuid(), null, inactiveUser.getLastSonarlintConnectionDate(), false)); assertThat(data.getProjects()) - .extracting(TelemetryData.Project::getProjectUuid, TelemetryData.Project::getLanguage, TelemetryData.Project::getLoc, TelemetryData.Project::getLastAnalysis) + .extracting(TelemetryData.Project::projectUuid, TelemetryData.Project::language, TelemetryData.Project::loc, TelemetryData.Project::lastAnalysis) .containsExactlyInAnyOrder( tuple(project1.uuid(), "java", 70L, analysisDate), tuple(project1.uuid(), "js", 30L, analysisDate), @@ -177,8 +177,8 @@ public class TelemetryDataLoaderImplTest { tuple(project2.uuid(), "java", 180L, analysisDate), tuple(project2.uuid(), "js", 20L, analysisDate)); assertThat(data.getProjectStatistics()) - .extracting(TelemetryData.ProjectStatistics::getBranchCount, TelemetryData.ProjectStatistics::getPullRequestCount, - TelemetryData.ProjectStatistics::getScm, TelemetryData.ProjectStatistics::getCi, TelemetryData.ProjectStatistics::getDevopsPlatform) + .extracting(TelemetryData.ProjectStatistics::branchCount, TelemetryData.ProjectStatistics::pullRequestCount, + TelemetryData.ProjectStatistics::scm, TelemetryData.ProjectStatistics::ci, TelemetryData.ProjectStatistics::devopsPlatform) .containsExactlyInAnyOrder( tuple(1L, 0L, "scm-1", "ci-1", "azure_devops_cloud"), tuple(1L, 0L, "scm-2", "ci-2", "github_cloud")); @@ -198,8 +198,8 @@ public class TelemetryDataLoaderImplTest { private void assertDatabaseMetadata(TelemetryData.Database database) { try (DbSession dbSession = db.getDbClient().openSession(false)) { DatabaseMetaData metadata = dbSession.getConnection().getMetaData(); - assertThat(database.getName()).isEqualTo("H2"); - assertThat(database.getVersion()).isEqualTo(metadata.getDatabaseProductVersion()); + assertThat(database.name()).isEqualTo("H2"); + assertThat(database.version()).isEqualTo(metadata.getDatabaseProductVersion()); } catch (SQLException e) { throw new RuntimeException(e); } @@ -233,14 +233,14 @@ public class TelemetryDataLoaderImplTest { TelemetryData data = communityUnderTest.load(); - assertThat(data.getProjects()).extracting(TelemetryData.Project::getProjectUuid, TelemetryData.Project::getLanguage, TelemetryData.Project::getLoc) + assertThat(data.getProjects()).extracting(TelemetryData.Project::projectUuid, TelemetryData.Project::language, TelemetryData.Project::loc) .containsExactlyInAnyOrder( tuple(project.uuid(), "java", 100L), tuple(project.uuid(), "js", 50L), tuple(project.uuid(), "kotlin", 30L)); assertThat(data.getProjectStatistics()) - .extracting(TelemetryData.ProjectStatistics::getBranchCount, TelemetryData.ProjectStatistics::getPullRequestCount, - TelemetryData.ProjectStatistics::getScm, TelemetryData.ProjectStatistics::getCi) + .extracting(TelemetryData.ProjectStatistics::branchCount, TelemetryData.ProjectStatistics::pullRequestCount, + TelemetryData.ProjectStatistics::scm, TelemetryData.ProjectStatistics::ci) .containsExactlyInAnyOrder( tuple(2L, 0L, "undetected", "undetected")); } @@ -378,7 +378,7 @@ public class TelemetryDataLoaderImplTest { db.components().insertPublicProject(); TelemetryData data = communityUnderTest.load(); assertThat(data.getProjectStatistics()) - .extracting(TelemetryData.ProjectStatistics::getDevopsPlatform, TelemetryData.ProjectStatistics::getScm, TelemetryData.ProjectStatistics::getCi) + .extracting(TelemetryData.ProjectStatistics::devopsPlatform, TelemetryData.ProjectStatistics::scm, TelemetryData.ProjectStatistics::ci) .containsExactlyInAnyOrder(tuple("undetected", "undetected", "undetected")); } diff --git a/server/sonar-webserver-es/src/main/java/org/sonar/server/es/RecoveryIndexer.java b/server/sonar-webserver-es/src/main/java/org/sonar/server/es/RecoveryIndexer.java index 3e0d918eff8..a1570886c9c 100644 --- a/server/sonar-webserver-es/src/main/java/org/sonar/server/es/RecoveryIndexer.java +++ b/server/sonar-webserver-es/src/main/java/org/sonar/server/es/RecoveryIndexer.java @@ -78,22 +78,7 @@ public class RecoveryIndexer implements Startable { this.loopLimit = getSetting(PROPERTY_LOOP_LIMIT, DEFAULT_LOOP_LIMIT); } - private static final class Indexer { - private final IndexType indexType; - private final ResilientIndexer delegate; - - private Indexer(IndexType indexType, ResilientIndexer delegate) { - this.indexType = indexType; - this.delegate = delegate; - } - - public IndexType getIndexType() { - return indexType; - } - - public ResilientIndexer getDelegate() { - return delegate; - } + private record Indexer(IndexType indexType, ResilientIndexer delegate) { } @Override diff --git a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/QProfileRestoreSummary.java b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/QProfileRestoreSummary.java index e7e8ba4a9f9..02639d59162 100644 --- a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/QProfileRestoreSummary.java +++ b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/QProfileRestoreSummary.java @@ -23,20 +23,9 @@ import org.sonar.db.qualityprofile.QProfileDto; import static java.util.Objects.requireNonNull; -public final class QProfileRestoreSummary { - private final QProfileDto profile; - private final BulkChangeResult ruleChanges; - +public record QProfileRestoreSummary(QProfileDto profile, BulkChangeResult ruleChanges) { public QProfileRestoreSummary(QProfileDto profile, BulkChangeResult ruleChanges) { this.profile = requireNonNull(profile); this.ruleChanges = requireNonNull(ruleChanges); } - - public QProfileDto getProfile() { - return profile; - } - - public BulkChangeResult getRuleChanges() { - return ruleChanges; - } } diff --git a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/DescendantProfilesSupplier.java b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/DescendantProfilesSupplier.java index 1b2948c7d09..e000b5d5289 100644 --- a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/DescendantProfilesSupplier.java +++ b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/DescendantProfilesSupplier.java @@ -29,27 +29,6 @@ public interface DescendantProfilesSupplier { Result get(Collection profiles, Collection ruleUuids); - final class Result { - private final Collection profiles; - private final Collection activeRules; - private final Collection activeRuleParams; - - public Result(Collection profiles, Collection activeRules, Collection activeRuleParams) { - this.profiles = profiles; - this.activeRules = activeRules; - this.activeRuleParams = activeRuleParams; - } - - public Collection getProfiles() { - return profiles; - } - - public Collection getActiveRules() { - return activeRules; - } - - public Collection getActiveRuleParams() { - return activeRuleParams; - } + record Result(Collection profiles, Collection activeRules, Collection activeRuleParams) { } } diff --git a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/RuleActivationContext.java b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/RuleActivationContext.java index ab1040443ec..6ca883a3217 100644 --- a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/RuleActivationContext.java +++ b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/builtin/RuleActivationContext.java @@ -197,8 +197,8 @@ public class RuleActivationContext { .filter(p -> p.getRulesProfileUuid().equals(baseRulesProfile.getUuid())) .collect(toArrayList(profilesByUuid.size())); DescendantProfilesSupplier.Result result = descendantProfilesSupplier.get(baseProfiles, rulesByUuid.keySet()); - register(result.getProfiles()); - register(result.getActiveRules(), result.getActiveRuleParams()); + register(result.profiles()); + register(result.activeRules(), result.activeRuleParams()); descendantsLoaded = true; } diff --git a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/ws/RestoreAction.java b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/ws/RestoreAction.java index 726c768e2ad..bd24147b5c7 100644 --- a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/ws/RestoreAction.java +++ b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/qualityprofile/ws/RestoreAction.java @@ -92,7 +92,7 @@ public class RestoreAction implements QProfileWsAction { } private void writeResponse(JsonWriter json, QProfileRestoreSummary summary) { - QProfileDto profile = summary.getProfile(); + QProfileDto profile = summary.profile(); String languageKey = profile.getLanguage(); Language language = languages.get(languageKey); @@ -108,7 +108,7 @@ public class RestoreAction implements QProfileWsAction { } jsonProfile.endObject(); - BulkChangeResult ruleChanges = summary.getRuleChanges(); + BulkChangeResult ruleChanges = summary.ruleChanges(); json.prop("ruleSuccesses", ruleChanges.countSucceeded()); json.prop("ruleFailures", ruleChanges.countFailed()); json.endObject().close(); diff --git a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/QProfileBackuperImplTest.java b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/QProfileBackuperImplTest.java index fb2d7bb4b90..7af587e0c9f 100644 --- a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/QProfileBackuperImplTest.java +++ b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/QProfileBackuperImplTest.java @@ -208,10 +208,10 @@ public class QProfileBackuperImplTest { QProfileRestoreSummary summary = underTest.restore(db.getSession(), backup, (String) null); - assertThat(summary.getProfile().getName()).isEqualTo("foo"); - assertThat(summary.getProfile().getLanguage()).isEqualTo("js"); + assertThat(summary.profile().getName()).isEqualTo("foo"); + assertThat(summary.profile().getLanguage()).isEqualTo("js"); - assertThat(reset.calledProfile.getKee()).isEqualTo(summary.getProfile().getKee()); + assertThat(reset.calledProfile.getKee()).isEqualTo(summary.profile().getKee()); assertThat(reset.calledActivations).isEmpty(); } @@ -287,10 +287,10 @@ public class QProfileBackuperImplTest { QProfileRestoreSummary summary = underTest.restore(db.getSession(), backup, "bar"); - assertThat(summary.getProfile().getName()).isEqualTo("bar"); - assertThat(summary.getProfile().getLanguage()).isEqualTo("js"); + assertThat(summary.profile().getName()).isEqualTo("bar"); + assertThat(summary.profile().getLanguage()).isEqualTo("js"); - assertThat(reset.calledProfile.getKee()).isEqualTo(summary.getProfile().getKee()); + assertThat(reset.calledProfile.getKee()).isEqualTo(summary.profile().getKee()); assertThat(reset.calledActivations).isEmpty(); } diff --git a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/ws/RestoreActionTest.java b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/ws/RestoreActionTest.java index dcc4ea96c35..26047cc0848 100644 --- a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/ws/RestoreActionTest.java +++ b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/qualityprofile/ws/RestoreActionTest.java @@ -82,7 +82,7 @@ public class RestoreActionTest { TestResponse response = restore(""); assertThat(backuper.restoredBackup).isEqualTo(""); - assertThat(backuper.restoredSummary.getProfile().getName()).isEqualTo("the-name-in-backup"); + assertThat(backuper.restoredSummary.profile().getName()).isEqualTo("the-name-in-backup"); JsonAssert.assertJson(response.getInput()).isSimilarTo("{" + " \"profile\": {" + " \"name\": \"the-name-in-backup\"," + -- 2.39.5