Skip to content

Commit 46c9604

Browse files
runningcodeclaude
andauthored
build: Remove unnecessary per-test JVM heap caps (#6209)
The sample systemTest tasks only run HTTP clients against sample apps in separate processes, so the 128m/1g cap copied from the old root config isn't needed. sentry-android-core keeps its 2g max for Robolectric, but the 256m min heap was never required. Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
1 parent 6bd6f75 commit 46c9604

25 files changed

Lines changed: 1 addition & 100 deletions

File tree

‎sentry-android-core/build.gradle.kts‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,7 @@ android {
4848
isIncludeAndroidResources = true
4949
// Robolectric loads the android-all jar into each test JVM, which needs more heap
5050
// than the default.
51-
all {
52-
it.minHeapSize = "256m"
53-
it.maxHeapSize = "2g"
54-
}
51+
all { it.maxHeapSize = "2g" }
5552
}
5653
}
5754

‎sentry-samples/sentry-samples-console-opentelemetry-noagent/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,6 @@ tasks.register<Test>("systemTest").configure {
7070

7171
maxParallelForks = 1
7272

73-
// Cap JVM args per test
74-
minHeapSize = "128m"
75-
maxHeapSize = "1g"
76-
7773
filter { includeTestsMatching("io.sentry.systemtest*") }
7874
}
7975

‎sentry-samples/sentry-samples-console-otlp/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,6 @@ tasks.register<Test>("systemTest").configure {
7373

7474
maxParallelForks = 1
7575

76-
// Cap JVM args per test
77-
minHeapSize = "128m"
78-
maxHeapSize = "1g"
79-
8076
filter { includeTestsMatching("io.sentry.systemtest*") }
8177
}
8278

‎sentry-samples/sentry-samples-console/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,10 +74,6 @@ tasks.register<Test>("systemTest").configure {
7474

7575
maxParallelForks = 1
7676

77-
// Cap JVM args per test
78-
minHeapSize = "128m"
79-
maxHeapSize = "1g"
80-
8177
filter { includeTestsMatching("io.sentry.systemtest*") }
8278
}
8379

‎sentry-samples/sentry-samples-jul/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,6 @@ tasks.register<Test>("systemTest").configure {
6565

6666
maxParallelForks = 1
6767

68-
// Cap JVM args per test
69-
minHeapSize = "128m"
70-
maxHeapSize = "1g"
71-
7268
filter { includeTestsMatching("io.sentry.systemtest*") }
7369
}
7470

‎sentry-samples/sentry-samples-log4j2/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,10 +71,6 @@ tasks.register<Test>("systemTest").configure {
7171

7272
maxParallelForks = 1
7373

74-
// Cap JVM args per test
75-
minHeapSize = "128m"
76-
maxHeapSize = "1g"
77-
7874
filter { includeTestsMatching("io.sentry.systemtest*") }
7975
}
8076

‎sentry-samples/sentry-samples-logback/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,6 @@ tasks.register<Test>("systemTest").configure {
6565

6666
maxParallelForks = 1
6767

68-
// Cap JVM args per test
69-
minHeapSize = "128m"
70-
maxHeapSize = "1g"
71-
7268
filter { includeTestsMatching("io.sentry.systemtest*") }
7369
}
7470

‎sentry-samples/sentry-samples-spring-7/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,6 @@ tasks.register<Test>("systemTest").configure {
8181

8282
maxParallelForks = 1
8383

84-
// Cap JVM args per test
85-
minHeapSize = "128m"
86-
maxHeapSize = "1g"
87-
8884
filter { includeTestsMatching("io.sentry.systemtest*") }
8985
}
9086

‎sentry-samples/sentry-samples-spring-boot-4-log4j2/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,6 @@ tasks.register<Test>("systemTest").configure {
9999

100100
maxParallelForks = 1
101101

102-
// Cap JVM args per test
103-
minHeapSize = "128m"
104-
maxHeapSize = "1g"
105-
106102
filter { includeTestsMatching("io.sentry.systemtest*") }
107103
}
108104

‎sentry-samples/sentry-samples-spring-boot-4-opentelemetry-noagent/build.gradle.kts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,6 @@ tasks.register<Test>("systemTest").configure {
8989

9090
maxParallelForks = 1
9191

92-
// Cap JVM args per test
93-
minHeapSize = "128m"
94-
maxHeapSize = "1g"
95-
9692
filter { includeTestsMatching("io.sentry.systemtest*") }
9793
}
9894

0 commit comments

Comments
 (0)