Repository: reef
Updated Branches:
  refs/heads/master 58a86f9b1 -> de6a9627b


[REEF-1151] Remove illformed `String.format` usage in 
TestClassHierarchyRoundTrip

JIRA:
  [REEF-1151](https://issues.apache.org/jira/browse/REEF-1151)

Pull Request:
  This closes #792


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

Branch: refs/heads/master
Commit: de6a9627b87675f7f41285e348ec65e7548a052d
Parents: 58a86f9
Author: Dongjoon Hyun <[email protected]>
Authored: Mon Jan 25 13:35:09 2016 -0800
Committer: Markus Weimer <[email protected]>
Committed: Thu Jan 28 09:59:17 2016 -0800

----------------------------------------------------------------------
 .../avro/TestClassHierarchyRoundTrip.java               | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/reef/blob/de6a9627/lang/java/reef-tang/tang/src/test/java/org/apache/reef/tang/implementation/avro/TestClassHierarchyRoundTrip.java
----------------------------------------------------------------------
diff --git 
a/lang/java/reef-tang/tang/src/test/java/org/apache/reef/tang/implementation/avro/TestClassHierarchyRoundTrip.java
 
b/lang/java/reef-tang/tang/src/test/java/org/apache/reef/tang/implementation/avro/TestClassHierarchyRoundTrip.java
index 376d066..57de1db 100644
--- 
a/lang/java/reef-tang/tang/src/test/java/org/apache/reef/tang/implementation/avro/TestClassHierarchyRoundTrip.java
+++ 
b/lang/java/reef-tang/tang/src/test/java/org/apache/reef/tang/implementation/avro/TestClassHierarchyRoundTrip.java
@@ -45,7 +45,8 @@ public class TestClassHierarchyRoundTrip extends 
TestClassHierarchy {
       ns = serializer.fromFile(file);
       file.delete();
     } catch (final IOException e) {
-      Assert.fail(String.format("IOException when serialize/deserialize the 
ClassHierarchy", e));
+      e.printStackTrace();
+      Assert.fail("IOException when serialize/deserialize the ClassHierarchy");
     }
   }
 
@@ -58,7 +59,8 @@ public class TestClassHierarchyRoundTrip extends 
TestClassHierarchy {
       ns = serializer.fromTextFile(textFile);
       textFile.delete();
     } catch (final IOException e) {
-      Assert.fail(String.format("IOException when serialize/deserialize the 
ClassHierarchy", e));
+      e.printStackTrace();
+      Assert.fail("IOException when serialize/deserialize the ClassHierarchy");
     }
   }
 
@@ -68,7 +70,8 @@ public class TestClassHierarchyRoundTrip extends 
TestClassHierarchy {
     try {
       ns = serializer.fromByteArray(serializer.toByteArray(ns));
     } catch (final IOException e) {
-      Assert.fail(String.format("IOException when serialize/deserialize the 
ClassHierarchy", e));
+      e.printStackTrace();
+      Assert.fail("IOException when serialize/deserialize the ClassHierarchy");
     }
   }
 
@@ -78,7 +81,8 @@ public class TestClassHierarchyRoundTrip extends 
TestClassHierarchy {
     try {
       ns = serializer.fromString(serializer.toString(ns));
     } catch (final IOException e) {
-      Assert.fail(String.format("IOException when serialize/deserialize the 
ClassHierarchy", e));
+      e.printStackTrace();
+      Assert.fail("IOException when serialize/deserialize the ClassHierarchy");
     }
   }
 

Reply via email to