diff options
| author | Paul Holzinger <pholzing@redhat.com> | 2022-04-27 14:28:38 +0200 |
|---|---|---|
| committer | Paul Holzinger <pholzing@redhat.com> | 2022-04-29 14:06:38 +0200 |
| commit | 69c479b16e19f4f919fa820aeafe90cb113b8e0a (patch) | |
| tree | 97047161fa35ce1a1fc1a1d5d555422411932ae8 /test/utils/common_function_test.go | |
| parent | ab3e072a0c3d321fd12cbd1f6ef8e322c6d9214a (diff) | |
| download | podman-69c479b16e19f4f919fa820aeafe90cb113b8e0a.tar.gz podman-69c479b16e19f4f919fa820aeafe90cb113b8e0a.tar.bz2 podman-69c479b16e19f4f919fa820aeafe90cb113b8e0a.zip | |
enable errcheck linter
The errcheck linter makes sure that errors are always check and not
ignored by accident. It spotted a lot of unchecked errors, mostly in the
tests but also some real problem in the code.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Diffstat (limited to 'test/utils/common_function_test.go')
| -rw-r--r-- | test/utils/common_function_test.go | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/test/utils/common_function_test.go b/test/utils/common_function_test.go index a73d75490..7092e40a1 100644 --- a/test/utils/common_function_test.go +++ b/test/utils/common_function_test.go @@ -113,8 +113,10 @@ var _ = Describe("Common functions test", func() { Expect(err).To(BeNil(), "Can not find the JSON file after we write it.") defer read.Close() - bytes, _ := ioutil.ReadAll(read) - json.Unmarshal(bytes, compareData) + bytes, err := ioutil.ReadAll(read) + Expect(err).ToNot(HaveOccurred()) + err = json.Unmarshal(bytes, compareData) + Expect(err).ToNot(HaveOccurred()) Expect(reflect.DeepEqual(testData, compareData)).To(BeTrue(), "Data changed after we store it to file.") }) |