[
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)
