>From Michael Blow <[email protected]>:

Michael Blow has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17834 )

Change subject: [NO ISSUE][*DB][TEST] += ability for modules to add jvm args to 
tests
......................................................................

[NO ISSUE][*DB][TEST] += ability for modules to add jvm args to tests

Change-Id: I5c344fd96c304ec22aed902a66eddbb6027facb4
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17834
Reviewed-by: Michael Blow <[email protected]>
Reviewed-by: Ali Alsuliman <[email protected]>
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
---
M asterixdb/pom.xml
1 file changed, 21 insertions(+), 1 deletion(-)

Approvals:
  Michael Blow: Looks good to me, but someone else must approve
  Ali Alsuliman: Looks good to me, approved
  Jenkins: Verified; Verified




diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index c1ee08f..a710303 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -62,6 +62,8 @@
     <skip.surefire.tests>${skipTests}</skip.surefire.tests>
     <skip.testResources>${maven.test.skip}</skip.testResources>
     
<testLog4jConfigFile>${root.dir}/asterix-app/src/test/resources/log4j2-asterixdb-test.xml</testLog4jConfigFile>
+    <extraSurefireArgLine/>
+    <extraFailsafeArgLine/>

     <!-- Definition of tests in various categories which may be excluded -->
     <repeated.tests>**/RepeatedTest.java</repeated.tests>
@@ -146,6 +148,7 @@
             -Xdebug
             
-Xrunjdwp:transport=dt_socket,server=y,address=8000,suspend=${debug.suspend.flag}
             ${coverageArgLine}
+            ${extraSurefireArgLine}
           </argLine>
           <includes>
             <include>${test.includes}</include>
@@ -163,7 +166,10 @@
           <runOrder>alphabetical</runOrder>
           <forkCount>1</forkCount>
           <reuseForks>false</reuseForks>
-          <argLine>${coverageArgLine}</argLine>
+          <argLine>
+            ${coverageArgLine}
+            ${extraFailsafeArgLine}
+          </argLine>
           <systemProperties>
             
<log4j.configurationFile>${testLog4jConfigFile}</log4j.configurationFile>
           </systemProperties>

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

Gerrit-Project: asterixdb
Gerrit-Branch: trinity
Gerrit-Change-Id: I5c344fd96c304ec22aed902a66eddbb6027facb4
Gerrit-Change-Number: 17834
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Blow <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail
Gerrit-MessageType: merged

Reply via email to