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

Fix pickler deserialization when enum cases are not alphabetically sorted #4093

Merged
merged 2 commits into from
Oct 9, 2024

Merge branch 'master' into fix-enum-pickler

52ea6fe
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

Fix pickler deserialization when enum cases are not alphabetically sorted #4093

Merge branch 'master' into fix-enum-pickler
52ea6fe
Select commit
Loading
Failed to load commit list.
GitHub Actions / Test report for 2.12-JS-11 succeeded Oct 9, 2024 in 0s

481 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientBasicTests.xml 57✅ 424ms
client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientMultipartTests.xml 3✅ 221ms
client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientRequestTests.xml 3✅ 11ms
client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientWebSocketZioTests.xml 3✅ 307ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.annotations.DeriveEndpointIOTest.xml 30✅ 8ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.AttributeTest.xml 3✅ 0ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.CodecTest.xml 6✅ 145ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.EndpointTest.xml 76✅ 27ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.EnumsCodecDerivationTest.xml 4✅ 4ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.FormCodecDerivationTest.xml 14✅ 16ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.LegacySchemaGenericAutoTest.xml 8✅ 27ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.MultipartCodecDerivationTest.xml 13✅ 25ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.SchemaGenericAutoTest.xml 29✅ 16ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.ValueClassCodecDerivationTest.xml 1✅ 3ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.RichEndpointInputTest.xml 5✅ 0ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.RichEndpointOutputTest.xml 1✅ 2ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.UrlencodedDataTest.xml 4✅ 1ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.LegacySchemaApplyValidationTest.xml 1✅ 4ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.namespacing.SchemaMacroNamespaceTest.xml 2✅ 9ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaAnnotationsTest.xml 1✅ 1ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaApplyValidationTest.xml 13✅ 305ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaMacroTest.xml 26✅ 13ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaMacroTest2.xml 5✅ 3ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaTest.xml 13✅ 11ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.typelevel.MatchTypeTest.xml 8✅ 5ms
core/target/js-2.12/test-reports/TEST-sttp.tapir.ValidatorTest.xml 28✅ 5ms
integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.EndpointIOInstancesSpec.xml 3✅ 2ms
integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ExampleFunctorLawSpec.xml 5✅ 23ms
integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ModifyFunctorInstancesTest.xml 4✅ 5ms
integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.TapirCodecCatsTest.xml 22✅ 136ms
integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ValidatorCatsTest.xml 3✅ 0ms
integrations/enumeratum/target/js-2.12/test-reports/TEST-sttp.tapir.codec.enumeratum.TapirCodecEnumeratumTest.xml 5✅ 19ms
integrations/newtype/target/js-2.12/test-reports/TEST-sttp.tapir.codec.newtype.TapirCodecNewTypeTest.xml 6✅ 2ms
json/jsoniter/target/js-2.12/test-reports/TEST-sttp.tapir.json.jsoniter.TapirJsonJsoniterTests.xml 1✅ 6ms
json/playjson/target/js-2.12/test-reports/TEST-sttp.tapir.json.play.TapirJsonPlayTests.xml 9✅ 15ms
json/upickle/target/js-2.12/test-reports/TEST-sttp.tapir.json.upickle.TapirJsonuPickleTests.xml 5✅ 6ms
json/zio/target/js-2.12/test-reports/TEST-sttp.tapir.json.zio.TapirJsonZioTest.xml 8✅ 10ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.cors.CORSConfigTest.xml 2✅ 1ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.cors.CORSInterceptorTest.xml 2✅ 0ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.decodefailure.DefaultDecodeFailureHandlerTest.xml 5✅ 3ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.DecodeBasicInputsTest.xml 1✅ 2ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.FilterServerEndpointsTest.xml 9✅ 11ms
server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.ServerInterpreterTest.xml 3✅ 7ms
testing/target/js-2.12/test-reports/TEST-sttp.tapir.testing.EndpointVerifierTest.xml 31✅ 34ms

✅ client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientBasicTests.xml

57 tests were completed in 424ms with 57 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.client.sttp.SttpClientBasicTests 57✅ 424ms

✅ client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientMultipartTests.xml

3 tests were completed in 221ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.client.sttp.SttpClientMultipartTests 3✅ 221ms

✅ client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientRequestTests.xml

3 tests were completed in 11ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.client.sttp.SttpClientRequestTests 3✅ 11ms

✅ client/sttp-client/target/js-2.12/test-reports/TEST-sttp.tapir.client.sttp.SttpClientWebSocketZioTests.xml

3 tests were completed in 307ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.client.sttp.SttpClientWebSocketZioTests 3✅ 307ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.annotations.DeriveEndpointIOTest.xml

30 tests were completed in 8ms with 30 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.annotations.DeriveEndpointIOTest 30✅ 8ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.AttributeTest.xml

3 tests were completed in 0ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.AttributeTest 3✅ 0ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.CodecTest.xml

6 tests were completed in 145ms with 6 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.CodecTest 6✅ 145ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.EndpointTest.xml

76 tests were completed in 27ms with 76 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.EndpointTest 76✅ 27ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.EnumsCodecDerivationTest.xml

4 tests were completed in 4ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.EnumsCodecDerivationTest 4✅ 4ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.FormCodecDerivationTest.xml

14 tests were completed in 16ms with 14 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.FormCodecDerivationTest 14✅ 16ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.LegacySchemaGenericAutoTest.xml

8 tests were completed in 27ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.LegacySchemaGenericAutoTest 8✅ 27ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.MultipartCodecDerivationTest.xml

13 tests were completed in 25ms with 13 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.MultipartCodecDerivationTest 13✅ 25ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.SchemaGenericAutoTest.xml

29 tests were completed in 16ms with 29 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.SchemaGenericAutoTest 29✅ 16ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.generic.ValueClassCodecDerivationTest.xml

1 tests were completed in 3ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.generic.ValueClassCodecDerivationTest 1✅ 3ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.RichEndpointInputTest.xml

5 tests were completed in 0ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.internal.RichEndpointInputTest 5✅ 0ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.RichEndpointOutputTest.xml

1 tests were completed in 2ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.internal.RichEndpointOutputTest 1✅ 2ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.internal.UrlencodedDataTest.xml

4 tests were completed in 1ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.internal.UrlencodedDataTest 4✅ 1ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.LegacySchemaApplyValidationTest.xml

1 tests were completed in 4ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.LegacySchemaApplyValidationTest 1✅ 4ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.namespacing.SchemaMacroNamespaceTest.xml

2 tests were completed in 9ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.namespacing.SchemaMacroNamespaceTest 2✅ 9ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaAnnotationsTest.xml

1 tests were completed in 1ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.SchemaAnnotationsTest 1✅ 1ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaApplyValidationTest.xml

13 tests were completed in 305ms with 13 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.SchemaApplyValidationTest 13✅ 305ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaMacroTest.xml

26 tests were completed in 13ms with 26 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.SchemaMacroTest 26✅ 13ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaMacroTest2.xml

5 tests were completed in 3ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.SchemaMacroTest2 5✅ 3ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.SchemaTest.xml

13 tests were completed in 11ms with 13 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.SchemaTest 13✅ 11ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.typelevel.MatchTypeTest.xml

8 tests were completed in 5ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.typelevel.MatchTypeTest 8✅ 5ms

✅ core/target/js-2.12/test-reports/TEST-sttp.tapir.ValidatorTest.xml

28 tests were completed in 5ms with 28 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.ValidatorTest 28✅ 5ms

✅ integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.EndpointIOInstancesSpec.xml

3 tests were completed in 2ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.integ.cats.EndpointIOInstancesSpec 3✅ 2ms

✅ integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ExampleFunctorLawSpec.xml

5 tests were completed in 23ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.integ.cats.ExampleFunctorLawSpec 5✅ 23ms

✅ integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ModifyFunctorInstancesTest.xml

4 tests were completed in 5ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.integ.cats.ModifyFunctorInstancesTest 4✅ 5ms

✅ integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.TapirCodecCatsTest.xml

22 tests were completed in 136ms with 22 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.integ.cats.TapirCodecCatsTest 22✅ 136ms

✅ integrations/cats/target/js-2.12/test-reports/TEST-sttp.tapir.integ.cats.ValidatorCatsTest.xml

3 tests were completed in 0ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.integ.cats.ValidatorCatsTest 3✅ 0ms

✅ integrations/enumeratum/target/js-2.12/test-reports/TEST-sttp.tapir.codec.enumeratum.TapirCodecEnumeratumTest.xml

5 tests were completed in 19ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.codec.enumeratum.TapirCodecEnumeratumTest 5✅ 19ms

✅ integrations/newtype/target/js-2.12/test-reports/TEST-sttp.tapir.codec.newtype.TapirCodecNewTypeTest.xml

6 tests were completed in 2ms with 6 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.codec.newtype.TapirCodecNewTypeTest 6✅ 2ms

✅ json/jsoniter/target/js-2.12/test-reports/TEST-sttp.tapir.json.jsoniter.TapirJsonJsoniterTests.xml

1 tests were completed in 6ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.json.jsoniter.TapirJsonJsoniterTests 1✅ 6ms

✅ json/playjson/target/js-2.12/test-reports/TEST-sttp.tapir.json.play.TapirJsonPlayTests.xml

9 tests were completed in 15ms with 9 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.json.play.TapirJsonPlayTests 9✅ 15ms

✅ json/upickle/target/js-2.12/test-reports/TEST-sttp.tapir.json.upickle.TapirJsonuPickleTests.xml

5 tests were completed in 6ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.json.upickle.TapirJsonuPickleTests 5✅ 6ms

✅ json/zio/target/js-2.12/test-reports/TEST-sttp.tapir.json.zio.TapirJsonZioTest.xml

8 tests were completed in 10ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.json.zio.TapirJsonZioTest 8✅ 10ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.cors.CORSConfigTest.xml

2 tests were completed in 1ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interceptor.cors.CORSConfigTest 2✅ 1ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.cors.CORSInterceptorTest.xml

2 tests were completed in 0ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interceptor.cors.CORSInterceptorTest 2✅ 0ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interceptor.decodefailure.DefaultDecodeFailureHandlerTest.xml

5 tests were completed in 3ms with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interceptor.decodefailure.DefaultDecodeFailureHandlerTest 5✅ 3ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.DecodeBasicInputsTest.xml

1 tests were completed in 2ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interpreter.DecodeBasicInputsTest 1✅ 2ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.FilterServerEndpointsTest.xml

9 tests were completed in 11ms with 9 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interpreter.FilterServerEndpointsTest 9✅ 11ms

✅ server/core/target/js-2.12/test-reports/TEST-sttp.tapir.server.interpreter.ServerInterpreterTest.xml

3 tests were completed in 7ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.server.interpreter.ServerInterpreterTest 3✅ 7ms

✅ testing/target/js-2.12/test-reports/TEST-sttp.tapir.testing.EndpointVerifierTest.xml

31 tests were completed in 34ms with 31 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
sttp.tapir.testing.EndpointVerifierTest 31✅ 34ms