]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-7509 Revert to not check empty ranges to not break Java plugin 1008/head
authorJulien HENRY <julien.henry@sonarsource.com>
Mon, 23 May 2016 12:45:23 +0000 (14:45 +0200)
committerJulien HENRY <julien.henry@sonarsource.com>
Mon, 23 May 2016 14:36:00 +0000 (16:36 +0200)
sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultInputFile.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlighting.java
sonar-plugin-api/src/test/java/org/sonar/api/batch/sensor/highlighting/internal/DefaultHighlightingTest.java

index 98fb14864c259eb4c5bcb0263791dd0d0354242a..2d0c593a5defe79b179514410ef0e93dfe39ad05 100644 (file)
@@ -238,7 +238,6 @@ public class DefaultInputFile extends DefaultInputComponent implements InputFile
   public TextRange newRange(int startLine, int startLineOffset, int endLine, int endLineOffset) {
     TextPointer start = newPointer(startLine, startLineOffset);
     TextPointer end = newPointer(endLine, endLineOffset);
-    Preconditions.checkArgument(start.compareTo(end) < 0, "Start pointer %s should be before end pointer %s", start, end);
     return newRangeValidPointers(start, end);
   }
 
@@ -263,7 +262,6 @@ public class DefaultInputFile extends DefaultInputComponent implements InputFile
    * Create Range from global offsets. Used for backward compatibility with older API.
    */
   public TextRange newRange(int startOffset, int endOffset) {
-    Preconditions.checkArgument(startOffset < endOffset, "Start offset %s should be strictly before end offset %s", startOffset, endOffset);
     return newRangeValidPointers(newPointer(startOffset), newPointer(endOffset));
   }
 
index 6ccc541dad3bb9e621c113942007a4e3bd6d46df..34df663e54ab0cbaa0b3f9226d3450765246e0fa 100644 (file)
@@ -79,6 +79,8 @@ public class DefaultHighlighting extends DefaultStorable implements NewHighlight
     checkInputFileNotNull();
     TextRange newRange;
     try {
+      // TODO remove when SONAR-7664 is implemented
+      Preconditions.checkArgument(startOffset < endOffset, "start offset should be strictly before end offset");
       newRange = inputFile.newRange(startOffset, endOffset);
     } catch (Exception e) {
       throw new IllegalArgumentException("Unable to highlight file " + inputFile, e);
index 14cae212525ec625fe477d8a9efc29614d7047cf..bd3766962f4ca3f8d1cd12c6938cd53a5ae66bd6 100644 (file)
@@ -107,18 +107,6 @@ public class DefaultHighlightingTest {
       .save();
   }
 
-  @Test
-  public void should_prevent_start_equal_end2() {
-    throwable.expect(IllegalArgumentException.class);
-    throwable
-      .expectMessage("Unable to highlight file");
-
-    new DefaultHighlighting(mock(SensorStorage.class))
-      .onFile(INPUT_FILE)
-      .highlight(1, 10, 1, 10, KEYWORD)
-      .save();
-  }
-
   @Test
   public void should_prevent_boudaries_overlapping() {
     throwable.expect(IllegalStateException.class);