Repository: sqoop Updated Branches: refs/heads/sqoop2 7fff9ae90 -> d43dc1b0b
SQOOP-2077: Sqoop2: Enclose missing/extra lines in output of HdfsAsserts.assertMapreduceOutput (Jarek Jarcec Cecho via Abraham Elmahrek) Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/d43dc1b0 Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/d43dc1b0 Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/d43dc1b0 Branch: refs/heads/sqoop2 Commit: d43dc1b0b54b66e6e42236c0585554dd3e98427e Parents: 7fff9ae Author: Abraham Elmahrek <[email protected]> Authored: Thu Feb 5 15:17:30 2015 -0800 Committer: Abraham Elmahrek <[email protected]> Committed: Thu Feb 5 15:17:30 2015 -0800 ---------------------------------------------------------------------- .../main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sqoop/blob/d43dc1b0/test/src/main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java ---------------------------------------------------------------------- diff --git a/test/src/main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java b/test/src/main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java index adc638a..b115723 100644 --- a/test/src/main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java +++ b/test/src/main/java/org/apache/sqoop/test/asserts/HdfsAsserts.java @@ -67,10 +67,11 @@ public class HdfsAsserts { } if(!setLines.isEmpty() || !notFound.isEmpty()) { + LOG.error("Output do not match expectations."); LOG.error("Expected lines that weren't present in the files:"); - LOG.error("\t" + StringUtils.join(setLines, "\n\t")); + LOG.error("\t'" + StringUtils.join(setLines, "'\n\t'") + "'"); LOG.error("Extra lines in files that weren't expected:"); - LOG.error("\t" + StringUtils.join(notFound, "\n\t")); + LOG.error("\t'" + StringUtils.join(notFound, "'\n\t'") + "'"); fail("Output do not match expectations."); } }
