-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inconsistency in when BeanContainer.createInstance() may be called #688
Comments
It looks like the TCK asserts that it can't be called before |
This is what I would expect to be the correct behavior.
Interestingly, the same javadoc, just a few lines below, also defines that it throws an exception if invoked before ADV :) |
It does :)
Huh, so it does. I guess it's just the Javadoc that needs cleaned up then. I will check for any other references and make a PR. |
Thanks! |
The Javadoc for BeanContainer.createInstance() defines its behaviour when called during
AfterBeanDiscovery
. The Javadoc forBeanManager
says it's not valid before AfterBeanDiscovery.However, the spec (3.9.1) says that it's not valid before
AfterDeploymentValidation
which is fired afterAfterBeanDiscovery
.The spec and the Javadoc disagree here, which one should be changed?
The text was updated successfully, but these errors were encountered: