You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2020/04/27 16:47:17 UTC

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

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 <ja...@les7arts.com>
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: 'test_email@example.com',
             userLogin: userLogin
         ]