Merge pull request #8577 from undefinedor/bugfix/fix-unreachable-code

kvdb+watchtower: fix unreachable code

docs/release-notes/release-notes-0.18.0.md

@@ -104,6 +104,8 @@
   which with the default fee allocation in place will eventually lead to the
   downsizing to the fee floor (1 sat/vByte) in the worst case.
 
+* [Removed](https://github.com/lightningnetwork/lnd/pull/8577) some unreachable code
+
 # New Features
 ## Functional Enhancements
 
@@ -424,6 +426,7 @@ bitcoin peers' feefilter values into account](https://github.com/lightningnetwor
 * Slyghtning
 * Tee8z
 * Turtle
+* Hao Wang
 * w3irdrobot
 * Yong Yu
 * Ziggie

kvdb/backend.go

@@ -301,6 +301,4 @@ func GetTestBackend(path, name string) (Backend, func(), error) {
 		}
 		return db, empty, nil
 	}
-
-	return nil, nil, fmt.Errorf("unknown backend")
 }

watchtower/wtclient/session_negotiator.go

@@ -333,9 +333,6 @@ tryNextCandidate:
 
 			goto retryWithBackoff
 		}
-
-		// Success.
-		return
 	}
 }
 
@@ -393,8 +390,6 @@ func (n *sessionNegotiator) createSession(tower *Tower, keyIndex uint32) error {
 			return nil
 		}
 	}
-
-	return ErrFailedNegotiation
 }
 
 // tryAddress executes a single create session dance using the given address.