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.

Sign up for GitHub

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

Jump to bottom

Create negative tests for build compatible extensions #321

Closed
manovotn opened this issue Dec 7, 2021 · 2 comments · Fixed by #329
Closed

Create negative tests for build compatible extensions #321

manovotn opened this issue Dec 7, 2021 · 2 comments · Fixed by #329
Assignees
@manovotn

Comments

@manovotn
Copy link
Contributor

manovotn commented Dec 7, 2021 •
edited
Loading

Build compatible extensions (in their @Enhancement and @Registration) phases state that their methods need to have exactly one parameter from certain types.

We should provide tests that break these restrictions (multiple or no params) and are expecting a deployment problem.
@manovotn manovotn self-assigned this Dec 7, 2021
@Ladicek
Copy link
Contributor

Ladicek commented Dec 7, 2021

It's not @Synthesis, it's @Registration.

@manovotn
Copy link
Contributor Author

manovotn commented Dec 7, 2021

It's not @Synthesis, it's @Registration.

OFC, I am bad at talking and typing simultaneously :)
I've edited the issue description.

This was referenced Dec 7, 2021
Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@manovotn manovotn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add negative tests for build compatible extensions with invalid metho… manovotn/cdi-tck
2 participants
@Ladicek @manovotn