Skip to content
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

initial TCK tests for method invokers #502

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Nov 14, 2023

@Ladicek Ladicek added this to the CDI 4.1 milestone Nov 14, 2023
@Ladicek
Copy link
Contributor Author

Ladicek commented Nov 14, 2023

Draft for now, until we settle on the specification text. I'm providing this for early review and to facilitate conversations about the spec proposal itself.

@Ladicek
Copy link
Contributor Author

Ladicek commented Jan 5, 2024

Rebased and added one commit (to be squashed) that updates this PR to match the recent specification proposal changes.

@Ladicek
Copy link
Contributor Author

Ladicek commented Jan 17, 2024

Updated to match current jakartaee/cdi#697, including audit XML entries.

@Ladicek
Copy link
Contributor Author

Ladicek commented Jan 30, 2024

Added initial tests for invoker lookups, matching jakartaee/cdi#749

@Ladicek
Copy link
Contributor Author

Ladicek commented Feb 14, 2024

Rebased and updated to current status of jakartaee/cdi#749, including audit XML entries.

@manovotn
Copy link
Contributor

Please rebase this PR as master has the latest CDI API (Beta1) in it.

@Ladicek
Copy link
Contributor Author

Ladicek commented Feb 15, 2024

Rebased and marked as ready for review.

@Ladicek Ladicek marked this pull request as ready for review February 15, 2024 16:49
@starksm64 starksm64 merged commit 0bc29f8 into jakartaee:master Feb 16, 2024
4 checks passed
@Ladicek Ladicek deleted the method-invokers branch February 19, 2024 08:06
@Ladicek Ladicek linked an issue Feb 27, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TCK coverage for method invokers
3 participants