You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@crunch.apache.org by gr...@apache.org on 2012/09/20 15:33:32 UTC
git commit: Remove STDOUT logging in integration tests
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 <gr...@apache.org>
Authored: Thu Sep 20 15:30:43 2012 +0200
Committer: Gabriel Reid <gr...@apache.org>
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));
}
}