Skip to content

Commit

Permalink
Adds integration testing for Ion 1.1 (#673)
Browse files Browse the repository at this point in the history
* Updates to latest commit of ion-tests submodule
* Adds support for equivs and non-equivs test cases in structs
  • Loading branch information
popematt authored Oct 19, 2023
1 parent 755bc99 commit fa3167d
Show file tree
Hide file tree
Showing 5 changed files with 235 additions and 130 deletions.
2 changes: 1 addition & 1 deletion ion-tests
Submodule ion-tests updated 1402 files
2 changes: 1 addition & 1 deletion src/ion_data/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ impl<T: Display> Display for IonData<T> {

impl<T: IonEq + IonOrd> PartialOrd for IonData<T> {
fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
Some(IonOrd::ion_cmp(&self.0, &other.0))
Some(self.cmp(other))
}
}

Expand Down
2 changes: 1 addition & 1 deletion src/types/struct.rs
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ where

by_name
.entry(field_name.clone())
.or_insert_with(IndexVec::new)
.or_default()
.push(by_index.len());
by_index.push((field_name, field_value));
}
Expand Down
Loading

0 comments on commit fa3167d

Please sign in to comment.