From 2fed189d2e69fc69fe0829062af4250fb3560298 Mon Sep 17 00:00:00 2001 From: Marten Seemann Date: Sun, 28 Nov 2021 16:49:26 +0400 Subject: [PATCH] reenable Badger tests on 64 bit platforms --- pstoreds/ds_test.go | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/pstoreds/ds_test.go b/pstoreds/ds_test.go index b76bc09..58d9fc6 100644 --- a/pstoreds/ds_test.go +++ b/pstoreds/ds_test.go @@ -4,28 +4,33 @@ import ( "context" "io/ioutil" "os" + "strconv" "testing" "time" - "github.com/stretchr/testify/require" - ds "github.com/ipfs/go-datastore" badger "github.com/ipfs/go-ds-badger" leveldb "github.com/ipfs/go-ds-leveldb" pstore "github.com/libp2p/go-libp2p-core/peerstore" pt "github.com/libp2p/go-libp2p-peerstore/test" + + "github.com/stretchr/testify/require" ) type datastoreFactory func(tb testing.TB) (ds.Batching, func()) var dstores = map[string]datastoreFactory{ - //"Badger": badgerStore, + "Badger": badgerStore, "Leveldb": leveldbStore, } func TestDsPeerstore(t *testing.T) { for name, dsFactory := range dstores { + if name == "Badger" && strconv.IntSize == 32 { + t.Skip("Badger doesn't work on 32 bits") + } + t.Run(name, func(t *testing.T) { pt.TestPeerstore(t, peerstoreFactory(t, dsFactory, DefaultOpts())) }) @@ -97,8 +102,6 @@ func BenchmarkDsPeerstore(b *testing.B) { } } -// Doesn't work on 32bit because badger. -//lint:ignore U1000 disabled for now func badgerStore(tb testing.TB) (ds.Batching, func()) { dataPath, err := ioutil.TempDir(os.TempDir(), "badger") if err != nil {