I think the NHibernateIntegration is the best place to start. Maybe we need to make some changes on the nh facility too.
Cheers, Henry Conceição On Wed, Mar 30, 2011 at 6:22 AM, John Surcombe <[email protected]> wrote: > Henry, > > I'm not sure if NHibernateIntegration is the right place for this. My > feeling is that this may actually be a problem with AutoTx. The test > should be that if a ResourceAdapter throws a suitable exception on > Commit() (simulating what happens in our NHibernateIntegration case), > then AfterCompletion should still fire. > > If on the other hand you feel that it's correct that AfterCompletion > doesn't get called if the commit fails, then the change should be that > NHibernateIntegration should not use AfterCompletion to clear up the > session, but should do it some other way. It's hard to imagine a test > for this though - such a test would check that NHibernateIntegration > is using some other hook to do its clearing up, but I don't know if > such a hook exists. > > Cheers now, > John > > > On Mar 29, 10:27 pm, Henry Conceição <[email protected]> > wrote: >> Hi John, >> >> I think you have a point here and this looks like a bug. Can you >> please convert your simple project in nhibernateintegration test and >> open a ticket? >> > > -- > You received this message because you are subscribed to the Google Groups > "Castle Project Development List" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/castle-project-devel?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Castle Project Development List" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.
