Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests for kotlinx-coroutines-debug module fail on JDK 17 due to outdated BlockHound version #3701

Closed
FenstonSingel opened this issue Apr 5, 2023 · 1 comment
Labels

Comments

@FenstonSingel
Copy link
Contributor

When running on JDK 17 (and potentially some earlier versions as well), BlockHound 1.0.2.RELEASE cannot process JDK class files with a new class file version, leading to failure of several tests for the kotlinx-coroutines-debug module.

Bumping BlockHound version to 1.0.7.RELEASE solves the inability to process class files, but discovers the problem of changed JVMTI behavior (see corresponding BlockHound issue and JDK issue). Since the corresponding BlockHound issue is not solved at the moment of writing this, the following workaround needs to be added to kotlinx-coroutines-debug Gradle buildscript:

tasks.withType(Test).configureEach {
    if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_13)) {
        jvmArgs += ["-XX:+AllowRedefinitionToAddDeleteMethods"]
    }
}
@FenstonSingel
Copy link
Contributor Author

Note: issue #3700 prevents the described fixes from being applied right away.

qwwdfsad added a commit that referenced this issue Jun 2, 2023
* Also, exclude integration from docs, it shouldn't be there

Fixes #3701
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant