MAPREDUCE-7102. Fix TestJavaSerialization for Windows due a mismatch line separator. Contributed by Giovanni Matteo Fumarola.

Author: Inigo Goiri

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/TestJavaSerialization.java

@@ -91,6 +91,8 @@ private void cleanAndCreateInput(FileSystem fs) throws IOException {
     wr.write("b a\n");
     wr.close();
   }
+
+  @SuppressWarnings("deprecation")
   @Test
   public void testMapReduceJob() throws Exception {
 
@@ -131,16 +133,17 @@ public void testMapReduceJob() throws Exception {
         FileUtil.stat2Paths(fs.listStatus(OUTPUT_DIR,
           new Utils.OutputFileUtils.OutputFilesFilter()));
     assertEquals(1, outputFiles.length);
-    InputStream is = fs.open(outputFiles[0]);
-    String reduceOutput = org.apache.commons.io.IOUtils.toString(is);
-    String[] lines = reduceOutput.split(System.getProperty("line.separator"));
-    assertEquals("Unexpected output; received output '" + reduceOutput + "'",
-      "a\t1", lines[0]);
-    assertEquals("Unexpected output; received output '" + reduceOutput + "'",
-      "b\t1", lines[1]);
-    assertEquals("Reduce output has extra lines; output is '" + reduceOutput
-        + "'", 2, lines.length);
-    is.close();
+    try (InputStream is = fs.open(outputFiles[0])) {
+      String reduceOutput = org.apache.commons.io.IOUtils.toString(is);
+      String[] lines = reduceOutput.split("\n");
+      assertEquals("Unexpected output; received output '" + reduceOutput + "'",
+          "a\t1", lines[0]);
+      assertEquals("Unexpected output; received output '" + reduceOutput + "'",
+          "b\t1", lines[1]);
+      assertEquals(
+          "Reduce output has extra lines; output is '" + reduceOutput + "'", 2,
+          lines.length);
+    }
   }
 
   /**