When move to tycho build, i saw code for test mix on some plugins
it's better separate it, so when build product it don't have test code
test bundle: org.adempiere.extend org.adempiere.payment.processor org.idempiere.fitnesse.* org.idempiere.ui.zk.selenium
bundle contain class for test bundle org.adempiere.base have class EMailTest, MRoleTest, AccessSqlParserTest,.... bundle org.adempiere.ui.zk have class on package org.adempiere.webui.component.test bundle org.compiere.db.postgresql.provider have class Convert_PostgreSQLTest
About payment.processor - not sure about that - probably there are people using it.
library look quite old, maybe out date already. the people use it need to provide update version.
Carlos Ruiz March 22, 2019 at 2:17 PM
Bringing comment from ________ BTW, I think the extend plugin could be deleted - or at least disconnected by default from core and just keep it like the ui.zk.example plugin.
This would be about deleting the callout registered in column Test.Integer and disabling that plugin.
WDYT? ________
So, I think is OK to remove extend with that migration scipt.
Now, about fitnesse and selenium, yes, definitely they don't need to be released with server, neither in installers. I think they are not, but if they are we can remove it.
About payment.processor - not sure about that - probably there are people using it.
When move to tycho build, i saw code for test mix on some plugins
it's better separate it, so when build product it don't have test code
test bundle:
org.adempiere.extend
org.adempiere.payment.processor
org.idempiere.fitnesse.*
org.idempiere.ui.zk.selenium
bundle contain class for test
bundle org.adempiere.base have class EMailTest, MRoleTest, AccessSqlParserTest,....
bundle org.adempiere.ui.zk have class on package org.adempiere.webui.component.test
bundle org.compiere.db.postgresql.provider have class Convert_PostgreSQLTest