-
Notifications
You must be signed in to change notification settings - Fork 103
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
Updated TCK test lists #1078
Updated TCK test lists #1078
Conversation
…n numbers and referred to list in the respective Platform Specification documents. Signed-off-by: Ed Bratt <ed.bratt@oracle.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The section 1.5 Additional Requirements seems to be untouched. Should we modify or remove that .
Update: Sorry I see that was updated.
Fixed Typo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that more changes are needed for Activation, Mail, DSOL. I'll provide more feedback later but I think it would be better to separate the addition of these TCKs instead a separate pull request for further discussion that might also cover the (old) Authentication source repo issue as well (which is similar to Mail TCK imo).
Updated: Since GlassFish is able to pass the Standalone TCKs (Activation, Mail, DSOL) and we do not know of any problems with running these TCKs (any problems can be reported as TCK challenges), I will give my approval of this pull request after the minor typo is corrected.
It seems like the Activation 2.1 TCK doesn't work with WildFly, so created jakartaee/jaf-tck#55 TCK challenge for that. I'm hoping it is just a user mistake. |
Also created jakartaee/mail-tck#47 to ensure that Java Mail 2.1 TCK can be used by all EE 10 implementations, including those that do no support JPMS. Further testing is needed to determine if the Mail 2.1 TCK can work with non-JPMS EE 10 implementations (as per EE 10 Platform requirements). |
Never mind. now passing thanks to @lukasj ! |
As described in #1077. Removed version numbers and referred to list in the respective Platform Specification documents.
CC @alwin-joseph @anajosep @arjantijms @cesarhernandezgt @dblevins @m0mus @edbratt @gurunrao @jansupol @jgallimore @kazumura @kwsutter @LanceAndersen @bhatpmk @RohitKumarJain @shighbar @gthoman @brideck @scottmarlow
Signed-off-by: Ed Bratt ed.bratt@oracle.com