[ofbiz-plugins] branch trunk updated: Improved: Convert testSendOrderChangeNotification to Groovy

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[ofbiz-plugins] branch trunk updated: Improved: Convert testSendOrderChangeNotification to Groovy

jleroux@apache.org
This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-plugins.git


The following commit(s) were added to refs/heads/trunk by this push:
     new ba24629  Improved: Convert testSendOrderChangeNotification to Groovy
ba24629 is described below

commit ba246299d9612deaba90b9fbb364f7343672f9f5
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Mon Apr 27 18:47:12 2020 +0200

    Improved: Convert testSendOrderChangeNotification to Groovy
   
    (OFBIZ-11233)
   
    In the XML version we were sending note (Test note) and comments
    (Test Comments) as well in serviceCtx, but in the converted version we are
    not sending these fields.
   
    Thanks: Suraj for spotting it and alerting me on dev ML
---
 .../groovy/org/apache/ofbiz/ecommerce/OrderNotificationTests.groovy     | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ecommerce/src/main/groovy/org/apache/ofbiz/ecommerce/OrderNotificationTests.groovy b/ecommerce/src/main/groovy/org/apache/ofbiz/ecommerce/OrderNotificationTests.groovy
index 660bea7..403c8c5 100644
--- a/ecommerce/src/main/groovy/org/apache/ofbiz/ecommerce/OrderNotificationTests.groovy
+++ b/ecommerce/src/main/groovy/org/apache/ofbiz/ecommerce/OrderNotificationTests.groovy
@@ -40,6 +40,8 @@ class OrderNotificationTests extends OFBizTestCase {
     void testSendOrderChangeNotification() {
         Map serviceCtx = [
             orderId: 'TEST_DEMO10090',
+            note: 'Test Note',
+            comments: 'Test comments',
             sendTo: '[hidden email]',
             userLogin: userLogin
         ]