[asterixdb] branch master updated: [NO ISSUE] Print expected and actual result for jsonregex

[ht...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

htowaileb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git


The following commit(s) were added to refs/heads/master by this push:
     new 8b0eaf6  [NO ISSUE] Print expected and actual result for jsonregex
8b0eaf6 is described below

commit 8b0eaf621c7d68c305dc79dcd2508e28ca88c732
Author: Hussain Towaileb <Hu...@Couchbase.com>
AuthorDate: Tue Jul 9 19:40:38 2019 +0300

    [NO ISSUE] Print expected and actual result for jsonregex
    
    - user model changes: no
    - storage format changes: no
    - interface changes: no
    
    Details:
    - Print both the expected and the actual result from the test
    when a jsonregex comparison fails.
    
    Change-Id: I46a1c734cc01672ed89224b42c6db148e9935b87
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/3483
    Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
    Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
    Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
    Reviewed-by: Till Westmann <ti...@apache.org>
---
 .../src/test/java/org/apache/asterix/test/common/TestExecutor.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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 220ec38..344a3ba 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
@@ -260,8 +260,8 @@ public class TestExecutor {
                     throw new NullPointerException("Error parsing expected or actual result file for " + scriptFile);
                 }
                 if (!TestHelper.equalJson(expectedJson, actualJson)) {
-                    throw new ComparisonException(
-                            "Result for " + scriptFile + " didn't match the expected JSON" + "\n" + actualJson);
+                    throw new ComparisonException("Result for " + scriptFile + " didn't match the expected JSON"
+                            + "\nexpected result:\n" + expectedJson + "\nactual result:\n" + actualJson);
                 }
                 return;
             }