Thanks Deepak,

This has indeed been fixed

Jacques


Le 05/11/2016 à 10:21, Deepak Dixit a écrit :
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 <deepak.ni...@hotwaxsystems.com
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 <
jacques.le.r...@les7arts.com> 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 <
jacques.le.r...@les7arts.com> 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, dee...@apache.org 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