Once QuickJS supports WeakRef, enable regular tests:

[github-actions]

suspend fun detectImmediateCollection()
suspend fun detectDelayedCollection()
suspend fun detectLeak()

redwood/redwood-leak-detector-zipline-test/src/commonMain/kotlin/app/cash/redwood/leaks/zipline/LeakDetectorTestService.kt

Line 23 in d5defde

// TODO Once QuickJS supports WeakRef, enable regular tests:

/*
 * Copyright (C) 2024 Square, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package app.cash.redwood.leaks.zipline

import app.cash.zipline.ZiplineService

interface LeakDetectorTestService : ZiplineService {
  fun leakDetectorDisabled()

  // TODO Once QuickJS supports WeakRef, enable regular tests:
  //  suspend fun detectImmediateCollection()
  //  suspend fun detectDelayedCollection()
  //  suspend fun detectLeak()

  companion object {
    const val SERVICE_NAME = "leakDetectorTest"
  }
}

[JakeWharton]

bellard/quickjs#337

[swankjesse]

Maybe we can use the FinalizationRegistry bindings in Zipline?