Skip to content

Commit

Permalink
fix: add missing assertion to test (#3765)
Browse files Browse the repository at this point in the history
# Description

The test should be asserting that the parse method returns an error

## Problem\*

Resolves <!-- Link to GitHub Issue -->

## Summary\*



## Additional Context



## Documentation\*

Check one:
- [ ] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[Exceptional Case]** Documentation to be submitted in a separate
PR.

# PR Checklist\*

- [ ] I have tested the changes locally.
- [ ] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
  • Loading branch information
kevaundray authored Dec 11, 2023
1 parent 010fdb6 commit bcbe116
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tooling/noirc_abi/src/input_parser/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,6 @@ mod test {
#[test]
fn rejects_noncanonical_fields() {
let noncanonical_field = FieldElement::modulus().to_string();
let parsed_field = parse_str_to_field(&noncanonical_field);
println!("{parsed_field:?}");
assert!(parse_str_to_field(&noncanonical_field).is_err());
}
}

0 comments on commit bcbe116

Please sign in to comment.