You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by il...@apache.org on 2019/01/23 10:22:54 UTC

[ignite] branch master updated: IGNITE-10629 Fix old style tests that slipped through - Fixes #5879.

This is an automated email from the ASF dual-hosted git repository.

ilyak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new afde85b  IGNITE-10629 Fix old style tests that slipped through - Fixes #5879.
afde85b is described below

commit afde85be4cd7d22a5195a42e5f9bd837c9a9b180
Author: Oleg Ignatenko <oi...@gridgain.com>
AuthorDate: Wed Jan 23 13:22:15 2019 +0300

    IGNITE-10629 Fix old style tests that slipped through - Fixes #5879.
    
    Signed-off-by: Ilya Kasnacheev <il...@gmail.com>
---
 ...terActivateDeactivateTestWithPersistenceAndMemoryReuse.java |  2 ++
 .../db/IgnitePdsCacheWalDisabledOnRebalancingTest.java         |  1 +
 .../jta/GridPartitionedCacheJtaLookupClassNameSelfTest.java    |  6 +++---
 .../internal/websession/WebSessionTransactionalSelfTest.java   | 10 +++++++---
 4 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteClusterActivateDeactivateTestWithPersistenceAndMemoryReuse.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteClusterActivateDeactivateTestWithPersistenceAndMemoryReuse.java
index c95fe3d..b7b4a88 100644
--- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteClusterActivateDeactivateTestWithPersistenceAndMemoryReuse.java
+++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteClusterActivateDeactivateTestWithPersistenceAndMemoryReuse.java
@@ -58,6 +58,8 @@ public class IgniteClusterActivateDeactivateTestWithPersistenceAndMemoryReuse ex
         super.testDeactivateInactiveCluster();
     }
 
+    /** {@inheritDoc} */
+    @Test
     @Override public void testReActivateSimple_5_Servers_4_Clients_FromServer() throws Exception {
         if (MvccFeatureChecker.forcedMvcc())
             fail("https://issues.apache.org/jira/browse/IGNITE-10750");
diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/db/IgnitePdsCacheWalDisabledOnRebalancingTest.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/db/IgnitePdsCacheWalDisabledOnRebalancingTest.java
index 46ac18b..d8cbdfb 100644
--- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/db/IgnitePdsCacheWalDisabledOnRebalancingTest.java
+++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/db/IgnitePdsCacheWalDisabledOnRebalancingTest.java
@@ -274,6 +274,7 @@ public class IgnitePdsCacheWalDisabledOnRebalancingTest extends GridCommonAbstra
      *
      * @throws Exception If failed.
      */
+    @Test
     public void testRebalancedPartitionsOwningWithConcurrentAffinityChange() throws Exception {
         Ignite ig0 = startGridsMultiThreaded(4);
         fillCache(ig0.dataStreamer(CACHE3_NAME), CACHE_SIZE, GENERATING_FUNC);
diff --git a/modules/jta/src/test/java/org/apache/ignite/internal/processors/cache/jta/GridPartitionedCacheJtaLookupClassNameSelfTest.java b/modules/jta/src/test/java/org/apache/ignite/internal/processors/cache/jta/GridPartitionedCacheJtaLookupClassNameSelfTest.java
index 4c95e57..573a543 100644
--- a/modules/jta/src/test/java/org/apache/ignite/internal/processors/cache/jta/GridPartitionedCacheJtaLookupClassNameSelfTest.java
+++ b/modules/jta/src/test/java/org/apache/ignite/internal/processors/cache/jta/GridPartitionedCacheJtaLookupClassNameSelfTest.java
@@ -27,6 +27,7 @@ import org.apache.ignite.configuration.IgniteConfiguration;
 import org.apache.ignite.internal.IgniteEx;
 import org.apache.ignite.testframework.GridTestUtils;
 import org.junit.Ignore;
+import org.junit.Test;
 
 /**
  * Lookup class name based JTA integration test using PARTITIONED cache.
@@ -37,10 +38,9 @@ public class GridPartitionedCacheJtaLookupClassNameSelfTest extends AbstractCach
         cfg.getTransactionConfiguration().setTxManagerLookupClassName(TestTmLookup.class.getName());
     }
 
-    /**
-     *
-     */
+    /** */
     @Ignore("https://issues.apache.org/jira/browse/IGNITE-10723")
+    @Test
     public void testIncompatibleTmLookup() {
         final IgniteEx ignite = grid(0);
 
diff --git a/modules/web/src/test/java/org/apache/ignite/internal/websession/WebSessionTransactionalSelfTest.java b/modules/web/src/test/java/org/apache/ignite/internal/websession/WebSessionTransactionalSelfTest.java
index 2a37ff4..0559184 100644
--- a/modules/web/src/test/java/org/apache/ignite/internal/websession/WebSessionTransactionalSelfTest.java
+++ b/modules/web/src/test/java/org/apache/ignite/internal/websession/WebSessionTransactionalSelfTest.java
@@ -18,6 +18,7 @@
 package org.apache.ignite.internal.websession;
 
 import org.junit.Ignore;
+import org.junit.Test;
 
 /**
  * Tests web sessions with TRANSACTIONAL cache.
@@ -30,19 +31,22 @@ public class WebSessionTransactionalSelfTest extends WebSessionSelfTest {
 
     /** {@inheritDoc} */
     @Ignore("https://issues.apache.org/jira/browse/IGNITE-810")
-    @Override public void testRestarts() throws Exception {
+    @Test
+    @Override public void testRestarts() {
         // No-op.
     }
 
     /** {@inheritDoc} */
     @Ignore("https://issues.apache.org/jira/browse/IGNITE-810")
-    @Override public void testInvalidatedSession() throws Exception {
+    @Test
+    @Override public void testInvalidatedSession() {
         // No-op.
     }
 
     /** {@inheritDoc} */
     @Ignore("https://issues.apache.org/jira/browse/IGNITE-810")
-    @Override public void testClientReconnectRequest() throws Exception {
+    @Test
+    @Override public void testClientReconnectRequest() {
         // No-op.
     }
 }