[incubator-servicecomb-saga] branch SCB-784 updated: SCB-784 Fixed the acceptance test error

[ningjiang]

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

ningjiang pushed a commit to branch SCB-784
in repository https://gitbox.apache.org/repos/asf/incubator-servicecomb-saga.git


The following commit(s) were added to refs/heads/SCB-784 by this push:
     new 735fdf2  SCB-784 Fixed the acceptance test error
735fdf2 is described below

commit 735fdf2b14a37e63491dcb0c440955a707a81592
Author: Willem Jiang <jiangni...@huawei.com>
AuthorDate: Mon Jul 30 11:23:32 2018 +0800

    SCB-784 Fixed the acceptance test error
---
 .../src/test/java/org/apache/servicecomb/saga/PackStepdefs.java     | 6 +++++-
 .../src/test/resources/pack_compensation_scenario.feature           | 4 ++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git 
a/acceptance-tests/acceptance-pack-spring-demo/src/test/java/org/apache/servicecomb/saga/PackStepdefs.java
 
b/acceptance-tests/acceptance-pack-spring-demo/src/test/java/org/apache/servicecomb/saga/PackStepdefs.java
index 5488f0c..19651e7 100644
--- 
a/acceptance-tests/acceptance-pack-spring-demo/src/test/java/org/apache/servicecomb/saga/PackStepdefs.java
+++ 
b/acceptance-tests/acceptance-pack-spring-demo/src/test/java/org/apache/servicecomb/saga/PackStepdefs.java
@@ -148,7 +148,11 @@ public class PackStepdefs implements En {
     await().atMost(5, SECONDS).until(() -> {
       actualMaps.clear();
       Collections.addAll(actualMaps, retrieveDataMaps(address, dataProcessor));
-
+      // write the log if the Map size is not same
+      boolean result = expectedMaps.size() == actualMaps.size();
+      if (!result) {
+        LOG.warn("The response message size is not we expected. ExpectedMap 
size is {},  ActualMap size is {}", expectedMaps.size(), actualMaps.size());
+      }
       return expectedMaps.size() == actualMaps.size();
     });
 
diff --git 
a/acceptance-tests/acceptance-pack-spring-demo/src/test/resources/pack_compensation_scenario.feature
 
b/acceptance-tests/acceptance-pack-spring-demo/src/test/resources/pack_compensation_scenario.feature
index 931d316..b88db29 100644
--- 
a/acceptance-tests/acceptance-pack-spring-demo/src/test/resources/pack_compensation_scenario.feature
+++ 
b/acceptance-tests/acceptance-pack-spring-demo/src/test/resources/pack_compensation_scenario.feature
@@ -32,8 +32,8 @@ Feature: Alpha records transaction events
       | hotel   | TxAbortedEvent     |
       | booking | TxAbortedEvent     |
       | car     | TxCompensatedEvent |
-     
-
+      | car     | SagaEndedEvent     |
+    
     Then Car Service contains the following booking orders
       | name | amount | confirmed | cancelled |
       | Sean | 5      | false     | true      |