edgedb / edgedb-rust

The official Rust binding for EdgeDB
https://edgedb.com
Apache License 2.0
213 stars 26 forks source link

Fix errors within the implement_tuple! macro. #219

Closed UndefinedBHVR closed 1 year ago

UndefinedBHVR commented 1 year ago

This PR fixes the three errors I noticed in the implement_tuple! macro within query_args.rs as discussed within the Discord channel.

In particular, it fixes the lack of a format statement for the call of DescriptorMismatch::with_message() which resulted in: ["provided {} positional arguments, but no arguments expected by the server"] It now will properly respond the argument counts it received as such: ["provided 1 positional arguments, but no arguments expected by the server"]

The other two errors corrected are regard to the position of the expected and received argument count for enc.ctx.field_number() This fix corrects the parameters being out of order for the function call, which would result in ["expected 1 fields, got 2"] When it should have been ["expected 2 fields, got 1"]

edgedb-cla[bot] commented 1 year ago

All commit authors signed the Contributor License Agreement.
CLA signed