[ https://issues.apache.org/jira/browse/IGNITE-6842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413763#comment-16413763 ]
ASF GitHub Bot commented on IGNITE-6842: ---------------------------------------- GitHub user Mmuzaf opened a pull request: https://github.com/apache/ignite/pull/3699 IGNITE-6842: add default behavior by stopping all instances for tests 1. Stop all grids by default in aftreTestsStop method; 2. Throw exception in case stopping process fails; You can merge this pull request into a Git repository by running: $ git pull https://github.com/Mmuzaf/ignite ignite-6842_2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/ignite/pull/3699.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3699 ---- commit abeab5dafd90f385e8eaec57e561eb52570b9136 Author: Maxim Muzafarov <maxmuzaf@...> Date: 2018-03-26T12:10:24Z IGNITE-6842: add default behavior by stopping all instances for tests ---- > Stop all nodes after test by default. > ------------------------------------- > > Key: IGNITE-6842 > URL: https://issues.apache.org/jira/browse/IGNITE-6842 > Project: Ignite > Issue Type: Improvement > Reporter: Alexei Scherbakov > Assignee: Maxim Muzafarov > Priority: Major > Labels: newbie > Fix For: 2.5 > > Attachments: IgniteStopGridsTestSuite.java, > StopGridsStateFirstTest.java, StopGridsStateSecondTest.java > > > Currently it's required to manually call stopAllGrids() after test completion. > This leads to errors in subsequent tests if someone forgets to call it and to > additional boilerplate code in tests. > Right choice is to make this default behavior. -- This message was sent by Atlassian JIRA (v7.6.3#76005)