Skip to content

Commit

Permalink
chore: give better error logging in some cases
Browse files Browse the repository at this point in the history
  • Loading branch information
serenibyss committed Jun 9, 2021
1 parent c188897 commit 9b91a5b
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions cmd/influxd/inspect/tsm_verify_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"encoding/binary"
"io/ioutil"
"os"
"strings"
"testing"

"github.com/influxdata/influxdb/v2/tsdb/engine/tsm1"
Expand All @@ -24,7 +23,7 @@ func TestInvalidChecksum(t *testing.T) {

out, err := ioutil.ReadAll(b)
require.NoError(t, err)
require.True(t, strings.Contains(string(out), "Broken Blocks: 1 / 1"))
require.Contains(t, string(out), "Broken Blocks: 1 / 1")
}

func TestValidChecksum(t *testing.T) {
Expand All @@ -39,7 +38,7 @@ func TestValidChecksum(t *testing.T) {

out, err := ioutil.ReadAll(b)
require.NoError(t, err)
require.True(t, strings.Contains(string(out), "Broken Blocks: 0 / 1"))
require.Contains(t, string(out), "Broken Blocks: 0 / 1")
}

func TestInvalidUTF8(t *testing.T) {
Expand All @@ -64,7 +63,7 @@ func TestValidUTF8(t *testing.T) {

out, err := ioutil.ReadAll(b)
require.NoError(t, err)
require.True(t, strings.Contains(string(out), "Invalid Keys: 0 / 1"))
require.Contains(t, string(out), "Invalid Keys: 0 / 1")
}

func newUTFTest(t *testing.T, withError bool) string {
Expand Down Expand Up @@ -116,7 +115,7 @@ func newChecksumTest(t *testing.T, withError bool) string {
defer fh.Close()

written, err := fh.WriteAt([]byte("foob"), 5)
require.True(t, written == 4)
require.Equal(t, 4, written)
require.NoError(t, err)
}

Expand Down

0 comments on commit 9b91a5b

Please sign in to comment.