removed singleton pattern from service loaders

Former-commit-id: eb822f5f15ca7bd87f9b3bff39467f77b64f27b7

dependency-check-core/src/main/java/org/owasp/dependencycheck/Engine.java

@@ -107,7 +107,7 @@ public class Engine {
             analyzers.put(phase, new ArrayList<Analyzer>());
         }
 
-        final AnalyzerService service = AnalyzerService.getInstance();
+        final AnalyzerService service = new AnalyzerService();
         final Iterator<Analyzer> iterator = service.getAnalyzers();
         while (iterator.hasNext()) {
             final Analyzer a = iterator.next();
@@ -410,7 +410,7 @@ public class Engine {
      * Cycles through the cached web data sources and calls update on all of them.
      */
     private void doUpdates() {
-        final UpdateService service = UpdateService.getInstance();
+        final UpdateService service = new UpdateService();
         final Iterator<CachedWebDataSource> iterator = service.getDataSources();
         while (iterator.hasNext()) {
             final CachedWebDataSource source = iterator.next();

dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/AnalyzerService.java

@@ -21,15 +21,13 @@ import java.util.Iterator;
 import java.util.ServiceLoader;
 
 /**
+ * The Analyzer Service Loader. This class loads all services that implement
+ * org.owasp.dependencycheck.analyzer.Analyzer.
  *
  * @author Jeremy Long <jeremy.long@owasp.org>
  */
-public final class AnalyzerService {
+public class AnalyzerService {
 
-    /**
-     * The analyzer service singleton.
-     */
-    private static AnalyzerService service;
     /**
      * The service loader for analyzers.
      */
@@ -38,22 +36,10 @@ public final class AnalyzerService {
     /**
      * Creates a new instance of AnalyzerService.
      */
-    private AnalyzerService() {
+    public AnalyzerService() {
         loader = ServiceLoader.load(Analyzer.class);
     }
 
-    /**
-     * Retrieve the singleton instance of AnalyzerService.
-     *
-     * @return a singleton AnalyzerService.
-     */
-    public static synchronized AnalyzerService getInstance() {
-        if (service == null) {
-            service = new AnalyzerService();
-        }
-        return service;
-    }
-
     /**
      * Returns an Iterator for all instances of the Analyzer interface.
      *

dependency-check-core/src/main/java/org/owasp/dependencycheck/data/update/UpdateService.java

@@ -21,39 +21,25 @@ import java.util.Iterator;
 import java.util.ServiceLoader;
 
 /**
+ * The CachedWebDataSource Service Loader. This class loads all services that implement
+ * org.owasp.dependencycheck.data.update.CachedWebDataSource.
  *
  * @author Jeremy Long <jeremy.long@owasp.org>
  */
-public final class UpdateService {
+public class UpdateService {
 
-    /**
-     * the singleton reference to the service.
-     */
-    private static UpdateService service;
     /**
      * the service loader for CachedWebDataSource.
      */
     private final ServiceLoader<CachedWebDataSource> loader;
 
     /**
-     * Creates a new instance of UpdateService
+     * Creates a new instance of UpdateService.
      */
-    private UpdateService() {
+    public UpdateService() {
         loader = ServiceLoader.load(CachedWebDataSource.class);
     }
 
-    /**
-     * Retrieve the singleton instance of UpdateService.
-     *
-     * @return a singleton UpdateService.
-     */
-    public static synchronized UpdateService getInstance() {
-        if (service == null) {
-            service = new UpdateService();
-        }
-        return service;
-    }
-
     /**
      * Returns an Iterator for all instances of the CachedWebDataSource interface.
      *

dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AnalyzerServiceTest.java

@@ -33,7 +33,7 @@ public class AnalyzerServiceTest extends BaseTest {
      */
     @Test
     public void testGetAnalyzers() {
-        AnalyzerService instance = AnalyzerService.getInstance();
+        AnalyzerService instance = new AnalyzerService();
         Iterator<Analyzer> result = instance.getAnalyzers();
 
         boolean found = false;