asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Blow (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Clarify failure message on extra actual data
Date Sat, 10 Jun 2017 21:12:58 GMT
Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1829

Change subject: Clarify failure message on extra actual data
......................................................................

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
---
M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/29/1829/1

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 @@
             }
             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());

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1829
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I948a5083eea8d50bc359971909cd08c94e30a233
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mblow@apache.org>

Mime
View raw message