Skip to content

Commit

Permalink
itest: remove obsolete itest
Browse files Browse the repository at this point in the history
  • Loading branch information
bhandras committed Nov 26, 2024
1 parent 6d2dd69 commit 5b300dd
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 43 deletions.
4 changes: 0 additions & 4 deletions itest/list_on_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -666,10 +666,6 @@ var allTestCases = []*lntest.TestCase{
Name: "open channel locked balance",
TestFunc: testOpenChannelLockedBalance,
},
{
Name: "nativesql no migration",
TestFunc: testNativeSQLNoMigration,
},
{
Name: "sweep cpfp anchor outgoing timeout",
TestFunc: testSweepCPFPAnchorOutgoingTimeout,
Expand Down
39 changes: 0 additions & 39 deletions itest/lnd_misc_test.go
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package itest

import (
"context"
"encoding/hex"
"fmt"
"os"
Expand Down Expand Up @@ -1271,44 +1270,6 @@ func testSignVerifyMessageWithAddr(ht *lntest.HarnessTest) {
require.False(ht, respValid.Valid, "external signature did validate")
}

// testNativeSQLNoMigration tests that nodes that have invoices would not start
// up with native SQL enabled, as we don't currently support migration of KV
// invoices to the new SQL schema.
func testNativeSQLNoMigration(ht *lntest.HarnessTest) {
alice := ht.Alice

// Make sure we run the test with SQLite or Postgres.
if alice.Cfg.DBBackend != node.BackendSqlite &&
alice.Cfg.DBBackend != node.BackendPostgres {

ht.Skip("node not running with SQLite or Postgres")
}

// Skip the test if the node is already running with native SQL.
if alice.Cfg.NativeSQL {
ht.Skip("node already running with native SQL")
}

alice.RPC.AddInvoice(&lnrpc.Invoice{
Value: 10_000,
})

alice.SetExtraArgs([]string{"--db.use-native-sql"})

// Restart the node manually as we're really only interested in the
// startup error.
require.NoError(ht, alice.Stop())
require.NoError(ht, alice.StartLndCmd(context.Background()))

// We expect the node to fail to start up with native SQL enabled, as we
// have an invoice in the KV store.
require.Error(ht, alice.WaitForProcessExit())

// Reset the extra args and restart alice.
alice.SetExtraArgs(nil)
require.NoError(ht, alice.Start(ht.Context()))
}

// testSendSelectedCoins tests that we're able to properly send the selected
// coins from the wallet to a single target address.
func testSendSelectedCoins(ht *lntest.HarnessTest) {
Expand Down

0 comments on commit 5b300dd

Please sign in to comment.