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

DeepClone in QuarkusTestExtension returns null #9886

Closed
alesj opened this issue Jun 9, 2020 · 2 comments · Fixed by #9938
Closed

DeepClone in QuarkusTestExtension returns null #9886

alesj opened this issue Jun 9, 2020 · 2 comments · Fixed by #9938
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@alesj
Copy link
Contributor

alesj commented Jun 9, 2020

Custom JUnit5 extensions providing non-primitive test method arguments have issues wrt TCCL.
This is "solved" with DeepClone workaround.
But in our case this DeepClone impl returns null for a simple JDK Supplier instance:

@alesj alesj added the kind/bug Something isn't working label Jun 9, 2020
@geoand geoand self-assigned this Jun 9, 2020
@pepperbob
Copy link

pepperbob commented Jun 10, 2020

Problems with cloning using XStream serialisation/deserialisation is also the cause of the issues #9677 and #9884

There is quite a list of limitations in the XStream-FAQ, e.g. My lambda expression is serialized to null!

@geoand
Copy link
Contributor

geoand commented Jun 10, 2020

Yeah, we'll have to address that

geoand added a commit to geoand/quarkus that referenced this issue Jun 11, 2020
geoand added a commit to geoand/quarkus that referenced this issue Jun 11, 2020
geoand added a commit to geoand/quarkus that referenced this issue Jun 15, 2020
geoand added a commit that referenced this issue Jun 15, 2020
Fix deepClone issue with Supplier
@gsmet gsmet added this to the 1.5.2.Final milestone Jun 18, 2020
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jun 18, 2020
johnaohara pushed a commit to johnaohara/quarkus that referenced this issue Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants