From a393e95018742e2485f08ff68f84c149292072fa Mon Sep 17 00:00:00 2001 From: Cao Shufeng Date: Mon, 8 May 2017 17:20:25 +0800 Subject: [PATCH] Remove leaked tmp file in unit tests --- pkg/util/iptables/iptables_test.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/pkg/util/iptables/iptables_test.go b/pkg/util/iptables/iptables_test.go index 17cd665f04..b38714415e 100644 --- a/pkg/util/iptables/iptables_test.go +++ b/pkg/util/iptables/iptables_test.go @@ -1042,6 +1042,7 @@ func TestRestoreAll(t *testing.T) { }, } runner := newInternal(&fexec, dbus.NewFake(nil, nil), ProtocolIpv4, TestLockfilePath) + defer os.Remove(TestLockfilePath) defer runner.Destroy() err := runner.RestoreAll([]byte{}, NoFlushTables, RestoreCounters) @@ -1086,6 +1087,7 @@ func TestRestoreAllWait(t *testing.T) { }, } runner := newInternal(&fexec, dbus.NewFake(nil, nil), ProtocolIpv4, TestLockfilePath) + defer os.Remove(TestLockfilePath) defer runner.Destroy() err := runner.RestoreAll([]byte{}, NoFlushTables, RestoreCounters) @@ -1131,6 +1133,7 @@ func TestRestoreAllWaitOldIptablesRestore(t *testing.T) { }, } runner := newInternal(&fexec, dbus.NewFake(nil, nil), ProtocolIpv4, TestLockfilePath) + defer os.Remove(TestLockfilePath) defer runner.Destroy() err := runner.RestoreAll([]byte{}, NoFlushTables, RestoreCounters) @@ -1177,6 +1180,7 @@ func TestRestoreAllGrabNewLock(t *testing.T) { } runner := newInternal(&fexec, dbus.NewFake(nil, nil), ProtocolIpv4, TestLockfilePath) + defer os.Remove(TestLockfilePath) defer runner.Destroy() // Grab the /run lock and ensure the RestoreAll fails @@ -1219,6 +1223,7 @@ func TestRestoreAllGrabOldLock(t *testing.T) { } runner := newInternal(&fexec, dbus.NewFake(nil, nil), ProtocolIpv4, TestLockfilePath) + defer os.Remove(TestLockfilePath) defer runner.Destroy() // Grab the abstract @xtables socket