search

jakartaee / jsonb-api

Jakarta JSON Binding
https://eclipse-ee4j.github.io/jsonb-api/
Other
78 stars 39 forks source link

UntypedMappingTest.testArrayMapping tck test should fail #267

Closed Verdent closed 3 years ago

Verdent commented 3 years ago

I have encountered some strange behavior in UntypedMappingTest.testArrayMapping test.

The test itself is using SimpleMappingTester class and passes List instance to its test method. This test method, on the line 46 marshalls List to the json, but the runtime type provided is actually TypeContainer class. I would assume that runtime type should be exact/more precise type of the handled instance. Therefore I think this test is not correct and it should fail since this List should be handled as TypeContainer instance in this case.

Is this proper behavior? Should I make a fix for the issue?

edbratt commented 2 years ago

Apologies for resurrecting this issue -- it appears this challenge was accepted -- was it actually a challenge, or just a bug? If it was a challenge, there should be either an accepted or invalid label attached. If it isn't a challenge, and it's just a bug in the TCK being developed, then the challenge label ought to be removed. Going further, if it was a successful challenge, there ought to be a record of a disabled test followed by a bug-fix update release to the TCK (bump the micro-version ID).