feat(graalvm-kt): support for powerassert #1565
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1565 +/- ##
==========================================
+ Coverage 39.80% 39.86% +0.06%
==========================================
Files 769 769
Lines 37004 37010 +6
Branches 5235 5230 -5
==========================================
+ Hits 14729 14754 +25
+ Misses 20510 20489 -21
- Partials 1765 1767 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
d0bc204 to
aed5cc7
Compare
31c097f to
bc62252
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for Kotlin Power Assert functionality to the Elide testing framework, enabling enhanced assertion output with detailed breakdowns when tests fail. The implementation integrates the Power Assert compiler plugin into the Kotlin compilation pipeline for improved test debugging experience.
- Integration of kotlin-power-assert-compiler-plugin-embeddable into build dependencies
- Configuration of Power Assert plugin with predefined assertion functions from kotlin.test
- Updates to compiler configuration to enable Power Assert by default for test builds
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| gradle/elide.versions.toml | Adds kotlin-powerAssert-embedded dependency reference |
| packages/graalvm-kt/build.gradle.kts | Includes Power Assert plugin in dependencies and embedded resources |
| packages/builder/build.gradle.kts | Adds Power Assert plugin dependency to builder module |
| packages/graalvm-kt/src/main/kotlin/elide/runtime/gvm/kotlin/KotlinCompilerConfig.kt | Implements Power Assert plugin configuration with function symbols and refactors plugin initialization |
| packages/builder/src/main/kotlin/elide/tooling/kotlin/KotlinCompiler.kt | Integrates Power Assert processor and adds enablePowerAssert parameter |
| packages/builder/src/main/kotlin/elide/tooling/jvm/JvmBuildConfigurator.kt | Updates plugin enablement logic from explicit true checks to false checks |
| packages/tooling/src/main/kotlin/elide/tooling/project/manifest/ElidePackageManifest.kt | Enables experimental features by default |
| packages/builder/api/builder.api | Updates API signature for configureDefaultPlugins method |
| packages/cli/src/main/kotlin/elide/tool/cli/cmd/repl/ToolShellCommand.kt | Improves test failure output formatting |
| packages/cli/src/projects/ktjvm/src/test/kotlin/sample/testGreeting.kt | Updates test expectation to demonstrate Power Assert functionality |
Comments suppressed due to low confidence (1)
packages/graalvm-kt/src/main/kotlin/elide/runtime/gvm/kotlin/KotlinCompilerConfig.kt
Outdated
Show resolved
Hide resolved
packages/graalvm-kt/src/main/kotlin/elide/runtime/gvm/kotlin/KotlinCompilerConfig.kt
Outdated
Show resolved
Hide resolved
packages/graalvm-kt/src/main/kotlin/elide/runtime/gvm/kotlin/KotlinCompilerConfig.kt
Outdated
Show resolved
Hide resolved
feat(cli): make kotlin plugins non-experimental feat(graalvm-kt): powerassert embedded deps feat(graalvm-kt): powerassert built-in chore: update pins for `tooling`/`builder` Signed-off-by: Sam Gammon <sam@elide.ventures>
bc62252 to
f202ad9
Compare
darvld
approved these changes
Jul 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds support for Kotlin Power Assert when running
elide test ...on Kotlin code. Continuance of #1374.Before
After