This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Use cmp in tests #188

Open

rbehjati opened this issue Dec 21, 2022 · 0 comments

Labels

Core good first issue

Contributor

rbehjati commented Dec 21, 2022

A conventional pattern in tests is to use the cmp package and the following pattern:

if diff := cmp.Diff(got, want); diff != "" {
  t.Errorf("<descriptive error message referring to got and want>")

Ideally this should be followed in all tests and much as possible.

The text was updated successfully, but these errors were encountered:

rbehjati added the good first issue label

rbehjati mentioned this issue

Refactor Internal Endorser #189

Closed

rbehjati added the Core label

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.