search

tarantool / go-tarantool

Tarantool 1.10+ client for Go language
https://pkg.go.dev/github.com/tarantool/go-tarantool/v2
BSD 2-Clause "Simplified" License
180 stars 57 forks source link

test: use coverage instead of equality to test error payload #391

Closed nshy closed 5 months ago

nshy commented 5 months ago

We are going to add missing 'user' payload field for ACCESS_DENIED error which will break current tests. Let fix tests to allow adding new payload fields for errors.

Need for https://github.com/tarantool/tarantool/issues/9108

oleg-jukovec commented 5 months ago

Thank you for the patch!

At the first glance it seems like it would be better to check different fields for different versions of Tarantool. But on the other hand, there may be too many ifs in the future. So I don't insist.

nshy commented 5 months ago

Thanx!