ignite-471-2: keep fixing tests
Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/119acfd4 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/119acfd4 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/119acfd4 Branch: refs/heads/ignite-471-2 Commit: 119acfd4a55190e95dbef751395bf854d247db51 Parents: 713c0e6 Author: Denis Magda <dma...@gridgain.com> Authored: Thu May 21 11:04:20 2015 +0300 Committer: Denis Magda <dma...@gridgain.com> Committed: Thu May 21 11:04:20 2015 +0300 ---------------------------------------------------------------------- .../internal/util/nio/GridNioSelfTest.java | 25 +++++++++++--------- .../ignite/testsuites/IgniteBasicTestSuite.java | 2 +- .../testsuites/IgniteUtilSelfTestSuite.java | 14 ++++++++++- 3 files changed, 28 insertions(+), 13 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/119acfd4/modules/core/src/test/java/org/apache/ignite/internal/util/nio/GridNioSelfTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/util/nio/GridNioSelfTest.java b/modules/core/src/test/java/org/apache/ignite/internal/util/nio/GridNioSelfTest.java index e3baeb0..254d3ed 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/util/nio/GridNioSelfTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/util/nio/GridNioSelfTest.java @@ -644,7 +644,7 @@ public class GridNioSelfTest extends GridCommonAbstractTest { try { client = createClient(U.getLocalHost(), PORT, U.getLocalHost()); - MessageWithId msg = new MessageWithId(); + MessageWithId msg = new MessageWithId(idProvider.getAndIncrement()); byte[] data = serializeMessage(msg); @@ -746,7 +746,7 @@ public class GridNioSelfTest extends GridCommonAbstractTest { client = createClient(U.getLocalHost(), PORT, U.getLocalHost()); while (cntr.getAndIncrement() < MSG_CNT * THREAD_CNT) { - MessageWithId msg = new MessageWithId(); + MessageWithId msg = new MessageWithId(idProvider.getAndIncrement()); byte[] data = serializeMessage(msg); @@ -829,13 +829,11 @@ public class GridNioSelfTest extends GridCommonAbstractTest { U.sleep(4000); info("After sleep."); - } - catch (Exception e) { + } catch (Exception e) { error("Failed to create client: " + e.getMessage()); fail("Failed to create client: " + e.getMessage()); - } - finally { + } finally { info("Test thread finished."); } } @@ -904,13 +902,11 @@ public class GridNioSelfTest extends GridCommonAbstractTest { U.sleep(4000); info("After sleep."); - } - catch (Exception e) { + } catch (Exception e) { error("Failed to create client: ", e); fail("Failed to create client: " + e.getMessage()); - } - finally { + } finally { info("Test thread finished."); } } @@ -1408,7 +1404,14 @@ public class GridNioSelfTest extends GridCommonAbstractTest { */ private static class MessageWithId implements Serializable { /** */ - private final int id = idProvider.getAndIncrement(); + private final int id; + + /** + * @param id Message ID. + */ + public MessageWithId(int id) { + this.id = id; + } /** */ @SuppressWarnings({"unused"}) http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/119acfd4/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteBasicTestSuite.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteBasicTestSuite.java b/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteBasicTestSuite.java index 1a642d4..ed8579f 100644 --- a/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteBasicTestSuite.java +++ b/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteBasicTestSuite.java @@ -55,7 +55,7 @@ public class IgniteBasicTestSuite extends TestSuite { TestSuite suite = new TestSuite("Ignite Basic Test Suite"); suite.addTest(IgniteLangSelfTestSuite.suite()); - suite.addTest(IgniteUtilSelfTestSuite.suite()); + suite.addTest(IgniteUtilSelfTestSuite.suite(ignoredTests)); suite.addTest(IgniteMarshallerSelfTestSuite.suite(ignoredTests)); suite.addTest(IgniteKernalSelfTestSuite.suite()); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/119acfd4/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteUtilSelfTestSuite.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteUtilSelfTestSuite.java b/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteUtilSelfTestSuite.java index 32cd038..92ad2e1 100644 --- a/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteUtilSelfTestSuite.java +++ b/modules/core/src/test/java/org/apache/ignite/testsuites/IgniteUtilSelfTestSuite.java @@ -26,10 +26,13 @@ import org.apache.ignite.internal.util.offheap.unsafe.*; import org.apache.ignite.internal.util.tostring.*; import org.apache.ignite.lang.*; import org.apache.ignite.spi.discovery.*; +import org.apache.ignite.testframework.*; import org.apache.ignite.thread.*; import org.apache.ignite.util.*; import org.apache.ignite.util.mbeans.*; +import java.util.*; + /** * Test suite for Ignite utility classes. */ @@ -39,6 +42,15 @@ public class IgniteUtilSelfTestSuite extends TestSuite { * @throws Exception If failed. */ public static TestSuite suite() throws Exception { + return suite(null); + } + + /** + * @param ignoredTests Tests don't include in the execution. + * @return Test suite. + * @throws Exception Thrown in case of the failure. + */ + public static TestSuite suite(Set<Class> ignoredTests) throws Exception { TestSuite suite = new TestSuite("Ignite Util Test Suite"); suite.addTestSuite(GridThreadPoolExecutorServiceSelfTest.class); @@ -66,7 +78,7 @@ public class IgniteUtilSelfTestSuite extends TestSuite { suite.addTestSuite(GridNioSessionMetaKeySelfTest.class); suite.addTestSuite(GridNioSelfTest.class); suite.addTestSuite(GridNioFilterChainSelfTest.class); - suite.addTestSuite(GridNioSslSelfTest.class); + GridTestUtils.addTestIfNeeded(suite, GridNioSslSelfTest.class, ignoredTests); suite.addTestSuite(GridNioDelimitedBufferTest.class); return suite;