ITQC/containernetworking-plugins
24
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 34 Wiki Activity

Took out os.Remove() for socketPath and pidfilePath now that os.RemoveAll() is used

Browse Source
This commit is contained in:
Michael Cambria 2018-10-08 16:01:31 -04:00
parent 3d349e4645
commit ef913eadd5
1 changed files with 3 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
plugins/ipam/dhcp/dhcp_test.go
View File

@ -41,7 +41,7 @@ import (
)
func getTmpDir() (string, error) {
tmpDir, err := ioutil.TempDir(tmpPrefix, "dhcp")
tmpDir, err := ioutil.TempDir(cniDirPrefix, "dhcp")
if err == nil {
tmpDir = filepath.ToSlash(tmpDir)
}
@ -108,7 +108,7 @@ const (
hostVethName string = "dhcp0"
contVethName string = "eth0"
pidfilePath string = "/var/run/cni/dhcp-client.pid"
tmpPrefix = "/run/cni"
cniDirPrefix string = "/var/run/cni"
)
var socketPath string
@ -116,17 +116,10 @@ var tmpDir string
var err error
var _ = BeforeSuite(func() {
os.Remove(socketPath)
os.Remove(pidfilePath)
err := os.MkdirAll(tmpPrefix, 0700)
err := os.MkdirAll(cniDirPrefix, 0700)
Expect(err).NotTo(HaveOccurred())
})
var _ = AfterSuite(func() {
os.Remove(socketPath)
os.Remove(pidfilePath)
})
var _ = Describe("DHCP Operations", func() {
var originalNS, targetNS ns.NetNS
var dhcpServerStopCh chan bool
@ -206,7 +199,6 @@ var _ = Describe("DHCP Operations", func() {
Expect(err).NotTo(HaveOccurred())
// Start the DHCP client daemon
os.MkdirAll(pidfilePath, 0755)
dhcpPluginPath, err := exec.LookPath("dhcp")
Expect(err).NotTo(HaveOccurred())
clientCmd = exec.Command(dhcpPluginPath, "daemon", "-socketpath", socketPath)
@ -229,8 +221,6 @@ var _ = Describe("DHCP Operations", func() {
Expect(originalNS.Close()).To(Succeed())
Expect(targetNS.Close()).To(Succeed())
os.Remove(socketPath)
os.Remove(pidfilePath)
defer os.RemoveAll(tmpDir)
})