From 343d233b4f4ae4bbd78fec472ce2d80ca70980b3 Mon Sep 17 00:00:00 2001 From: Shengjing Zhu Date: Mon, 25 Jan 2021 01:12:20 +0800 Subject: [PATCH] bandwidth: fix panic in tests Check error before using return value Signed-off-by: Shengjing Zhu --- plugins/meta/bandwidth/bandwidth_linux_test.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/plugins/meta/bandwidth/bandwidth_linux_test.go b/plugins/meta/bandwidth/bandwidth_linux_test.go index c3d9cb01..a75b2729 100644 --- a/plugins/meta/bandwidth/bandwidth_linux_test.go +++ b/plugins/meta/bandwidth/bandwidth_linux_test.go @@ -936,6 +936,7 @@ var _ = Describe("bandwidth test", func() { containerWithTbfRes, _, err = testutils.CmdAdd(containerWithTbfNS.Path(), "dummy", containerWithTbfIFName, []byte(ptpConf), func() error { r, err := invoke.DelegateAdd(context.TODO(), "ptp", []byte(ptpConf), nil) + Expect(err).NotTo(HaveOccurred()) Expect(r.Print()).To(Succeed()) return err @@ -944,6 +945,7 @@ var _ = Describe("bandwidth test", func() { containerWithoutTbfRes, _, err = testutils.CmdAdd(containerWithoutTbfNS.Path(), "dummy2", containerWithoutTbfIFName, []byte(ptpConf), func() error { r, err := invoke.DelegateAdd(context.TODO(), "ptp", []byte(ptpConf), nil) + Expect(err).NotTo(HaveOccurred()) Expect(r.Print()).To(Succeed()) return err @@ -1081,6 +1083,7 @@ var _ = Describe("bandwidth test", func() { containerWithTbfRes, _, err = testutils.CmdAdd(containerWithTbfNS.Path(), "dummy", containerWithTbfIFName, []byte(ptpConf), func() error { r, err := invoke.DelegateAdd(context.TODO(), "ptp", []byte(ptpConf), nil) + Expect(err).NotTo(HaveOccurred()) Expect(r.Print()).To(Succeed()) return err @@ -1089,6 +1092,7 @@ var _ = Describe("bandwidth test", func() { containerWithoutTbfRes, _, err = testutils.CmdAdd(containerWithoutTbfNS.Path(), "dummy2", containerWithoutTbfIFName, []byte(ptpConf), func() error { r, err := invoke.DelegateAdd(context.TODO(), "ptp", []byte(ptpConf), nil) + Expect(err).NotTo(HaveOccurred()) Expect(r.Print()).To(Succeed()) return err