uber / cadence-java-client / 2541

83%

master: 80%

Build Type

Pull #924

buildkite

Committed by natemort

Commit Message

Add missing proto to thrift enum mappings and unit tests for both EnumMappers

Add a suite of tests that will automatically fail if new enum types are added in IDL but the mappers aren't updated. This was done in a few ways:
- For enums that have mappers both directions, we roundtrip all thrift values and assert that they're equal.
- For enums that are identical across thrift/proto, we generate a mapping based on the value names and assert that the mapper matches it.
- For enums that aren't identical, we explicitly assert that our mapping contains all values.

Pull Request Pull Request #924: Add missing proto to thrift enum mappings and unit tests for EnumMapper

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

8 existing lines in 4 files now uncovered.

12893 of 19503 relevant lines covered (66.11%)

0.66 hits per line