glassfish icon indicating copy to clipboard operation
glassfish copied to clipboard

Issue #24900 add a few unit tests for ManagedConnectionImpl and ConnectionHolder to understand associateConnection logic

Open escay opened this issue 1 year ago • 1 comments

I wrote a few unit tests to start to understand associateConnection logic which is used by ConnectorXAResource in connector-runtime. Added some javadoc to understand ManagedConnectionImpl parameters. Added a listener null check and added unregister listener logic which seemed to be missing. Removed a boolean parameter from setActive(true) which was always true.

escay avatar Oct 08 '24 14:10 escay

Sorry, we had a broken metadata file in maven cache, that crashed your build, I have restarted the job now

dmatej avatar Oct 08 '24 21:10 dmatej