]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-10085 drop now unused QGChangeEventListeners#isEmpty
authorSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Wed, 20 Dec 2017 12:57:59 +0000 (13:57 +0100)
committerJanos Gyerik <janos.gyerik@sonarsource.com>
Fri, 5 Jan 2018 14:49:51 +0000 (15:49 +0100)
server/sonar-server/src/main/java/org/sonar/server/qualitygate/changeevent/QGChangeEventListeners.java
server/sonar-server/src/main/java/org/sonar/server/qualitygate/changeevent/QGChangeEventListenersImpl.java
server/sonar-server/src/test/java/org/sonar/server/qualitygate/changeevent/QGChangeEventListenersImplTest.java

index 1c6b434489f8b213428270fb237ee413c08db832..f3b26ec2510b0c687e654f1985da2043214cee56 100644 (file)
@@ -22,7 +22,6 @@ package org.sonar.server.qualitygate.changeevent;
 import java.util.Collection;
 
 public interface QGChangeEventListeners {
-  boolean isEmpty();
 
   void broadcastOnIssueChange(QGChangeEventFactory.IssueChangeData issueChangeData, Collection<QGChangeEvent> qgChangeEvents);
 }
index 892bd828916dd35d97e33a2ab222a391c591347a..f6047038d2e89e8172e3a160748d4254bd87818c 100644 (file)
@@ -51,11 +51,6 @@ public class QGChangeEventListenersImpl implements QGChangeEventListeners {
     this.listeners = listeners;
   }
 
-  @Override
-  public boolean isEmpty() {
-    return listeners.length == 0;
-  }
-
   @Override
   public void broadcastOnIssueChange(QGChangeEventFactory.IssueChangeData issueChangeData, Collection<QGChangeEvent> changeEvents) {
     if (listeners.length == 0 || issueChangeData.getComponents().isEmpty() || issueChangeData.getIssues().isEmpty() || changeEvents.isEmpty()) {
index 64fee2db04ede49e0957790507b2ce8a933a8b9e..f96602d71b19ed9ceebe0cc938fdafa87f87aed9 100644 (file)
@@ -57,27 +57,6 @@ public class QGChangeEventListenersImplTest {
 
   private QGChangeEventListenersImpl underTest = new QGChangeEventListenersImpl(new QGChangeEventListener[] {listener1, listener2, listener3});
 
-  @Test
-  public void isEmpty_returns_true_for_constructor_without_argument() {
-    QGChangeEventListenersImpl underTest = new QGChangeEventListenersImpl();
-
-    assertThat(underTest.isEmpty()).isTrue();
-  }
-
-  @Test
-  public void isEmpty_returns_false_for_constructor_with_one_argument() {
-    QGChangeEventListenersImpl underTest = new QGChangeEventListenersImpl(new QGChangeEventListener[] {listener2});
-
-    assertThat(underTest.isEmpty()).isFalse();
-  }
-
-  @Test
-  public void isEmpty_returns_false_for_constructor_with_multiple_arguments() {
-    QGChangeEventListenersImpl underTest = new QGChangeEventListenersImpl(new QGChangeEventListener[] {listener2, listener3});
-
-    assertThat(underTest.isEmpty()).isFalse();
-  }
-
   @Test
   public void broadcastOnIssueChange_has_no_effect_when_issueChangeData_is_empty() {
     QGChangeEventFactory.IssueChangeData issueChangeData = new QGChangeEventFactory.IssueChangeData(emptyList(), emptyList());