Skip to content

Provide support for publishing test execution life-cycle events#22477

Closed
maverick1601 wants to merge 1 commit intospring-projects:masterfrom
maverick1601:18490
Closed

Provide support for publishing test execution life-cycle events#22477
maverick1601 wants to merge 1 commit intospring-projects:masterfrom
maverick1601:18490

Conversation

@maverick1601
Copy link
Copy Markdown

@maverick1601 maverick1601 commented Feb 26, 2019

Adds new TestExecutionListener for publishing events to the test application context.
These may be consumed by @eventlistener annotated methods to react to the test
life-cycle. The test execution listener isn't enabled by default.

Implements #18490

@pivotal-issuemaster
Copy link
Copy Markdown

pivotal-issuemaster commented Feb 26, 2019

@maverick1601 Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link
Copy Markdown

pivotal-issuemaster commented Feb 26, 2019

@maverick1601 Thank you for signing the Contributor License Agreement!

@maverick1601 maverick1601 mentioned this pull request Feb 26, 2019
Closed
5 tasks
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Feb 26, 2019
@sbrannen sbrannen self-assigned this Feb 27, 2019
@sbrannen sbrannen added in: test Issues in the test module type: enhancement A general enhancement labels Feb 27, 2019
@sbrannen sbrannen added this to the 5.2 M1 milestone Feb 27, 2019
@sbrannen
Copy link
Copy Markdown
Member

sbrannen commented Feb 27, 2019

I noticed there are some code quality violations.

Would you mind addressing those in your PR?

@maverick1601
Copy link
Copy Markdown
Author

maverick1601 commented Feb 27, 2019

I will fix those

@sbrannen sbrannen removed the status: waiting-for-triage An issue we've not yet triaged or decided on label Feb 27, 2019
@maverick1601
Copy link
Copy Markdown
Author

maverick1601 commented Feb 27, 2019 •
edited
Loading

Seems like most of them are fixed. There is one violation, because the dummyTestMethod() used in the integration test needs documentation. But for me this is a false positive.

@sbrannen
Copy link
Copy Markdown
Member

sbrannen commented Feb 28, 2019

Seems like most of them are fixed.

Thanks

sbrannen
sbrannen requested changes Feb 28, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@sbrannen sbrannen left a comment

Choose a reason for hiding this comment

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

Looking good!

Please make the requested changes, and I'll then make a final review.

Cheers,

Sam

Comment thread spring-test/src/main/java/org/springframework/test/context/TestExecutionListener.java Outdated
Comment thread ...-test/src/main/java/org/springframework/test/context/event/AfterClassTestExecutionEvent.java Outdated
Comment thread ...-test/src/main/java/org/springframework/test/context/event/AfterClassTestExecutionEvent.java Outdated
Comment thread spring-test/src/main/java/org/springframework/test/context/event/package-info.java Outdated
Comment thread .../springframework/test/context/event/EventPublishingTestExecutionListenerIntegrationTest.java Outdated
Comment thread .../springframework/test/context/event/EventPublishingTestExecutionListenerIntegrationTest.java Outdated
Comment thread spring-test/src/main/java/org/springframework/test/context/event/annotation/AfterTestClass.java Outdated
Comment thread ...st/java/org/springframework/test/context/event/EventPublishingTestExecutionListenerTest.java Outdated
Comment thread .../springframework/test/context/event/EventPublishingTestExecutionListenerIntegrationTest.java Outdated
Comment thread ...st/java/org/springframework/test/context/event/EventPublishingTestExecutionListenerTest.java Outdated
@maverick1601
Copy link
Copy Markdown
Author

maverick1601 commented Feb 28, 2019

Resolved all review tasks, hopefully.

@sbrannen
Copy link
Copy Markdown
Member

sbrannen commented Feb 28, 2019

Resolved all review tasks, hopefully.

Yep. Looks good. I think the only thing you missed is the copyright date for the test classes, but that's not a big deal. 😉

Thanks for making the changes!

Provide support for publishing test execution life-cycle events
e9b63ce 
Adds new TestExecutionListener for publishing events to the test application context.
These may be consumed by @eventlistener annotated methods to react to the test
life-cycle. The test execution listener isn't enabled by default.

Implements spring-projects#18490
@maverick1601
Copy link
Copy Markdown
Author

maverick1601 commented Feb 28, 2019 •
edited
Loading

Fixed the test class copyrights, as well. These were all due to the cherry-pick from 2016 ;-)

@sbrannen
Copy link
Copy Markdown
Member

sbrannen commented Feb 28, 2019

Thanks for all of the hard work, @maverick1601.

I'll see if I can merge this in the coming days.

@sbrannen
Copy link
Copy Markdown
Member

sbrannen commented Mar 1, 2019

This work has been merged into master in 34fee86 and refined in 2d6624d.

@sbrannen sbrannen closed this Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sbrannen sbrannen sbrannen approved these changes

Assignees

@sbrannen sbrannen

Labels

in: test Issues in the test module type: enhancement A general enhancement

Projects

None yet

Milestone

5.2 M1

Development

Successfully merging this pull request may close these issues.

4 participants

@maverick1601 @pivotal-issuemaster @sbrannen @spring-projects-issues