]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-2189 Remove the component SquidSearch
authorsimonbrandhof <simon.brandhof@gmail.com>
Mon, 21 Feb 2011 10:51:32 +0000 (11:51 +0100)
committersimonbrandhof <simon.brandhof@gmail.com>
Mon, 21 Feb 2011 10:51:32 +0000 (11:51 +0100)
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidExecutor.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidPlugin.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidSearchProxy.java [deleted file]
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidSensor.java
plugins/sonar-squid-java-plugin/src/test/java/org/sonar/plugins/squid/SquidExecutorTest.java
plugins/sonar-squid-java-plugin/src/test/java/org/sonar/plugins/squid/SquidPluginTest.java
plugins/sonar-squid-java-plugin/src/test/java/org/sonar/plugins/squid/SquidSearchProxyTest.java [deleted file]
sonar-deprecated/src/main/java/org/sonar/api/batch/SquidSearch.java [deleted file]

index a8b26ed1b2a7ba01f7991c2b7c3629f8e56afea7..01494e6ea56700e57250ea3337d22c69a120242a 100644 (file)
@@ -213,9 +213,8 @@ public final class SquidExecutor {
     return bytecodeScanned;
   }
 
-  public void initSonarProxy(SquidSearchProxy proxy) {
+  void flush() {
     squid.flush();
-    proxy.setTarget(squid);
   }
 
   public Squid getSquid() {
index 7196337aca9f35c72275e0bf07732d5b741a7b74..075cfd34a5a7e68467f1c9f03eab6d77e601d5de 100644 (file)
@@ -59,7 +59,7 @@ public class SquidPlugin implements Plugin {
   }
 
   public List getExtensions() {
-    return Arrays.asList(SquidSearchProxy.class, SquidSensor.class, SquidRuleRepository.class, JavaSourceImporter.class,
+    return Arrays.asList(SquidSensor.class, SquidRuleRepository.class, JavaSourceImporter.class,
         ClassComplexityDistributionBuilder.class, FunctionComplexityDistributionBuilder.class, ClassesDecorator.class,
         ChidamberKemererDistributionBuilder.class, FunctionsDecorator.class);
   }
diff --git a/plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidSearchProxy.java b/plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/SquidSearchProxy.java
deleted file mode 100644 (file)
index 0dad86d..0000000
+++ /dev/null
@@ -1,57 +0,0 @@
-/*
- * Sonar, open source software quality management tool.
- * Copyright (C) 2008-2011 SonarSource
- * mailto:contact AT sonarsource DOT com
- *
- * Sonar 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.
- *
- * Sonar 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 Sonar; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02
- */
-package org.sonar.plugins.squid;
-
-import java.util.Collection;
-
-import org.sonar.api.BatchExtension;
-import org.sonar.api.batch.SquidSearch;
-import org.sonar.squid.Squid;
-import org.sonar.squid.api.Query;
-import org.sonar.squid.api.SourceCode;
-
-public class SquidSearchProxy implements SquidSearch, BatchExtension {
-
-  private Squid target;
-
-  public Squid getTarget() {
-    return target;
-  }
-
-  public void setTarget(Squid target) {
-    this.target = target;
-  }
-
-  public Collection<SourceCode> search(Query... query) {
-    checkTarget();
-    return target.search(query);
-  }
-
-  public SourceCode search(String key) {
-    checkTarget();
-    return target.search(key);
-  }
-
-  private void checkTarget() {
-    if (target == null) {
-      throw new IllegalStateException("Squid service is not initialized");
-    }
-  }
-}
index 9024be2db3d105227fed9a94bbe2f9c0c7e26510..1e4da4853f377b72e87d8845a06231d67c6b2dd4 100644 (file)
@@ -41,14 +41,12 @@ import java.util.List;
 @DependedUpon(value = JavaUtils.BARRIER_AFTER_SQUID, classes = NoSonarFilter.class)
 public class SquidSensor implements Sensor {
 
-  private SquidSearchProxy proxy;
   private NoSonarFilter noSonarFilter;
   private RulesProfile profile;
   private ProjectClasspath projectClasspath;
   private ResourceCreationLock lock;
 
-  public SquidSensor(RulesProfile profile, SquidSearchProxy proxy, NoSonarFilter noSonarFilter, ProjectClasspath projectClasspath, ResourceCreationLock lock) {
-    this.proxy = proxy;
+  public SquidSensor(RulesProfile profile, NoSonarFilter noSonarFilter, ProjectClasspath projectClasspath, ResourceCreationLock lock) {
     this.noSonarFilter = noSonarFilter;
     this.profile = profile;
     this.projectClasspath = projectClasspath;
@@ -79,7 +77,7 @@ public class SquidSensor implements Sensor {
     SquidExecutor squidExecutor = new SquidExecutor(analyzePropertyAccessors, fieldNamesToExcludeFromLcom4Computation, factory, charset);
     squidExecutor.scan(getMainSourceFiles(project), getMainBytecodeFiles(project));
     squidExecutor.save(project, context, noSonarFilter);
-    squidExecutor.initSonarProxy(proxy);
+    squidExecutor.flush();
   }
 
   private void browseTestSources(Project project, SensorContext context) {
index 0238de92434b9028a63e0306e7081fdee88f826a..dc0b363b0b529dc212c37b1a48e5944f12340103 100644 (file)
@@ -115,15 +115,6 @@ public class SquidExecutorTest {
     assertTrue(SquidExecutor.hasBytecode(Arrays.asList(new File("test-resources/commons-collections-3.2.1/bin"))));
   }
 
-  @Test
-  public void flushSquidAfterUsage() {
-    Squid squid = mock(Squid.class);
-    SquidExecutor executor = new SquidExecutor(squid);
-    executor.initSonarProxy(new SquidSearchProxy());
-
-    verify(squid).flush();
-  }
-
   private CheckFactory createCheckFactory() {
     RulesProfile profile = RulesProfile.create();
     CheckFactory checkFactory = AnnotationCheckFactory.create(profile, "repo", Collections.<Class> emptyList());
index 0357049ac113ca16f92b6d4ea00bfa203caa941f..52db97389cf1cd1471c43980559245fbbe59419e 100644 (file)
@@ -31,6 +31,6 @@ public class SquidPluginTest {
   @Test
   public void coverageForFun() {
     assertThat(new SquidPlugin().getKey(), not(nullValue()));
-    assertThat(new SquidPlugin().getExtensions().size(), is(9));
+    assertThat(new SquidPlugin().getExtensions().size(), is(8));
   }
 }
diff --git a/plugins/sonar-squid-java-plugin/src/test/java/org/sonar/plugins/squid/SquidSearchProxyTest.java b/plugins/sonar-squid-java-plugin/src/test/java/org/sonar/plugins/squid/SquidSearchProxyTest.java
deleted file mode 100644 (file)
index 016bb31..0000000
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * Sonar, open source software quality management tool.
- * Copyright (C) 2008-2011 SonarSource
- * mailto:contact AT sonarsource DOT com
- *
- * Sonar 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.
- *
- * Sonar 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 Sonar; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02
- */
-package org.sonar.plugins.squid;
-
-import org.junit.Test;
-import org.sonar.squid.Squid;
-import org.sonar.squid.api.SourceFile;
-import org.sonar.squid.indexer.QueryByType;
-
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.verify;
-
-public class SquidSearchProxyTest {
-
-  @Test
-  public void isProxy() {
-    Squid squid = mock(Squid.class);
-    SquidSearchProxy proxy = new SquidSearchProxy();
-    proxy.setTarget(squid);
-
-    proxy.search("foo");
-    verify(squid).search("foo");
-
-    QueryByType query = new QueryByType(SourceFile.class);
-    proxy.search(query);
-    verify(squid).search(query);
-  }
-}
diff --git a/sonar-deprecated/src/main/java/org/sonar/api/batch/SquidSearch.java b/sonar-deprecated/src/main/java/org/sonar/api/batch/SquidSearch.java
deleted file mode 100644 (file)
index 71c8718..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * Sonar, open source software quality management tool.
- * Copyright (C) 2008-2011 SonarSource
- * mailto:contact AT sonarsource DOT com
- *
- * Sonar 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.
- *
- * Sonar 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 Sonar; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02
- */
-package org.sonar.api.batch;
-
-import java.util.Collection;
-
-import org.sonar.api.BatchExtension;
-import org.sonar.squid.api.Query;
-import org.sonar.squid.api.SourceCode;
-
-/**
- * The extension point to access the Squid data tree
- *
- * @since 1.11
- * @deprecated since 2.6. Data on classes/methods are now injected into sonar index. There's no need to use
- * this component anymore.
- */
-@Deprecated
-public interface SquidSearch extends BatchExtension {
-  /**
-   * Returns a list of SourceCode objects base a set of queries given
-   *
-   * @param query the set of query
-   * @return SourceCode objects
-   */
-  Collection<SourceCode> search(Query... query);
-
-  /**
-   * Returns a SourceObject given its key
-   *
-   * @param key the key
-   * @return SourceCode object
-   */
-  SourceCode search(String key);
-}