Merge pull request firecracker-microvm#35 from xibz/issue_33

ensure tests have stopped VM upon completion
fangn2 · Dec 6, 2018 · 8b22e08 · 8b22e08
2 parents c52148e + 01a168e
commit 8b22e08
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/cmd/firectl/main_test.go b/cmd/firectl/main_test.go
@@ -147,6 +147,8 @@ func TestLogFiles(t *testing.T) {
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
+	defer m.StopVMM()
+
 	_, err = m.Init(ctx)
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)

diff --git a/machine_test.go b/machine_test.go
@@ -156,6 +156,7 @@ func TestStartVMM(t *testing.T) {
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
+	defer m.StopVMM()
 
 	timeout, cancel := context.WithTimeout(ctx, 250*time.Millisecond)
 	defer cancel()