Skip to content

Commit

Permalink
Refactor unit test memory limit so it is unset by default
Browse files Browse the repository at this point in the history
A fixed limit didn't work for all use cases. This approach restores the
previous default and allows developers to configure their own if
required.
  • Loading branch information
markt-asf committed Feb 5, 2025
1 parent 9651aa9 commit e9c7cc4
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions build.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2027,6 +2027,8 @@
<condition property="opens.javautilconcurrent" value="--add-opens=java.base/java.util.concurrent=ALL-UNNAMED" else="-Dtest14=1">
<javaversion atleast="9"/>
</condition>
<!-- Set memory properties (unset by default) -->
<property name="test.xmx" value="-Dnoop"/>

<macrodef name="runtests"
description="Runs the unit tests using the specified connector.
Expand All @@ -2046,11 +2048,11 @@
errorproperty="test.result.error"
failureproperty="test.result.failure"
haltonfailure="${test.haltonfailure}"
threads="${test.threads}"
maxmemory="256m">
threads="${test.threads}">

<jvmarg value="${test.jvmarg.egd}"/>
<jvmarg value="-Dfile.encoding=UTF-8"/>
<jvmarg value="${test.xmx}"/>
<jvmarg value="-Djava.library.path=${test.apr.loc}"/>
<jvmarg value="${test.formatter}"/>
<jvmarg value="-Djava.net.preferIPv4Stack=${java.net.preferIPv4Stack}"/>
Expand Down

0 comments on commit e9c7cc4

Please sign in to comment.