You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by mb...@apache.org on 2017/06/11 00:24:15 UTC

asterixdb git commit: Clarify failure message on extra actual data

Repository: asterixdb
Updated Branches:
  refs/heads/master fac769e5c -> b8ad82581


Clarify failure message on extra actual data

Make exception message more obvious when we expect an <EOF> and more data is
present, not that the actual result was an unexpected <EOF>

Change-Id: I948a5083eea8d50bc359971909cd08c94e30a233
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1829
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
BAD: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Till Westmann <ti...@apache.org>


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

Branch: refs/heads/master
Commit: b8ad825811ccc20aa0145de6dc258d334c4f0ad4
Parents: fac769e
Author: Michael Blow <mb...@apache.org>
Authored: Sat Jun 10 17:11:17 2017 -0400
Committer: Michael Blow <mb...@apache.org>
Committed: Sat Jun 10 17:23:58 2017 -0700

----------------------------------------------------------------------
 .../src/test/java/org/apache/asterix/test/common/TestExecutor.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/b8ad8258/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
index c8c8aec..25b9f0a 100644
--- a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
+++ b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
@@ -224,7 +224,7 @@ public class TestExecutor {
             }
             lineActual = readerActual.readLine();
             if (lineActual != null) {
-                throw createLineChangedException(scriptFile, "<EOF>", lineActual, num);
+                throw createLineChangedException(scriptFile, "expected <EOF>", lineActual, num);
             }
         } catch (Exception e) {
             System.err.println("Actual results file: " + actualFile.toString());