From 28c8e8b6ddd82b55395e7a137cb6142310b1512c Mon Sep 17 00:00:00 2001 From: Oliver Gugger Date: Fri, 30 Sep 2022 14:17:12 +0200 Subject: [PATCH] channeldb: remove unused tx argument --- channeldb/db.go | 2 +- channeldb/meta.go | 5 ++--- channeldb/meta_test.go | 12 ++++++------ 3 files changed, 9 insertions(+), 10 deletions(-) diff --git a/channeldb/db.go b/channeldb/db.go index a4c9d13af..9390b9352 100644 --- a/channeldb/db.go +++ b/channeldb/db.go @@ -1417,7 +1417,7 @@ func (c *ChannelStateDB) DeleteChannelOpeningState(outPoint []byte) error { // applies migration functions to the current database and recovers the // previous state of db if at least one error/panic appeared during migration. func (d *DB) syncVersions(versions []mandatoryVersion) error { - meta, err := d.FetchMeta(nil) + meta, err := d.FetchMeta() if err != nil { if err == ErrMetaNotFound { meta = &Meta{} diff --git a/channeldb/meta.go b/channeldb/meta.go index 9173224b8..55efa8d60 100644 --- a/channeldb/meta.go +++ b/channeldb/meta.go @@ -28,9 +28,8 @@ type Meta struct { DbVersionNumber uint32 } -// FetchMeta fetches the meta data from boltdb and returns filled meta -// structure. -func (d *DB) FetchMeta(tx kvdb.RTx) (*Meta, error) { +// FetchMeta fetches the metadata from boltdb and returns filled meta structure. +func (d *DB) FetchMeta() (*Meta, error) { var meta *Meta err := kvdb.View(d, func(tx kvdb.RTx) error { diff --git a/channeldb/meta_test.go b/channeldb/meta_test.go index 5b1e45cc7..9918d73a2 100644 --- a/channeldb/meta_test.go +++ b/channeldb/meta_test.go @@ -91,7 +91,7 @@ func TestVersionFetchPut(t *testing.T) { t.Fatal(err) } - meta, err := db.FetchMeta(nil) + meta, err := db.FetchMeta() if err != nil { t.Fatal(err) } @@ -107,7 +107,7 @@ func TestVersionFetchPut(t *testing.T) { t.Fatalf("update of meta failed %v", err) } - meta, err = db.FetchMeta(nil) + meta, err = db.FetchMeta() if err != nil { t.Fatal(err) } @@ -228,7 +228,7 @@ func TestMigrationWithPanic(t *testing.T) { // Check that version of database and data wasn't changed. afterMigrationFunc := func(d *DB) { - meta, err := d.FetchMeta(nil) + meta, err := d.FetchMeta() if err != nil { t.Fatal(err) } @@ -303,7 +303,7 @@ func TestMigrationWithFatal(t *testing.T) { // Check that version of database and initial data wasn't changed. afterMigrationFunc := func(d *DB) { - meta, err := d.FetchMeta(nil) + meta, err := d.FetchMeta() if err != nil { t.Fatal(err) } @@ -377,7 +377,7 @@ func TestMigrationWithoutErrors(t *testing.T) { // Check that version of database and data was properly changed. afterMigrationFunc := func(d *DB) { - meta, err := d.FetchMeta(nil) + meta, err := d.FetchMeta() if err != nil { t.Fatal(err) } @@ -469,7 +469,7 @@ func TestMigrationDryRun(t *testing.T) { // Check that version of database version is not modified. afterMigrationFunc := func(d *DB) { err := kvdb.View(d, func(tx kvdb.RTx) error { - meta, err := d.FetchMeta(nil) + meta, err := d.FetchMeta() if err != nil { t.Fatal(err) }