aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java
diff options
context:
space:
mode:
Diffstat (limited to 'sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java')
-rw-r--r--sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java195
1 files changed, 195 insertions, 0 deletions
diff --git a/sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java b/sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java
new file mode 100644
index 00000000000..bef5eb6c57c
--- /dev/null
+++ b/sonar-scanner-engine/src/test/java/org/sonar/scm/svn/ChangedLinesComputerTest.java
@@ -0,0 +1,195 @@
+/*
+ * SonarQube
+ * Copyright (C) 2009-2020 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.scm.svn;
+
+import java.io.IOException;
+import java.io.OutputStreamWriter;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Map;
+import java.util.Set;
+import org.junit.Test;
+
+import static java.util.Collections.singleton;
+import static org.assertj.core.api.Assertions.assertThat;
+
+public class ChangedLinesComputerTest {
+
+ private final Path rootBaseDir = Paths.get("/foo");
+ private final ChangedLinesComputer underTest = new ChangedLinesComputer(rootBaseDir, new HashSet<>(Arrays.asList(
+ rootBaseDir.resolve("sample1"),
+ rootBaseDir.resolve("sample2"),
+ rootBaseDir.resolve("sample3"),
+ rootBaseDir.resolve("sample4"))));
+
+ @Test
+ public void do_not_count_deleted_line() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -1 +0,0 @@\n"
+ + "-deleted line\n";
+
+ printDiff(example);
+ assertThat(underTest.changedLines()).isEmpty();
+ }
+
+ @Test
+ public void count_single_added_line() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -0,0 +1 @@\n"
+ + "+added line\n";
+
+ printDiff(example);
+ assertThat(underTest.changedLines()).isEqualTo(Collections.singletonMap(rootBaseDir.resolve("sample1"), singleton(1)));
+ }
+
+ @Test
+ public void count_multiple_added_lines() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -1 +1,3 @@\n"
+ + " same line\n"
+ + "+added line 1\n"
+ + "+added line 2\n";
+
+ printDiff(example);
+ assertThat(underTest.changedLines()).isEqualTo(Collections.singletonMap(rootBaseDir.resolve("sample1"), new HashSet<>(Arrays.asList(2, 3))));
+ }
+
+ @Test
+ public void handle_index_using_absolute_paths() throws IOException {
+ String example = "Index: /foo/sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -1 +1,3 @@\n"
+ + " same line\n"
+ + "+added line 1\n"
+ + "+added line 2\n";
+
+ printDiff(example);
+ assertThat(underTest.changedLines()).isEqualTo(Collections.singletonMap(rootBaseDir.resolve("sample1"), new HashSet<>(Arrays.asList(2, 3))));
+ }
+
+ @Test
+ public void compute_from_multiple_hunks() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- lao\t2002-02-21 23:30:39.942229878 -0800\n"
+ + "+++ tzu\t2002-02-21 23:30:50.442260588 -0800\n"
+ + "@@ -1,7 +1,6 @@\n"
+ + "-The Way that can be told of is not the eternal Way;\n"
+ + "-The name that can be named is not the eternal name.\n"
+ + " The Nameless is the origin of Heaven and Earth;\n"
+ + "-The Named is the mother of all things.\n"
+ + "+The named is the mother of all things.\n"
+ + "+\n"
+ + " Therefore let there always be non-being,\n"
+ + " so we may see their subtlety,\n"
+ + " And let there always be being,\n"
+ + "@@ -9,3 +8,6 @@\n"
+ + " The two are the same,\n"
+ + " But after they are produced,\n"
+ + " they have different names.\n"
+ + "+They both may be called deep and profound.\n"
+ + "+Deeper and more profound,\n"
+ + "+The door of all subtleties!\n";
+ printDiff(example);
+ assertThat(underTest.changedLines()).isEqualTo(Collections.singletonMap(rootBaseDir.resolve("sample1"), new HashSet<>(Arrays.asList(2, 3, 11, 12, 13))));
+ }
+
+ @Test(expected = IllegalStateException.class)
+ public void crash_on_invalid_start_line_format() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -1 +x1,3 @@\n"
+ + " same line\n"
+ + "+added line 1\n"
+ + "+added line 2\n";
+
+ printDiff(example);
+ underTest.changedLines();
+ }
+
+ @Test
+ public void parse_diff_with_multiple_files() throws IOException {
+ String example = "Index: sample1\n"
+ + "===================================================================\n"
+ + "--- a/sample1\n"
+ + "+++ b/sample1\n"
+ + "@@ -1 +0,0 @@\n"
+ + "-deleted line\n"
+ + "Index: sample2\n"
+ + "===================================================================\n"
+ + "--- a/sample2\n"
+ + "+++ b/sample2\n"
+ + "@@ -0,0 +1 @@\n"
+ + "+added line\n"
+ + "Index: sample3\n"
+ + "===================================================================\n"
+ + "--- a/sample3\n"
+ + "+++ b/sample3\n"
+ + "@@ -0,0 +1,2 @@\n"
+ + "+added line 1\n"
+ + "+added line 2\n"
+ + "Index: sample3-not-included\n"
+ + "===================================================================\n"
+ + "--- a/sample3-not-included\n"
+ + "+++ b/sample3-not-included\n"
+ + "@@ -0,0 +1,2 @@\n"
+ + "+added line 1\n"
+ + "+added line 2\n"
+ + "Index: sample4\n"
+ + "===================================================================\n"
+ + "--- a/sample4\n"
+ + "+++ b/sample4\n"
+ + "@@ -1 +1,3 @@\n"
+ + " same line\n"
+ + "+added line 1\n"
+ + "+added line 2\n";
+
+ printDiff(example);
+ Map<Path, Set<Integer>> expected = new HashMap<>();
+ expected.put(rootBaseDir.resolve("sample2"), Collections.singleton(1));
+ expected.put(rootBaseDir.resolve("sample3"), new HashSet<>(Arrays.asList(1, 2)));
+ expected.put(rootBaseDir.resolve("sample4"), new HashSet<>(Arrays.asList(2, 3)));
+
+ assertThat(underTest.changedLines()).isEqualTo(expected);
+ }
+
+ private void printDiff(String unifiedDiff) throws IOException {
+ try (OutputStreamWriter writer = new OutputStreamWriter(underTest.receiver())) {
+ writer.write(unifiedDiff);
+ }
+ }
+}