tests: use subtests

[safcm/safcm.git] / cmd / safcm-remote / sync / packages_debian_test.go

diff --git a/cmd/safcm-remote/sync/packages_debian_test.go b/cmd/safcm-remote/sync/packages_debian_test.go
index f67dda9484bd5e4a5821138a7e1e4dc0014fb11a..51b6b746865da76bee636a81e7717a9d1ca9b995 100644 (file)
--- a/cmd/safcm-remote/sync/packages_debian_test.go
+++ b/cmd/safcm-remote/sync/packages_debian_test.go
@@ -300,9 +300,9 @@ install ok installed        golang-1.16-src
        }
 
        for _, tc := range tests {
+               t.Run(tc.name, func(t *testing.T) {
                s, res := prepareSync(tc.req, &testRunner{
                        t:         t,
-                       name:      tc.name,
                        expCmds:   tc.expCmds,
                        resStdout: tc.stdout,
                        resStderr: tc.stderr,
@@ -312,18 +312,19 @@ install ok installed      golang-1.16-src
                err := s.syncPackagesDebian()
                // Ugly but the simplest way to compare errors (including nil)
                if fmt.Sprintf("%s", err) != fmt.Sprintf("%s", tc.expErr) {
-                       t.Errorf("%s: err = %#v, want %#v",
-                               tc.name, err, tc.expErr)
+                       t.Errorf("err = %#v, want %#v",
+                               err, tc.expErr)
                }
                dbg := res.Wait()
 
                if !reflect.DeepEqual(tc.expResp, s.resp) {
-                       t.Errorf("%s: resp: %s", tc.name,
+                       t.Errorf("resp: %s",
                                cmp.Diff(tc.expResp, s.resp))
                }
                if !reflect.DeepEqual(tc.expDbg, dbg) {
-                       t.Errorf("%s: dbg: %s", tc.name,
+                       t.Errorf("dbg: %s",
                                cmp.Diff(tc.expDbg, dbg))
                }
+               })
        }
 }