Merge pull request #78 from OP-TED/feature/TEDEFO-2401-pre-release-ve…

…rsions-in-sdk-downloader

Updated the Dependency factory to use SNAPSHOT versions only when unit testing.

src/main/java/eu/europa/ted/eforms/viewer/DependencyFactory.java

@@ -13,19 +13,36 @@
 import eu.europa.ted.efx.interfaces.TranslatorOptions;
 
 public class DependencyFactory implements TranslatorDependencyFactory {
-  private Path sdkRoot;
-
-  @SuppressWarnings("unused")
-  private DependencyFactory() {}
+  final private Path sdkRoot;
+  final private boolean sdkSnapshotsAllowed;
 
+  /**
+   * Public constructor used by the EfxTranslator does not allow the use of
+   * SNAPSHOT versions of the SDK.
+   * 
+   * @param sdkRoot The root directory where the SDK will be downloaded.
+   */
   public DependencyFactory(Path sdkRoot) {
+    this(sdkRoot, false);
+  }
+
+  /**
+   * Subclasses can use this constructor to allow the use of SNAPSHOT versions of
+   * the SDK.
+   * 
+   * @param sdkRoot          The root directory where the SDK will be downloaded.
+   * @param resolveSnapshots If true, SNAPSHOT versions of the SDK will be
+   *                         downloaded if needed.
+   */
+  protected DependencyFactory(Path sdkRoot, boolean resolveSnapshots) {
     this.sdkRoot = sdkRoot;
+    this.sdkSnapshotsAllowed = resolveSnapshots;
   }
 
   @Override
   public SymbolResolver createSymbolResolver(String sdkVersion) {
     try {
-      SdkDownloader.downloadSdk(sdkVersion, sdkRoot);
+      SdkDownloader.downloadSdk(sdkVersion, sdkRoot, this.sdkSnapshotsAllowed);
 
       return ComponentFactory.getSymbolResolver(sdkVersion, sdkRoot);
     } catch (InstantiationException | IOException e) {

src/test/java/eu/europa/ted/eforms/viewer/DependencyFactoryForUnitTesting.java

@@ -0,0 +1,14 @@
+package eu.europa.ted.eforms.viewer;
+
+import java.nio.file.Path;
+
+/**
+ * Extends {@link DependencyFactory} to allow the use of SNAPSHOT versions when
+ * unit testing.
+ */
+public class DependencyFactoryForUnitTesting extends DependencyFactory {
+
+    public DependencyFactoryForUnitTesting(Path sdkRoot) {
+        super(sdkRoot, true);
+    }
+}

src/test/java/eu/europa/ted/eforms/viewer/NoticeViewerTest.java

@@ -99,7 +99,7 @@ void testEfxToXsl(String sdkVersion) throws IOException, InstantiationException
     final String viewId = "X02";
     final Path xsl =
         XslGenerator.Builder
-            .create(new DependencyFactory(SDK_ROOT_DIR))
+            .create(new DependencyFactoryForUnitTesting(SDK_ROOT_DIR))
             .build()
             .generateFile(sdkVersion, NoticeViewer.getEfxPath(sdkVersion, viewId, SDK_ROOT_DIR),
                 NoticeViewerConstants.DEFAULT_TRANSLATOR_OPTIONS, true);
@@ -145,7 +145,7 @@ private void testGenerateHtmlFromString(final String language, final String noti
         Files.readString(noticeXmlPath, NoticeViewerConstants.DEFAULT_CHARSET);
     final Path xslPath =
         XslGenerator.Builder
-            .create(new DependencyFactory(SDK_ROOT_DIR))
+            .create(new DependencyFactoryForUnitTesting(SDK_ROOT_DIR))
             .build()
             .generateFile(sdkVersion, NoticeViewer.getEfxPath(sdkVersion, viewId, SDK_ROOT_DIR),
                 NoticeViewerConstants.DEFAULT_TRANSLATOR_OPTIONS, true);