You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by pa...@apache.org on 2020/09/10 09:16:35 UTC

[shardingsphere] branch master updated: #6872, add log for BatchIT

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

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


The following commit(s) were added to refs/heads/master by this push:
     new a3c00e1  #6872, add log for BatchIT
     new 1736a62  Merge pull request #7382 from tuohai666/add_log
a3c00e1 is described below

commit a3c00e1707ed63eb65b88c51d742a968dc3cb583
Author: Zhang Yonglun <zh...@apache.org>
AuthorDate: Thu Sep 10 16:57:59 2020 +0800

    #6872, add log for BatchIT
---
 .../test/java/org/apache/shardingsphere/dbtest/engine/BatchIT.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/engine/BatchIT.java b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/engine/BatchIT.java
index 35b424c..83f2930 100644
--- a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/engine/BatchIT.java
+++ b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/engine/BatchIT.java
@@ -19,6 +19,7 @@ package org.apache.shardingsphere.dbtest.engine;
 
 import lombok.AccessLevel;
 import lombok.Getter;
+import lombok.extern.slf4j.Slf4j;
 import org.apache.shardingsphere.dbtest.cases.assertion.root.IntegrateTestCase;
 import org.apache.shardingsphere.dbtest.cases.assertion.root.IntegrateTestCaseAssertion;
 import org.apache.shardingsphere.dbtest.cases.dataset.DataSet;
@@ -57,6 +58,7 @@ import java.util.Set;
 import static org.hamcrest.CoreMatchers.is;
 import static org.junit.Assert.assertThat;
 
+@Slf4j
 @Getter(AccessLevel.PROTECTED)
 public abstract class BatchIT extends BaseIT {
     
@@ -109,6 +111,9 @@ public abstract class BatchIT extends BaseIT {
                 DataSet expected = (DataSet) JAXBContext.newInstance(DataSet.class).createUnmarshaller().unmarshal(reader);
                 assertThat(actualUpdateCounts[count], is(expected.getUpdateCount()));
                 expectedList.add(expected);
+            } catch (final AssertionError ex) {
+                log.error("[ERROR] SQL::{}, Expect::{}", getSql(), each);
+                throw ex;
             }
             count++;
         }