[DISCUSSION] Integration tests instability causes by Ids collision (AUTO generation strategy)

2013-02-02 Thread Andrei Shakirin
Hi, Analyses shows, that real reason of failed/unstable tests is not AUTO generation strategy itself as reported in [1], but collision between newly created and pre-configured entries. Actually following entities have AUTO generation strategy: - AbstractDerAttr - AbstractVirAttr - Notification -

[jira] [Resolved] (SYNCOPE-289) Prepare CXF Rest integration tests migration

2013-02-02 Thread Andrei Shakirin (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-289?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrei Shakirin resolved SYNCOPE-289. - Resolution: Fixed > Prepare CXF Rest integration tests migration > -

Re: [DISCUSSION] Integration tests instability causes by Ids collision (AUTO generation strategy)

2013-02-02 Thread Francesco Chicchiriccò
On 02/02/2013 10:10, Andrei Shakirin wrote: Hi, Analyses shows, that real reason of failed/unstable tests is not AUTO generation strategy itself as reported in [1], but collision between newly created and pre-configured entries. Actually following entities have AUTO generation strategy: - Abst

Re: IncompatiblePolicyException was made unchecked

2013-02-02 Thread Francesco Chicchiriccò
On 01/02/2013 17:44, Andrei Shakirin wrote: Can we agree on something here? My +1 for checked +1 from me as well. Regards. -- Francesco Chicchiriccò ASF Member, Apache Syncope PMC chair, Apache Cocoon PMC Member http://people.apache.org/~ilgrosso/

RE: [DISCUSSION] Integration tests instability causes by Ids collision (AUTO generation strategy)

2013-02-02 Thread Andrei Shakirin
Hi Francesco, > I agree with you that (A) is easy and sounds like a workaround but... > we're talking about test data, hence workarounds don't smell like as usual :-) > IMO, relying upon some default test data is actually very comfortable, so I > wouldn't opt for (B). Hmm ... for sure quality of

Re: IncompatiblePolicyException was made unchecked

2013-02-02 Thread Christian Schneider
Basically I prefer unchecked but this would rather be a general change to not use checked exceptions at all. I do not insist on this style though and will go with the majority. Christian Am 02.02.2013 11:26, schrieb Francesco Chicchiriccò: On 01/02/2013 17:44, Andrei Shakirin wrote: Can we ag

[Report] CXF migration status

2013-02-02 Thread Andrei Shakirin
Hi, Would like to shortly report status of CXF migration: It is basically finished. We still have 2 Failures and 4 Errors in JAXRS tests - it should be fixed in next days (the reason of two is already clear). Still open for 1.1.0 (SYNCOPE -231): 1) Fix rest errors in tests (6) 2) Fi

[jira] [Closed] (SYNCOPE-298) Persistence beans: change AUTO Id generation strategy to TABLE

2013-02-02 Thread Andrei Shakirin (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrei Shakirin closed SYNCOPE-298. --- Resolution: Won't Fix Wouldn't be fixed because of another solution with pre-configured IDs

Re: [DISCUSSION] Integration tests instability causes by Ids collision (AUTO generation strategy)

2013-02-02 Thread Francesco Chicchiriccò
On 02/02/2013 11:58, Andrei Shakirin wrote: Hi Francesco, I agree with you that (A) is easy and sounds like a workaround but... we're talking about test data, hence workarounds don't smell like as usual :-) IMO, relying upon some default test data is actually very comfortable, so I wouldn't opt

Re: [Report] CXF migration status

2013-02-02 Thread Francesco Chicchiriccò
On 02/02/2013 12:07, Andrei Shakirin wrote: Hi, Would like to shortly report status of CXF migration: It looks very nice, thanks for reporting. As already said before, your (and the whole "CXF" team's) contribution has been very important and brought this project to a higher level in terms