Merge pull request #493 from yyforyongyu/db-params

liquidity: persist manager's params to disk

Author: yyforyongyu

cmd/loop/liquidity.go

@@ -223,9 +223,12 @@ func setRule(ctx *cli.Context) error {
 }
 
 var setParamsCommand = cli.Command{
-	Name:        "setparams",
-	Usage:       "update the parameters set for the liquidity manager",
-	Description: "Updates the parameters set for the liquidity manager.",
+	Name:  "setparams",
+	Usage: "update the parameters set for the liquidity manager",
+	Description: "Updates the parameters set for the liquidity manager. " +
+		"Note the parameters are persisted in db to save the trouble " +
+		"of setting them again upon loopd restart. To get the " +
+		"default values, use `getparams` before any `setparams`.",
 	Flags: []cli.Flag{
 		cli.IntFlag{
 			Name: "sweeplimit",

liquidity/autoloop_testcontext_test.go

@@ -166,6 +166,12 @@ func newAutoloopTestCtx(t *testing.T, parameters Parameters,
 		MinimumConfirmations: loop.DefaultSweepConfTarget,
 		Lnd:                  &testCtx.lnd.LndServices,
 		Clock:                testCtx.testClock,
+		PutLiquidityParams: func(_ []byte) error {
+			return nil
+		},
+		FetchLiquidityParams: func() ([]byte, error) {
+			return nil, nil
+		},
 	}
 
 	// SetParameters needs to make a call to our mocked restrictions call,
@@ -179,7 +185,7 @@ func newAutoloopTestCtx(t *testing.T, parameters Parameters,
 	// Create a manager with our test config and set our starting set of
 	// parameters.
 	testCtx.manager = NewManager(cfg)
-	err := testCtx.manager.SetParameters(context.Background(), parameters)
+	err := testCtx.manager.setParameters(context.Background(), parameters)
 	assert.NoError(t, err)
 	<-done
 	return testCtx