Thanks Jacques and Deepak Nigam, this has been fixed at r#1768167.

Thanks & Regards
--
Deepak Dixit
www.hotwaxsystems.com

On Sat, Nov 5, 2016 at 2:38 PM, Deepak Nigam <[email protected]
> wrote:

> I think this issue should be fixed after correcting the sequence of test
> cases in the file AutoAcctgFinAccountTests.xml.
>
> Thanks & Regards
> --
> Deepak Nigam
> Sr. Enterprise Software Engineer
> Hotwax Systems
> www.hotwaxsystems.com
>
> On Fri, Nov 4, 2016 at 4:43 PM, Jacques Le Roux <
> [email protected]> wrote:
>
> > Ha no sorry, it's another error but not related to organizationPartyId
> >
> > The FinAccountRole data are OK
> >
> > The error is here https://ci.apache.org/projects/ofbiz/logs/trunk/html/
> >
> > Jacques
> >
> >
> >
> > Le 04/11/2016 à 11:56, Deepak Dixit a écrit :
> >
> >> Let me check Jacques.
> >>
> >> Thanks for review.
> >>
> >> Thanks & Regards
> >> --
> >> Deepak Dixit
> >> www.hotwaxsystems.com
> >>
> >> On Fri, Nov 4, 2016 at 4:22 PM, Jacques Le Roux <
> >> [email protected]> wrote:
> >>
> >> Hi Deepak,
> >>>
> >>> The testUpdateFinAccountRole test fails because the data are wrong the
> >>> FinAccountRole entity PKs are
> >>>
> >>> finAccountId (String)
> >>> fromDate (java.sql.Timestamp)
> >>> partyId (String)
> >>> roleTypeId (String)
> >>>
> >>> When the data are
> >>>
> >>>      <!-- For Testing service updateFinAccountRole -->
> >>>      <FinAccount finAccountId="1004" finAccountTypeId="DEPOSIT_
> ACCOUNT"
> >>> finAccountName="Deposit Account" finAccountCode="1001"
> >>> currencyUomId="USD"
> >>> organizationPartyId="DEMO_COMPANY" fromDate="2016-11-03 00:00:00"
> >>> actualBalance="2000.00" availableBalance="1500.00"/>
> >>>      <FinAccountRole finAccountId="1004" partyId="DEMO_COMPANY"
> >>> roleTypeId="SUPPLIER" fromDate="2016-11-03 00:00:00"/>
> >>>
> >>> At least organizationPartyId is wrong, should be partyId
> >>>
> >>> Thanks
> >>>
> >>> Jacques
> >>>
> >>>
> >>> Le 03/11/2016 à 22:03, [email protected] a écrit :
> >>>
> >>> Author: deepak
> >>>> Date: Thu Nov  3 21:03:13 2016
> >>>> New Revision: 1767974
> >>>>
> >>>> URL: http://svn.apache.org/viewvc?rev=1767974&view=rev
> >>>> Log:
> >>>> Improved: Added missing file for accounting fin service test cases.
> >>>>
> >>>> Added:
> >>>>       ofbiz/trunk/applications/accounting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>>   (with props)
> >>>>
> >>>> Added: ofbiz/trunk/applications/accounting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accoun
> >>>> ting/minilang/test/AutoAcctgFinAccountTests.xml?rev=1767974&view=auto
> >>>> ============================================================
> >>>> ==================
> >>>> --- ofbiz/trunk/applications/accounting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> (added)
> >>>> +++ ofbiz/trunk/applications/accounting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> Thu Nov  3 21:03:13 2016
> >>>> @@ -0,0 +1,182 @@
> >>>> +<?xml version="1.0" encoding="UTF-8"?>
> >>>> +<!--
> >>>> +Licensed to the Apache Software Foundation (ASF) under one
> >>>> +or more contributor license agreements.  See the NOTICE file
> >>>> +distributed with this work for additional information
> >>>> +regarding copyright ownership.  The ASF licenses this file
> >>>> +to you under the Apache License, Version 2.0 (the
> >>>> +"License"); you may not use this file except in compliance
> >>>> +with the License.  You may obtain a copy of the License at
> >>>> +
> >>>> +http://www.apache.org/licenses/LICENSE-2.0
> >>>> +
> >>>> +Unless required by applicable law or agreed to in writing,
> >>>> +software distributed under the License is distributed on an
> >>>> +"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> >>>> +KIND, either express or implied.  See the License for the
> >>>> +specific language governing permissions and limitations
> >>>> +under the License.
> >>>> +-->
> >>>> +
> >>>> +<simple-methods xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance
> "
> >>>> +        xmlns="http://ofbiz.apache.org/Simple-Method";
> >>>> xsi:schemaLocation="http://ofbiz.apache.org/Simple-Method
> >>>> http://ofbiz.apache.org/dtds/simple-methods.xsd";>
> >>>> +    <simple-method method-name="testDeleteFinAccountRole"
> >>>> short-description="Test case for service deleteFinAccountRole"
> >>>> login-required="false">
> >>>> +        <set field="serviceCtx.finAccountId" value="1004"/>
> >>>> +        <set field="serviceCtx.partyId" value="DEMO_COMPANY"/>
> >>>> +        <set field="serviceCtx.roleTypeId" value="SUPPLIER"/>
> >>>> +        <set field="serviceCtx.fromDate" value="2016-11-03 00:00:00"
> >>>> type="Timestamp"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="deleteFinAccountRole"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-and list="finAccountRoles"
> >>>> entity-name="FinAccountRole">
> >>>> +            <field-map field-name="finAccountId" value="1004"/>
> >>>> +            <field-map field-name="partyId" value="DEMO_COMPANY"/>
> >>>> +            <field-map field-name="roleTypeId" value="SUPPLIER"/>
> >>>> +        </entity-and>
> >>>> +        <first-from-list entry="finAccountRole"
> >>>> list="finAccountRoles"/>
> >>>> +        <assert>
> >>>> +            <if-empty field="finAccountRole"/>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testUpdateFinAccountRole"
> >>>> short-description="Test case for service updateFinAccountRole"
> >>>> login-required="false">
> >>>> +        <now-timestamp field="nowTimestamp"/>
> >>>> +        <set field="serviceCtx.finAccountId" value="1004"/>
> >>>> +        <set field="serviceCtx.partyId" value="DEMO_COMPANY"/>
> >>>> +        <set field="serviceCtx.roleTypeId" value="SUPPLIER"/>
> >>>> +        <set field="serviceCtx.fromDate" value="2016-11-03 00:00:00"
> >>>> type="Timestamp"/>
> >>>> +        <set field="serviceCtx.thruDate" from-field="nowTimestamp"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="updateFinAccountRole"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-and list="finAccountRoles"
> >>>> entity-name="FinAccountRole">
> >>>> +            <field-map field-name="finAccountId" value="1004"/>
> >>>> +            <field-map field-name="partyId" value="DEMO_COMPANY"/>
> >>>> +            <field-map field-name="roleTypeId" value="SUPPLIER"/>
> >>>> +        </entity-and>
> >>>> +        <first-from-list entry="finAccountRole"
> >>>> list="finAccountRoles"/>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccountRole"/></not>
> >>>> +            <not><if-empty field="finAccountRole.thruDate"/></not>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testCreateFinAccountRole"
> >>>> short-description="Test case for service createFinAccountRole"
> >>>> login-required="false">
> >>>> +        <now-timestamp field="nowTimestamp"/>
> >>>> +        <set field="serviceCtx.finAccountId" value="1003"/>
> >>>> +        <set field="serviceCtx.partyId" value="DEMO_COMPANY"/>
> >>>> +        <set field="serviceCtx.roleTypeId"
> >>>> value="INTERNAL_ORGANIZATIO"/>
> >>>> +        <set field="serviceCtx.fromDate" from-field="nowTimestamp"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="createFinAccountRole"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-and list="finAccountRoles"
> >>>> entity-name="FinAccountRole">
> >>>> +            <field-map field-name="finAccountId" value="1003"/>
> >>>> +            <field-map field-name="partyId" value="DEMO_COMPANY"/>
> >>>> +            <field-map field-name="roleTypeId"
> >>>> value="INTERNAL_ORGANIZATIO"/>
> >>>> +        </entity-and>
> >>>> +        <first-from-list entry="finAccountRole"
> >>>> list="finAccountRoles"/>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccountRole"/></not>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testCreateFinAccountTrans"
> >>>> short-description="Test case for service createFinAccountTrans"
> >>>> login-required="false">
> >>>> +        <set field="serviceCtx.finAccountId" value="1003"/>
> >>>> +        <set field="serviceCtx.finAccountTransTypeId"
> >>>> value="ADJUSTMENT"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="createFinAccountTrans"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-and list="finAccountTransList"
> >>>> entity-name="FinAccountTrans">
> >>>> +            <field-map field-name="finAccountId" value="1003"/>
> >>>> +        </entity-and>
> >>>> +        <first-from-list entry="finAccountTrans"
> >>>> list="finAccountTransList"/>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccountTrans"/></not>
> >>>> +            <if-compare operator="equals" value="ADJUSTMENT"
> >>>> field="finAccountTrans.finAccountTransTypeId"></if-compare>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testCreateFinAccountStatus"
> >>>> short-description="Test case for service createFinAccountStatus"
> >>>> login-required="false">
> >>>> +        <now-timestamp field="nowTimestamp"/>
> >>>> +        <set field="serviceCtx.finAccountId" value="1003"/>
> >>>> +        <set field="serviceCtx.statusId" value="FNACT_ACTIVE"/>
> >>>> +        <set field="serviceCtx.statusDate"
> from-field="nowTimestamp"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="createFinAccountStatus"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-and list="finAccountStatuses"
> >>>> entity-name="FinAccountStatus">
> >>>> +            <field-map field-name="finAccountId" value="1003"/>
> >>>> +        </entity-and>
> >>>> +        <first-from-list entry="finAccountStatus"
> >>>> list="finAccountStatuses"/>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccountStatus"/></not>
> >>>> +            <if-compare operator="equals" value="FNACT_ACTIVE"
> >>>> field="finAccountStatus.statusId"></if-compare>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testDeleteFinAccount"
> >>>> short-description="Test case for service deleteFinAccount"
> >>>> login-required="false">
> >>>> +        <set field="serviceCtx.finAccountId" value="1002"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="deleteFinAccount"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-one entity-name="FinAccount"
> value-field="finAccount">
> >>>> +            <field-map field-name="finAccountId" value="1002"/>
> >>>> +        </entity-one>
> >>>> +        <assert>
> >>>> +            <if-empty field="finAccount"/>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testUpdateFinAccount"
> >>>> short-description="Test case for service updateFinAccount"
> >>>> login-required="false">
> >>>> +        <set field="serviceCtx.finAccountId" value="1001"/>
> >>>> +        <set field="serviceCtx.organizationPartyId"
> >>>> value="DEMO_COMPANY2"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="updateFinAccount"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-one entity-name="FinAccount"
> value-field="finAccount">
> >>>> +            <field-map field-name="finAccountId" value="1001"/>
> >>>> +        </entity-one>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccount"/></not>
> >>>> +            <if-compare operator="equals" value="DEMO_COMPANY2"
> >>>> field="finAccount.organizationPartyId"></if-compare>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +    <simple-method method-name="testCreateFinAccount"
> >>>> short-description="Test case for service createFinAccount"
> >>>> login-required="false">
> >>>> +        <set field="serviceCtx.finAccountId" value="1000"/>
> >>>> +        <set field="serviceCtx.finAccountTypeId"
> >>>> value="BANK_ACCOUNT"/>
> >>>> +        <set field="serviceCtx.finAccountName" value="Bank Account"
> />
> >>>> +        <set field="serviceCtx.finAccountCode" value="1000"/>
> >>>> +        <set field="serviceCtx.currencyUomId" value="USD"/>
> >>>> +        <set field="serviceCtx.organizationPartyId"
> >>>> value="DEMO_COMPANY"/>
> >>>> +        <entity-one entity-name="UserLogin" value-field="userLogin">
> >>>> +            <field-map field-name="userLoginId" value="system"/>
> >>>> +        </entity-one>
> >>>> +        <set field="serviceCtx.userLogin" from-field="userLogin"/>
> >>>> +        <call-service service-name="createFinAccount"
> >>>> in-map-name="serviceCtx"/>
> >>>> +        <entity-one entity-name="FinAccount"
> value-field="finAccount">
> >>>> +            <field-map field-name="finAccountId" value="1000"/>
> >>>> +        </entity-one>
> >>>> +        <assert>
> >>>> +            <not><if-empty field="finAccount"/></not>
> >>>> +            <if-compare operator="equals" value="BANK_ACCOUNT"
> >>>> field="finAccount.finAccountTypeId"></if-compare>
> >>>> +            <if-compare operator="equals" value="1000"
> >>>> field="finAccount.finAccountCode"></if-compare>
> >>>> +        </assert>
> >>>> +        <check-errors/>
> >>>> +    </simple-method>
> >>>> +</simple-methods>
> >>>> \ No newline at end of file
> >>>>
> >>>> Propchange: ofbiz/trunk/applications/accou
> >>>> nting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> ------------------------------------------------------------
> >>>> ------------------
> >>>>       svn:eol-style = native
> >>>>
> >>>> Propchange: ofbiz/trunk/applications/accou
> >>>> nting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> ------------------------------------------------------------
> >>>> ------------------
> >>>>       svn:keywords = Date Rev Author URL Id
> >>>>
> >>>> Propchange: ofbiz/trunk/applications/accou
> >>>> nting/minilang/test/AutoAcctgF
> >>>> inAccountTests.xml
> >>>> ------------------------------------------------------------
> >>>> ------------------
> >>>>       svn:mime-type = text/xml
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >
>

Reply via email to