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

Add a form of parametrized test to the test framework #781

Closed
Sirttas opened this issue Apr 4, 2024 · 2 comments
Closed

Add a form of parametrized test to the test framework #781

Sirttas opened this issue Apr 4, 2024 · 2 comments
Labels
enhancement New (or improvement to existing) feature or request

Comments

@Sirttas
Copy link
Contributor

Sirttas commented Apr 4, 2024

Following the short disruption in #761, I realized that what I wanted to add was a form of parametrized test and the implementation I suggested wasn't the right one.

The Implementation I would suggest is to do something similar to Junit:

@ParametrizedTestHolder(description = "Tests description")
@MethodSource("provideTestArguments")
static void someTest(final DynamicTest test, String param1, int param2) {
    ...
}

...

static Stream<Arguments> provideTestArguments() {
    return Stream.of(
        Arguments.of("test", 1),
        ...
    );
}
@Sirttas Sirttas added the enhancement New (or improvement to existing) feature or request label Apr 4, 2024
@Matyrobbrt
Copy link
Member

Wouldn't parametrized test be better made using... JUnit? We're planning to implement junit support, right now it's waiting on @marchermans.

@Matyrobbrt
Copy link
Member

We now have JUnit support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New (or improvement to existing) feature or request
Projects
None yet
Development

No branches or pull requests

2 participants