Skip to content

Conversation

leonard84
Copy link
Contributor

As a followup to #6898. The annotation org.junit.platform.commons.annotation.Testable can also be applied to the test class so it should also be added to the CLASS_ANNOTATIONS array. Spock-2.0 uses @Testable on the class instead of each test.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 12, 2019
@philwebb philwebb added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Sep 12, 2019
@philwebb philwebb added this to the 2.1.x milestone Sep 12, 2019
@philwebb philwebb changed the title Add Testable to CLASS_ANNOTATIONS in TestTypeExcludeFilter TestTypeExcludeFilter does not detect JUnit 5's @Testable annotation Sep 12, 2019
@snicoll snicoll self-assigned this Sep 16, 2019
@snicoll snicoll modified the milestones: 2.1.x, 2.1.9 Sep 16, 2019
@snicoll snicoll closed this in ece29bb Sep 16, 2019
@snicoll
Copy link
Member

snicoll commented Sep 16, 2019

@leonard84 thank you for making your first contribution to Spring Boot.

@leonard84 leonard84 deleted the fix-testable-exclude branch September 16, 2019 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants