lntest: avoid global ServeMux

Using the default, global ServeMux prevents the same process from
calling `lntest.NewNetworkHarness` multiple times, because we get a
panic when registering HTTP routes.

Instead, we use the ServeMux beloning to the fee service struct.
This commit is contained in:
Torkel Rogstad
2021-08-26 15:17:35 +02:00
parent 93d12cd9fc
commit d6045a0fd5
2 changed files with 10 additions and 4 deletions

View File

@@ -42,6 +42,10 @@ tag](https://github.com/lightningnetwork/lnd/pull/5335). A new flag
test](https://github.com/lightningnetwork/lnd/pull/5348) that would cause the
test to assert the wrong balance (the miner fee wasn't accounted for).
A bug has been [fixed](https://github.com/lightningnetwork/lnd/pull/5674) in
the `lntest` package that prevented multiple test harnesses to be created from
the same process.
## Forwarding Optimizations
[Decoding onion blobs is now done in

View File

@@ -46,12 +46,14 @@ func startFeeService() *feeService {
f.Fees = map[uint32]uint32{feeServiceTarget: 50000}
listenAddr := fmt.Sprintf(":%v", port)
mux := http.NewServeMux()
mux.HandleFunc("/fee-estimates.json", f.handleRequest)
f.srv = &http.Server{
Addr: listenAddr,
Handler: mux,
}
http.HandleFunc("/fee-estimates.json", f.handleRequest)
f.wg.Add(1)
go func() {
defer f.wg.Done()