You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by vo...@apache.org on 2018/04/02 08:35:30 UTC

ignite git commit: Removed test dependency on Hamcrest.

Repository: ignite
Updated Branches:
  refs/heads/master 61c9ffb7a -> d8fc15bc2


Removed test dependency on Hamcrest.


Project: http://git-wip-us.apache.org/repos/asf/ignite/repo
Commit: http://git-wip-us.apache.org/repos/asf/ignite/commit/d8fc15bc
Tree: http://git-wip-us.apache.org/repos/asf/ignite/tree/d8fc15bc
Diff: http://git-wip-us.apache.org/repos/asf/ignite/diff/d8fc15bc

Branch: refs/heads/master
Commit: d8fc15bc25dbdecd333816f27d6639ba13cebd19
Parents: 61c9ffb
Author: devozerov <vo...@gridgain.com>
Authored: Mon Apr 2 11:35:20 2018 +0300
Committer: devozerov <vo...@gridgain.com>
Committed: Mon Apr 2 11:35:20 2018 +0300

----------------------------------------------------------------------
 modules/core/pom.xml                                        | 7 -------
 .../pagemem/RobinHoodBackwardShiftHashMapTest.java          | 9 ++-------
 2 files changed, 2 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ignite/blob/d8fc15bc/modules/core/pom.xml
----------------------------------------------------------------------
diff --git a/modules/core/pom.xml b/modules/core/pom.xml
index e56d141..68c19a7 100644
--- a/modules/core/pom.xml
+++ b/modules/core/pom.xml
@@ -124,13 +124,6 @@
         </dependency>
 
         <dependency>
-            <groupId>org.hamcrest</groupId>
-            <artifactId>hamcrest-library</artifactId>
-            <version>1.3</version>
-            <scope>test</scope>
-        </dependency>
-
-        <dependency>
             <groupId>org.mockito</groupId>
             <artifactId>mockito-all</artifactId>
             <version>1.9.5</version>

http://git-wip-us.apache.org/repos/asf/ignite/blob/d8fc15bc/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/RobinHoodBackwardShiftHashMapTest.java
----------------------------------------------------------------------
diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/RobinHoodBackwardShiftHashMapTest.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/RobinHoodBackwardShiftHashMapTest.java
index e594d06..565b99e 100644
--- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/RobinHoodBackwardShiftHashMapTest.java
+++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/RobinHoodBackwardShiftHashMapTest.java
@@ -26,15 +26,12 @@ import org.apache.ignite.internal.pagemem.FullPageId;
 import org.apache.ignite.internal.util.GridLongList;
 import org.apache.ignite.internal.util.GridUnsafe;
 import org.apache.ignite.internal.util.typedef.internal.U;
-import org.hamcrest.MatcherAssert;
-import org.hamcrest.Matchers;
 import org.jetbrains.annotations.NotNull;
 import org.junit.Test;
 
 import static org.apache.ignite.IgniteSystemProperties.IGNITE_LONG_LONG_HASH_MAP_LOAD_FACTOR;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.assertThat;
 import static org.junit.Assert.assertTrue;
 
 /**
@@ -419,14 +416,12 @@ public class RobinHoodBackwardShiftHashMapTest {
                 assertTrue(val > 0);
             }
 
-            assertThat(sz - map.size(), Matchers.is(list.size()));
+            assertEquals(sz - map.size(), list.size());
 
             Map<FullPageId, Long> res = new HashMap<>();
 
             map.forEach(res::put);
 
-            assertThat(map.size(), Matchers.is(map.size()));
-
             for (int i = 0; i < cap; i++) {
                 GridLongList list1 = map.removeIf(i, i + 1, pred);
 
@@ -435,7 +430,7 @@ public class RobinHoodBackwardShiftHashMapTest {
 
             check.keySet().removeIf(entry -> pred.test(entry.groupId(), entry.pageId()));
 
-            MatcherAssert.assertThat(res, Matchers.is(check));
+            assertEquals(res, check);
 
         }, cap);
     }