Thanks Jacques for taking care of it. -- Best Regards, Suraj Khurana Senior Technical Consultant
On Mon, Apr 27, 2020 at 9:43 PM Jacques Le Roux < jacques.le.r...@les7arts.com> wrote: > I don't remember if it was intentional, I think I missed them and will add > them "right now" > > Thanks > > Le 27/04/2020 à 12:36, Suraj Khurana a écrit : > > I meant, 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. > > > > Tests will pass for sure, as they are not mandatory to send. > > > > -- > > Best Regards, > > Suraj Khurana > > Senior Technical Consultant > > > > > > On Mon, Apr 27, 2020 at 2:52 PM Jacques Le Roux < > > jacques.le.r...@les7arts.com> wrote: > > > >> Hi Suraj, > >> > >> If you mean that there is no explanation but "Tests pass", as it's Svn > we > >> can still amend it, what would you suggest? > >> > >> Thanks > >> > >> Jacques > >> > >> Le 27/04/2020 à 08:47, Suraj Khurana a écrit : > >>> Hello Jacques, > >>> > >>> I think we missed the note, comments field during conversion in this > >> commit. > >>> Is it intentional, or am I missing something? > >>> > >>> -- > >>> Best Regards, > >>> Suraj Khurana > >>> Senior Technical Consultant > >>> > >>> On Thu, Oct 3, 2019 at 5:00 PM <jler...@apache.org> wrote: > >>> > >>>> Author: jleroux > >>>> Date: Thu Oct 3 11:30:28 2019 > >>>> New Revision: 1867927 > >>>> > >>>> URL: http://svn.apache.org/viewvc?rev=1867927&view=rev > >>>> Log: > >>>> Improved: Convert testSendOrderChangeNotification to Groovy > >>>> (OFBIZ-11233) > >>>> > >>>> Tests pass > >>>> > >>>> Modified: > >>>> > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/minilang/test/OrderTests.xml > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/src/main/groovy/org/apache/ofbiz/order/OrderTests.groovy > >>>> Modified: > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/minilang/test/OrderTests.xml > >>>> URL: > >>>> > >> > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/minilang/test/OrderTests.xml?rev=1867927&r1=1867926&r2=1867927&view=diff > >>>> > >> > ============================================================================== > >>>> --- > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/minilang/test/OrderTests.xml > >>>> (original) > >>>> +++ > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/minilang/test/OrderTests.xml > >>>> Thu Oct 3 11:30:28 2019 > >>>> @@ -154,23 +154,4 @@ under the License. > >>>> </assert> > >>>> <check-errors/> > >>>> </simple-method> > >>>> - > >>>> - <simple-method method-name="testSendOrderChangeNotification" > >>>> short-description="Sends order change confirmation mail" > >>>> login-required="false"> > >>>> - <entity-one entity-name="UserLogin" value-field="userLogin"> > >>>> - <field-map field-name="userLoginId" value="system"/> > >>>> - </entity-one> > >>>> - <set-current-user-login value-field="userLogin"/> > >>>> - <set field="serviceCtx.orderId" value="TEST_DEMO10090"/> > >>>> - <set field="serviceCtx.note" value="Test Note"/> > >>>> - <set field="serviceCtx.comments" value="Test comments"/> > >>>> - <set field="serviceCtx.sendTo" value=" > >> newtest_em...@example.com > >>>> "/> > >>>> - <call-service service-name="sendOrderChangeNotification" > >>>> in-map-name="serviceCtx"> > >>>> - <results-to-map map-name="serviceResult"/> > >>>> - </call-service> > >>>> - <assert> > >>>> - <not><if-empty field="serviceResult"/></not> > >>>> - <if-compare operator="equals" > >> field="serviceResult.emailType" > >>>> value="PRDS_ODR_CHANGE" /> > >>>> - </assert> > >>>> - <check-errors/> > >>>> - </simple-method> > >>>> </simple-methods> > >>>> \ No newline at end of file > >>>> > >>>> Modified: > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/src/main/groovy/org/apache/ofbiz/order/OrderTests.groovy > >>>> URL: > >>>> > >> > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/src/main/groovy/org/apache/ofbiz/order/OrderTests.groovy?rev=1867927&r1=1867926&r2=1867927&view=diff > >>>> > >> > ============================================================================== > >>>> --- > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/src/main/groovy/org/apache/ofbiz/order/OrderTests.groovy > >>>> (original) > >>>> +++ > >>>> > >> > ofbiz/ofbiz-framework/trunk/applications/order/src/main/groovy/org/apache/ofbiz/order/OrderTests.groovy > >>>> Thu Oct 3 11:30:28 2019 > >>>> @@ -212,4 +212,17 @@ class OrderTests extends OFBizTestCase { > >>>> Map serviceResult = > >>>> dispatcher.runSync('associatedRequirementWithRequestItem', serviceCtx) > >>>> assert ServiceUtil.isSuccess(serviceResult) > >>>> } > >>>> + > >>>> + void testSendOrderChangeNotification() { > >>>> + Map serviceCtx = [ > >>>> + orderId: 'TEST_DEMO10090', > >>>> + sendTo: 'test_em...@example.com', > >>>> + userLogin: > >>>> EntityQuery.use(delegator).from('UserLogin').where('userLoginId', > >>>> 'system').cache().queryOne() > >>>> + ] > >>>> + Map serviceResult = > >>>> dispatcher.runSync('sendOrderChangeNotification', serviceCtx) > >>>> + assert ServiceUtil.isSuccess(serviceResult) > >>>> + assert serviceResult.emailType.equals("PRDS_ODR_CHANGE") > >>>> + } > >>>> + > >>>> + > >>>> } > >>>> > >>>> > >>>> >