Add support for JUnit's @ParameterizedClass#711
Add support for JUnit's @ParameterizedClass#711marcphilipp wants to merge 6 commits intograalvm:masterfrom
@ParameterizedClass#711Conversation
|
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application. When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated. If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public. |
oracle-contributor-agreement
bot
added
the
OCA Required
| implementation(libs.openjson) | ||
| testImplementation(platform(libs.test.junit.bom)) | ||
| testImplementation(libs.test.junit.jupiter.core) | ||
| testRuntimeOnly(libs.test.junit.platform.launcher) |
There was a problem hiding this comment.
This forces Gradle to use an aligned version of junit-platform-launcher. See https://junit.org/junit5/docs/current/user-guide/#running-tests-build-gradle-bom for details.
| includeGroup("org.junit") | ||
| includeGroupByRegex("org\\.junit\\..*") | ||
| } | ||
| } |
There was a problem hiding this comment.
This is only temporary until there's a release (candidate?) on Maven Central.
There was a problem hiding this comment.
Could you please add such valuable comments to the commit message bodies instead of the PR discussion?
| "org.junit.jupiter.engine.config.InstantiatingConfigurationParameterConverter", | ||
| // new in Junit 5.13? | ||
| "org.junit.jupiter.api.DisplayNameGenerator$IndicativeSentences", | ||
| "org.junit.jupiter.engine.discovery.ClassSelectorResolver" |
There was a problem hiding this comment.
These are also temporary since they have been added to the native-image.properties that ship with JUnit in the meantime.
| @BeforeAll | ||
| public static void setup() { | ||
| actualArgs.clear(); | ||
| expectedArgs.clear(); |
There was a problem hiding this comment.
This lifecycle method was never called in JUnit 5.11 and earlier. See https://junit.org/junit5/docs/5.11.0/user-guide/index.html#extensions-supported-utilities-search-semantics for details.
This led to both actual and expected args to accumulate which passed the assertions in the @AfterAll method below. Now that this method is called, these assertions failed for all but the first executed subclass.
marcphilipp
changed the title
@ParameterizedClass@ParameterizedClass
| @@ -0,0 +1,84 @@ | |||
| /* | |||
| * Copyright (c) 2020, 2023, Oracle and/or its affiliates. All rights reserved. | |||
There was a problem hiding this comment.
Should this just be 2025?
| registerTestClassForReflection(superClass); | ||
| } | ||
| for (var declaredClass : clazz.getDeclaredClasses()) { | ||
| if (!Modifier.isStatic(declaredClass.getModifiers())) { |
There was a problem hiding this comment.
Additionally, one could also look for @Nested but that would mean a closer coupling. Thoughts?
|
Thank you for signing the OCA. |
oracle-contributor-agreement
bot
added
OCA Verified
|
Superseded by #723 |
3 participants
JUnit 5.13 will introduce parameterized test classes which also allow
@Nestedtest classes inside of them, but those are not part of theTestPlanand thus not registered for reflection.This PR addresses that by registering all inner classes of test classes for reflection.
@Nestedtests in@ParameterizedClasses junit-team/junit-framework#4440