Skip to content

Commit

Permalink
fix(LSP): correct signature for assert and assert_eq (noir-lang#7184)
Browse files Browse the repository at this point in the history
  • Loading branch information
asterite authored Jan 24, 2025
1 parent 60076d5 commit b6fa180
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions tooling/lsp/src/requests/signature_help.rs
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ impl<'a> SignatureFinder<'a> {
self.hardcoded_signature_information(
active_parameter,
"assert",
&["predicate: bool", "[failure_message: str<N>]"],
&["predicate: bool", "[failure_message: T]"],
)
}

Expand All @@ -251,7 +251,7 @@ impl<'a> SignatureFinder<'a> {
self.hardcoded_signature_information(
active_parameter,
"assert_eq",
&["lhs: T", "rhs: T", "[failure_message: str<N>]"],
&["lhs: T", "rhs: T", "[failure_message: U]"],
)
}

Expand Down
8 changes: 4 additions & 4 deletions tooling/lsp/src/requests/signature_help/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -206,13 +206,13 @@ mod signature_help_tests {
assert_eq!(signature_help.signatures.len(), 1);

let signature = &signature_help.signatures[0];
assert_eq!(signature.label, "assert(predicate: bool, [failure_message: str<N>])");
assert_eq!(signature.label, "assert(predicate: bool, [failure_message: T])");

let params = signature.parameters.as_ref().unwrap();
assert_eq!(params.len(), 2);

check_label(&signature.label, &params[0].label, "predicate: bool");
check_label(&signature.label, &params[1].label, "[failure_message: str<N>]");
check_label(&signature.label, &params[1].label, "[failure_message: T]");

assert_eq!(signature.active_parameter, Some(0));
}
Expand All @@ -229,14 +229,14 @@ mod signature_help_tests {
assert_eq!(signature_help.signatures.len(), 1);

let signature = &signature_help.signatures[0];
assert_eq!(signature.label, "assert_eq(lhs: T, rhs: T, [failure_message: str<N>])");
assert_eq!(signature.label, "assert_eq(lhs: T, rhs: T, [failure_message: U])");

let params = signature.parameters.as_ref().unwrap();
assert_eq!(params.len(), 3);

check_label(&signature.label, &params[0].label, "lhs: T");
check_label(&signature.label, &params[1].label, "rhs: T");
check_label(&signature.label, &params[2].label, "[failure_message: str<N>]");
check_label(&signature.label, &params[2].label, "[failure_message: U]");

assert_eq!(signature.active_parameter, Some(0));
}
Expand Down

0 comments on commit b6fa180

Please sign in to comment.