Closed joliveira closed 9 years ago
@joliveira Looks good to me... Hadn't hit the master password issue before, but looks like there's no harm including it...
Will get this merged in soon...
@joliveira Thanks again for the contribution...
Great! Thank you!
On Sat, Oct 25, 2014 at 9:36 AM, Gavin Williams notifications@github.com wrote:
@joliveira https://github.com/joliveira Thanks again for the contribution...
— Reply to this email directly or view it on GitHub https://github.com/fatmcgav/fatmcgav-glassfish/pull/37#issuecomment-60488596 .
Added switch "--savemasterpassword" to the create-domain command. Using class create-domain after including the base glassfish class causes the create-domain to error stating to include the switch.
jmsresource with restype of javax.jms.ConnectionFactory was not in the list of valid restypes.
Updated the test cases for create-domain and added a test for the ConnectionFactory restype.