>From Murtadha Hubail <[email protected]>:

Murtadha Hubail has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18276 )

Change subject: [ASTERIXDB-3390][STO]: Add GCS Substitution in Tests
......................................................................

[ASTERIXDB-3390][STO]: Add GCS Substitution in Tests

Change-Id: Ib2f2c71fae06767b62e944f55d7a37a5cad79ab4
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18276
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Reviewed-by: Murtadha Hubail <[email protected]>
---
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
1 file changed, 24 insertions(+), 0 deletions(-)

Approvals:
  Murtadha Hubail: Looks good to me, approved
  Jenkins: Verified; Verified




diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
index f95111e..20fef82 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
@@ -2376,6 +2376,8 @@
                     str = applyS3Substitution(str, placeholders);
                 } else if 
(placeholder.getValue().equalsIgnoreCase("AzureBlob")) {
                     str = applyAzureSubstitution(str, placeholders);
+                } else if (placeholder.getValue().equalsIgnoreCase("GCS")) {
+                    str = applyGCSSubstitution(str, placeholders);
                 }
             } else {
                 // Any other place holders, just replace with the value
@@ -2481,6 +2483,11 @@
         return str;
     }

+    protected String applyGCSSubstitution(String str, List<Placeholder> 
placeholders) {
+        str = setGCSTemplateDefault(str);
+        return str;
+    }
+
     protected String setAzureTemplate(String str) {
         return str.replace("%template%", TEMPLATE);
     }
@@ -2489,6 +2496,10 @@
         return str.replace("%template%", TEMPLATE_DEFAULT);
     }

+    protected String setGCSTemplateDefault(String str) {
+        return str;
+    }
+
     protected void fail(boolean runDiagnostics, TestCaseContext testCaseCtx, 
CompilationUnit cUnit,
             List<TestFileContext> testFileCtxs, ProcessBuilder pb, File 
testFile, Exception e) throws Exception {
         if (runDiagnostics) {

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18276
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ib2f2c71fae06767b62e944f55d7a37a5cad79ab4
Gerrit-Change-Number: 18276
Gerrit-PatchSet: 3
Gerrit-Owner: Savyasach Reddy <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-MessageType: merged

Reply via email to