aboutsummaryrefslogtreecommitdiffstats
path: root/server
diff options
context:
space:
mode:
authorDuarte Meneses <duarte.meneses@sonarsource.com>2017-08-09 15:30:45 +0200
committerJanos Gyerik <janos.gyerik@sonarsource.com>2017-09-12 10:55:10 +0200
commitdd47b17bb4259ce422e3a643d07178f21185a406 (patch)
treed8865ba03ead2447c002cc3efd42b48fcdbb731f /server
parent47310c7a134fe15a07edec88c16a317d03246896 (diff)
downloadsonarqube-dd47b17bb4259ce422e3a643d07178f21185a406.tar.gz
sonarqube-dd47b17bb4259ce422e3a643d07178f21185a406.zip
SONAR-9674 Load branch information from scanner report
Diffstat (limited to 'server')
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/analysis/Branch.java2
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/DefaultBranchImpl.java6
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/TreeRootHolder.java1
-rw-r--r--server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java2
-rw-r--r--server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java2
5 files changed, 9 insertions, 4 deletions
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/analysis/Branch.java b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/analysis/Branch.java
index 641bea77437..352dfa1ee18 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/analysis/Branch.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/analysis/Branch.java
@@ -47,7 +47,7 @@ public interface Branch extends ComponentKeyGenerator {
* For short living branches, indicates the branch from which it was forked.
* It will be empty for other types of branches.
*/
- Optional<String> getMergeBranchName();
+ Optional<String> getMergeBranchUuid();
/**
* Whether the cross-project duplication tracker must be enabled
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/DefaultBranchImpl.java b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/DefaultBranchImpl.java
index bb340eabe44..c6db2d07aa8 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/DefaultBranchImpl.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/DefaultBranchImpl.java
@@ -42,6 +42,10 @@ public class DefaultBranchImpl implements Branch {
@Nullable
private final String branchName;
+ public DefaultBranchImpl() {
+ this(null);
+ }
+
public DefaultBranchImpl(@Nullable String name) {
this.branchName = name;
if (name != null && !ComponentKeys.isValidBranch(name)) {
@@ -61,7 +65,7 @@ public class DefaultBranchImpl implements Branch {
}
@Override
- public Optional<String> getMergeBranchName() {
+ public Optional<String> getMergeBranchUuid() {
return Optional.empty();
}
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/TreeRootHolder.java b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/TreeRootHolder.java
index 24c57476a2d..f9ae8404eff 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/TreeRootHolder.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/task/projectanalysis/component/TreeRootHolder.java
@@ -25,6 +25,7 @@ package org.sonar.server.computation.task.projectanalysis.component;
public interface TreeRootHolder {
/**
* The root of the tree, for example the project or the portfolio.
+ * With branches, it will refer to the root component of the branch.
*
* @throws IllegalStateException if the holder is empty (ie. there is no root yet)
*/
diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java
index 8b721ea8a73..4e96fd24484 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java
@@ -249,7 +249,7 @@ public class PostProjectAnalysisTasksExecutorTest {
}
@Override
- public Optional<String> getMergeBranchName() {
+ public Optional<String> getMergeBranchUuid() {
return Optional.empty();
}
diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java
index a0fd8666611..a0ab0a2933a 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/step/ReportPersistComponentsStepTest.java
@@ -922,7 +922,7 @@ public class ReportPersistComponentsStepTest extends BaseStepTest {
}
@Override
- public java.util.Optional<String> getMergeBranchName() {
+ public java.util.Optional<String> getMergeBranchUuid() {
return java.util.Optional.empty();
}