Hi David,
Currently the blockingTestScv is set to fail due to a transaction timeout (timeout is 20 sec, the service blocks for 30 sec), so using it in an eca for testScv is causing one of the service engine unit tests to fail. I can either revert that portion of the commit below or change the 20 sec timeout, any preference? Thanks Scott Snipped commit log: 2008/5/19 <[hidden email]>: > Author: jonesde > Date: Sun May 18 17:22:46 2008 > New Revision: 657701 > > URL: http://svn.apache.org/viewvc?rev=657701&view=rev > Log: > Added support for groovy services, and an example of the same; cleaned up > the way that bsh services work and updated the example and such to match, > now returns a context as one would expect > > Added: > ofbiz/trunk/framework/common/script/org/ofbiz/common/BshServiceTest.bsh > - copied, changed from r657609, > ofbiz/trunk/framework/common/src/org/ofbiz/common/BshServiceTest.bsh > > ofbiz/trunk/framework/common/script/org/ofbiz/common/GroovyServiceTest.groovy > > ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/GroovyEngine.java > (with props) > Removed: > ofbiz/trunk/framework/common/src/org/ofbiz/common/BshServiceTest.bsh > Modified: > ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/GroovyUtil.java > ofbiz/trunk/framework/common/servicedef/secas_test.xml > ofbiz/trunk/framework/common/servicedef/services_test.xml > ofbiz/trunk/framework/service/config/serviceengine.xml > > ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/BeanShellEngine.java > > > Modified: ofbiz/trunk/framework/common/servicedef/secas_test.xml > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/servicedef/secas_test.xml?rev=657701&r1=657700&r2=657701&view=diff > > ============================================================================== > --- ofbiz/trunk/framework/common/servicedef/secas_test.xml (original) > +++ ofbiz/trunk/framework/common/servicedef/secas_test.xml Sun May 18 > 17:22:46 2008 > @@ -28,7 +28,7 @@ > <condition field-name="message" operator="equals" value="12345"/> > <action service="testBsh" mode="sync"/> > </eca> > - <eca service="testBsh" event="commit"> > + <eca service="testScv" event="commit"> > <condition-service service-name="conditionReturnTrue"/> > <action service="blockingTestScv" mode="sync"/> > </eca> > > |
Let's just kill the ECA rule that calls it. I'm not sure what it was originally intended for anyway, and I just moved it away from the service I wanted to test (the bsh test service) so I could test it. It is supposed to cause an error, so we should probably just comment out the ECA and let it be called directly only for now... -David On May 24, 2008, at 4:23 AM, Scott Gray wrote: > Hi David, > > Currently the blockingTestScv is set to fail due to a transaction > timeout > (timeout is 20 sec, the service blocks for 30 sec), so using it in > an eca > for testScv is causing one of the service engine unit tests to > fail. I can > either revert that portion of the commit below or change the 20 sec > timeout, > any preference? > > Thanks > Scott > > Snipped commit log: > 2008/5/19 <[hidden email]>: > >> Author: jonesde >> Date: Sun May 18 17:22:46 2008 >> New Revision: 657701 >> >> URL: http://svn.apache.org/viewvc?rev=657701&view=rev >> Log: >> Added support for groovy services, and an example of the same; >> cleaned up >> the way that bsh services work and updated the example and such to >> match, >> now returns a context as one would expect >> >> Added: >> ofbiz/trunk/framework/common/script/org/ofbiz/common/ >> BshServiceTest.bsh >> - copied, changed from r657609, >> ofbiz/trunk/framework/common/src/org/ofbiz/common/BshServiceTest.bsh >> >> ofbiz/trunk/framework/common/script/org/ofbiz/common/ >> GroovyServiceTest.groovy >> >> ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/ >> GroovyEngine.java >> (with props) >> Removed: >> ofbiz/trunk/framework/common/src/org/ofbiz/common/ >> BshServiceTest.bsh >> Modified: >> ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ >> GroovyUtil.java >> ofbiz/trunk/framework/common/servicedef/secas_test.xml >> ofbiz/trunk/framework/common/servicedef/services_test.xml >> ofbiz/trunk/framework/service/config/serviceengine.xml >> >> ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/ >> BeanShellEngine.java >> >> >> Modified: ofbiz/trunk/framework/common/servicedef/secas_test.xml >> URL: >> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/servicedef/secas_test.xml?rev=657701&r1=657700&r2=657701&view=diff >> >> = >> = >> = >> = >> = >> = >> = >> = >> = >> ===================================================================== >> --- ofbiz/trunk/framework/common/servicedef/secas_test.xml (original) >> +++ ofbiz/trunk/framework/common/servicedef/secas_test.xml Sun May 18 >> 17:22:46 2008 >> @@ -28,7 +28,7 @@ >> <condition field-name="message" operator="equals" >> value="12345"/> >> <action service="testBsh" mode="sync"/> >> </eca> >> - <eca service="testBsh" event="commit"> >> + <eca service="testScv" event="commit"> >> <condition-service service-name="conditionReturnTrue"/> >> <action service="blockingTestScv" mode="sync"/> >> </eca> >> >> |
Thanks David, it's gone in rev. 659789
Regards Scott 2008/5/24 David E Jones <[hidden email]>: > > Let's just kill the ECA rule that calls it. I'm not sure what it was > originally intended for anyway, and I just moved it away from the service I > wanted to test (the bsh test service) so I could test it. > > It is supposed to cause an error, so we should probably just comment out > the ECA and let it be called directly only for now... > > -David > > > > > On May 24, 2008, at 4:23 AM, Scott Gray wrote: > > Hi David, >> >> Currently the blockingTestScv is set to fail due to a transaction timeout >> (timeout is 20 sec, the service blocks for 30 sec), so using it in an eca >> for testScv is causing one of the service engine unit tests to fail. I >> can >> either revert that portion of the commit below or change the 20 sec >> timeout, >> any preference? >> >> Thanks >> Scott >> >> Snipped commit log: >> 2008/5/19 <[hidden email]>: >> >> Author: jonesde >>> Date: Sun May 18 17:22:46 2008 >>> New Revision: 657701 >>> >>> URL: http://svn.apache.org/viewvc?rev=657701&view=rev >>> Log: >>> Added support for groovy services, and an example of the same; cleaned up >>> the way that bsh services work and updated the example and such to match, >>> now returns a context as one would expect >>> >>> Added: >>> ofbiz/trunk/framework/common/script/org/ofbiz/common/BshServiceTest.bsh >>> - copied, changed from r657609, >>> ofbiz/trunk/framework/common/src/org/ofbiz/common/BshServiceTest.bsh >>> >>> >>> ofbiz/trunk/framework/common/script/org/ofbiz/common/GroovyServiceTest.groovy >>> >>> >>> ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/GroovyEngine.java >>> (with props) >>> Removed: >>> ofbiz/trunk/framework/common/src/org/ofbiz/common/BshServiceTest.bsh >>> Modified: >>> ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/GroovyUtil.java >>> ofbiz/trunk/framework/common/servicedef/secas_test.xml >>> ofbiz/trunk/framework/common/servicedef/services_test.xml >>> ofbiz/trunk/framework/service/config/serviceengine.xml >>> >>> >>> ofbiz/trunk/framework/service/src/org/ofbiz/service/engine/BeanShellEngine.java >>> >>> >>> Modified: ofbiz/trunk/framework/common/servicedef/secas_test.xml >>> URL: >>> >>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/servicedef/secas_test.xml?rev=657701&r1=657700&r2=657701&view=diff >>> >>> >>> ============================================================================== >>> --- ofbiz/trunk/framework/common/servicedef/secas_test.xml (original) >>> +++ ofbiz/trunk/framework/common/servicedef/secas_test.xml Sun May 18 >>> 17:22:46 2008 >>> @@ -28,7 +28,7 @@ >>> <condition field-name="message" operator="equals" value="12345"/> >>> <action service="testBsh" mode="sync"/> >>> </eca> >>> - <eca service="testBsh" event="commit"> >>> + <eca service="testScv" event="commit"> >>> <condition-service service-name="conditionReturnTrue"/> >>> <action service="blockingTestScv" mode="sync"/> >>> </eca> >>> >>> >>> > |
Free forum by Nabble | Edit this page |