Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1980

Change subject: [NO ISSUE] Refactor LicenseIT for extensibility
......................................................................

[NO ISSUE] Refactor LicenseIT for extensibility

Change-Id: Idc82843905dbe3c3d26edcbbbebdde35fdefc775
---
M 
asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/LicensingIT.java
1 file changed, 10 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/80/1980/1

diff --git 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/LicensingIT.java
 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/LicensingIT.java
index e5a3d3e..e8864c3 100644
--- 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/LicensingIT.java
+++ 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/LicensingIT.java
@@ -61,15 +61,19 @@
 
     @Test
     public void testLicenseNoticeFilesPresent() throws IOException {
-        for (String name : Arrays.asList("LICENSE", "NOTICE")) {
+        for (String name : getRequiredArtifactNames()) {
             final String fileName = FileUtil.joinPath(installerDir, 
pathToLicensingFiles(), name);
             Assert.assertTrue(fileName + " missing", new 
File(fileName).exists());
         }
     }
 
+    protected List<String> getRequiredArtifactNames() {
+        return Arrays.asList(getLicenseArtifactName(), "NOTICE");
+    }
+
     @Test
     public void testNoMissingLicenses() throws IOException {
-        final File licenseFile = new File(FileUtil.joinPath(installerDir, 
pathToLicensingFiles(), "LICENSE"));
+        final File licenseFile = new File(FileUtil.joinPath(installerDir, 
pathToLicensingFiles(), getLicenseArtifactName()));
         List<String> badLines = new ArrayList<>();
         for (String line : FileUtils.readLines(licenseFile, 
StandardCharsets.UTF_8)) {
             if (line.matches("^\\s*MISSING:.*")) {
@@ -78,4 +82,8 @@
         }
         Assert.assertEquals("Missing licenses in " + licenseFile + ": " + 
badLines, 0, badLines.size());
     }
+
+    protected String getLicenseArtifactName() {
+        return "LICENSE";
+    }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1980
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc82843905dbe3c3d26edcbbbebdde35fdefc775
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <[email protected]>

Reply via email to