fixes validation issue with range timestamp in `valid_values` - Githubissues

amazon-ion / ion-schema-rust

Rust implementation of Ion Schema

https://amazon-ion.github.io/ion-schema/sandbox

Apache License 2.0

13 stars 6 forks source link

fixes validation issue with range timestamp in `valid_values` #107

Closed desaikd closed 2 years ago

desaikd commented 2 years ago

Description of changes: This PR works to fix issue with range timestamp validation with valid_values constraint.

List of changes:

adds a check to consider none of the range boundary value have an unknown offset as per Ion schema spec 1.0
defines a PartialEq for RangeBoundaryValue which correctly checks for equivalence
updates ion-schema-tests sub module to point to latest commit.
updates ion-rs version to 0.13.0 to get the correct Ord implementation of Timestamp
adds cargo clippy changes to add Eq for all implementations.

Test:

Removes tests from skip list for range timestamp in valid_values