Skip to content

[#160] Support of ExtendWith annotation#479

Merged
stephanlukasczyk merged 7 commits intodevelopmentfrom
izaitsev/bugs/160-extend-with-annotation-fix
Apr 11, 2025
Merged

[#160] Support of ExtendWith annotation#479
stephanlukasczyk merged 7 commits intodevelopmentfrom
izaitsev/bugs/160-extend-with-annotation-fix

Conversation

@Hello-zoka
Copy link
Collaborator

Description of changes made

Previously, both ExtendWith and RunWith annotations were replaced with RunWith ones since it is available in both JUnit versions. This fix makes any annotation used in JUnit 4 be replaced with RunWith, while for JUnit 5 replaces it with ExtendWith(which is a superset of runWith). Also introduces more logical variable naming.

Why is a merge request needed

These changes have the potential to lower the amount of compilation errors for JUnit 5 tests, in case of using unique for ExtenWith annotation APIs.

Other notes

Closes #160

  • I have checked that I am merging into the correct branch

@Hello-zoka Hello-zoka added new feature Implementation of a new feature refactoring Refactor the project labels Mar 27, 2025
Copy link
Collaborator

@stephanlukasczyk stephanlukasczyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please provide a clarification on my question in the JavaDisplayUtils and KotlinDisplayUtils? I am not sure whether I misunderstand it or whether there is a change missing.

@Hello-zoka
Copy link
Collaborator Author

Could you please provide a clarification on my question in the JavaDisplayUtils and KotlinDisplayUtils? I am not sure whether I misunderstand it or whether there is a change missing.

Yes, changes were missed. Fixed.

@stephanlukasczyk stephanlukasczyk force-pushed the izaitsev/bugs/160-extend-with-annotation-fix branch from 1580739 to 1b97ec4 Compare April 11, 2025 18:04
@stephanlukasczyk stephanlukasczyk merged commit 6a2a93a into development Apr 11, 2025
3 checks passed
@stephanlukasczyk stephanlukasczyk deleted the izaitsev/bugs/160-extend-with-annotation-fix branch April 11, 2025 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature Implementation of a new feature refactoring Refactor the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JavaClassBuilderHelper does not support @ExtendWith from JUnit5

2 participants