test: add missing error checks in cache tests

marquiz · marquiz · commit 2bb85fd88c62 · 2024-01-11T15:26:03.000+02:00
Signed-off-by: Markus Lehtonen &lt;markus.lehtonen@intel.com&gt;
diff --git a/pkg/cdi/cache_test.go b/pkg/cdi/cache_test.go
@@ -177,6 +177,10 @@ devices:
 				}
 			} else {
 				dir, err = mkTestDir(t, nil)
+				if err != nil {
+					t.Errorf("failed to create test directory: %v", err)
+					return
+				}
 			}
 
 			cache, err = NewCache(WithSpecDirs(
@@ -554,6 +558,7 @@ devices:
 							opts = append(opts, WithAutoRefresh(false))
 						}
 						cache, err = NewCache(opts...)
+						require.NoError(t, err)
 						require.NotNil(t, cache)
 					} else {
 						err = updateSpecDirs(t, dir, update.etc, update.run)

Original file line number	Diff line number	Diff line change
`@@ -177,6 +177,10 @@ devices:`
`177`	`177`	`}`
`178`	`178`	`} else {`
`179`	`179`	`dir, err = mkTestDir(t, nil)`
	`180`	`+ if err != nil {`
	`181`	`+ t.Errorf("failed to create test directory: %v", err)`
	`182`	`+ return`
	`183`	`+ }`
`180`	`184`	`}`
`181`	`185`
`182`	`186`	`cache, err = NewCache(WithSpecDirs(`
`@@ -554,6 +558,7 @@ devices:`
`554`	`558`	`opts = append(opts, WithAutoRefresh(false))`
`555`	`559`	`}`
`556`	`560`	`cache, err = NewCache(opts...)`
	`561`	`+ require.NoError(t, err)`
`557`	`562`	`require.NotNil(t, cache)`
`558`	`563`	`} else {`
`559`	`564`	`err = updateSpecDirs(t, dir, update.etc, update.run)`