Updated Branches: refs/heads/master 17ec15584 -> 22a33dd56
Remove STDOUT logging in integration tests Project: http://git-wip-us.apache.org/repos/asf/incubator-crunch/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-crunch/commit/22a33dd5 Tree: http://git-wip-us.apache.org/repos/asf/incubator-crunch/tree/22a33dd5 Diff: http://git-wip-us.apache.org/repos/asf/incubator-crunch/diff/22a33dd5 Branch: refs/heads/master Commit: 22a33dd56e1486a7734d10a462a398cbe8a27feb Parents: 17ec155 Author: Gabriel Reid <[email protected]> Authored: Thu Sep 20 15:30:43 2012 +0200 Committer: Gabriel Reid <[email protected]> Committed: Thu Sep 20 15:30:43 2012 +0200 ---------------------------------------------------------------------- .../it/java/org/apache/crunch/MapPObjectIT.java | 8 ++------ .../java/org/apache/crunch/MaterializeToMapIT.java | 5 ++--- 2 files changed, 4 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-crunch/blob/22a33dd5/crunch/src/it/java/org/apache/crunch/MapPObjectIT.java ---------------------------------------------------------------------- diff --git a/crunch/src/it/java/org/apache/crunch/MapPObjectIT.java b/crunch/src/it/java/org/apache/crunch/MapPObjectIT.java index 832b35c..c48284f 100644 --- a/crunch/src/it/java/org/apache/crunch/MapPObjectIT.java +++ b/crunch/src/it/java/org/apache/crunch/MapPObjectIT.java @@ -17,14 +17,11 @@ */ package org.apache.crunch; -import static junit.framework.Assert.assertTrue; +import static junit.framework.Assert.assertEquals; import java.io.IOException; -import java.lang.Integer; import java.util.Map; -import org.apache.crunch.PObject; -import org.apache.crunch.PTable; import org.apache.crunch.impl.mem.MemPipeline; import org.apache.crunch.impl.mr.MRPipeline; import org.apache.crunch.materialize.pobject.MapPObject; @@ -43,8 +40,7 @@ public class MapPObjectIT { public void assertMatches(Map<Integer, String> m) { for (Integer k : m.keySet()) { - System.out.println(k + " " + kvPairs.get(k).second() + " " + m.get(k)); - assertTrue(kvPairs.get(k).second().equals(m.get(k))); + assertEquals(kvPairs.get(k).second(), m.get(k)); } } http://git-wip-us.apache.org/repos/asf/incubator-crunch/blob/22a33dd5/crunch/src/it/java/org/apache/crunch/MaterializeToMapIT.java ---------------------------------------------------------------------- diff --git a/crunch/src/it/java/org/apache/crunch/MaterializeToMapIT.java b/crunch/src/it/java/org/apache/crunch/MaterializeToMapIT.java index f038306..7fef30e 100644 --- a/crunch/src/it/java/org/apache/crunch/MaterializeToMapIT.java +++ b/crunch/src/it/java/org/apache/crunch/MaterializeToMapIT.java @@ -17,7 +17,7 @@ */ package org.apache.crunch; -import static junit.framework.Assert.assertTrue; +import static junit.framework.Assert.assertEquals; import java.io.IOException; import java.util.Map; @@ -39,8 +39,7 @@ public class MaterializeToMapIT { public void assertMatches(Map<Integer, String> m) { for (Integer k : m.keySet()) { - System.out.println(k + " " + kvPairs.get(k).second() + " " + m.get(k)); - assertTrue(kvPairs.get(k).second().equals(m.get(k))); + assertEquals(kvPairs.get(k).second(), m.get(k)); } }
