From bfd62ceba90bf45e3e4025e140f53d7860be7508 Mon Sep 17 00:00:00 2001 From: Michael Crosby Date: Wed, 16 Aug 2017 16:29:43 -0400 Subject: [PATCH] Defer unlock for testing daemon Make sure we use defer when unlocking the daemon during testing. Signed-off-by: Michael Crosby --- daemon_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/daemon_test.go b/daemon_test.go index 577d4550e..3bc9f6955 100644 --- a/daemon_test.go +++ b/daemon_test.go @@ -60,7 +60,7 @@ func (d *daemon) waitForStart(ctx context.Context) (*Client, error) { func (d *daemon) Stop() error { d.Lock() - d.Unlock() + defer d.Unlock() if d.cmd == nil { return errors.New("daemon is not running") } @@ -69,7 +69,7 @@ func (d *daemon) Stop() error { func (d *daemon) Kill() error { d.Lock() - d.Unlock() + defer d.Unlock() if d.cmd == nil { return errors.New("daemon is not running") } @@ -78,7 +78,7 @@ func (d *daemon) Kill() error { func (d *daemon) Wait() error { d.Lock() - d.Unlock() + defer d.Unlock() if d.cmd == nil { return errors.New("daemon is not running") } @@ -87,7 +87,7 @@ func (d *daemon) Wait() error { func (d *daemon) Restart() error { d.Lock() - d.Unlock() + defer d.Unlock() if d.cmd == nil { return errors.New("daemon is not running") }