Browse Source

protoio: fix incorrect test assertion (#7606)

After writing and then reading a bunch of random messages, the test was
checking that it did not read the same number of messages that it wrote.
The sense of this check was inverted; they should match.

Introduced by accident in #7522. I'm not sure why this did not show up in CI.

Edit: I now know why it didn't show up in ci: #7608.
pull/7610/head
M. J. Fromberger 3 years ago
committed by GitHub
parent
commit
c24f003b55
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
  1. +1
    -1
      internal/libs/protoio/io_test.go

+ 1
- 1
internal/libs/protoio/io_test.go View File

@ -95,7 +95,7 @@ func iotest(t *testing.T, writer protoio.WriteCloser, reader protoio.ReadCloser)
} }
i++ i++
} }
require.NotEqual(t, size, i)
require.Equal(t, size, i, "messages read ≠ messages written")
if err := reader.Close(); err != nil { if err := reader.Close(); err != nil {
return err return err
} }


Loading…
Cancel
Save