Skip to content

Commit 19a417e

Browse files
Fix JSClosure calling syntax in stress tests
1 parent 42e879f commit 19a417e

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

Tests/JavaScriptKitTests/StressTests.swift

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ final class StressTests: XCTestCase {
8080
successCount = i + 1
8181

8282
// Test closure immediately to ensure it works under memory pressure
83-
let result = closure([JSValue.number(10)])
84-
XCTAssertEqual(result.number, 110.0) // 100 (capturedData.count) + 10
83+
let result = closure()
84+
XCTAssertEqual(result.number, 100.0) // capturedData.count
8585

8686
// More frequent GC to stress the system
8787
if i % 500 == 0 {
@@ -96,8 +96,8 @@ final class StressTests: XCTestCase {
9696
// Test random closures still work after extreme memory pressure
9797
for _ in 0..<min(100, closures.count) {
9898
let randomIndex = Int.random(in: 0..<closures.count)
99-
let result = closures[randomIndex]([JSValue.number(5)])
100-
XCTAssertTrue(result.number! > 5) // Should be 5 + capturedData.count (100+)
99+
let result = closures[randomIndex]()
100+
XCTAssertEqual(result.number, 100.0) // capturedData.count
101101
}
102102

103103
#if JAVASCRIPTKIT_WITHOUT_WEAKREFS
@@ -152,7 +152,7 @@ final class StressTests: XCTestCase {
152152
}
153153

154154
// Allocate closures with increasing complexity
155-
for i in 0..<closuresThisCycle {
155+
for _ in 0..<closuresThisCycle {
156156
let heavyData = String(repeating: "data", count: cycle + 100)
157157
let closure = JSClosure { arguments in
158158
// Force retention of heavy data
@@ -173,7 +173,7 @@ final class StressTests: XCTestCase {
173173
XCTAssertNotNil(cycleObjects[0]["large_array"].object)
174174
}
175175
if !cycleClosure.isEmpty {
176-
let result = cycleClosure[0](arguments: [])
176+
let result = cycleClosure[0]()
177177
XCTAssertNotNil(result.string)
178178
}
179179

0 commit comments

Comments
 (0)