Skip to content

Commit e344f3a

Browse files
vvolandvvoland
committed
test/client: Close buildkit client
This fixes hang in these tests which caused a timeout. Signed-off-by: Paweł Gronowski <[email protected]> (cherry picked from commit 3b0aa69)
1 parent 0df0faa commit e344f3a

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

‎client/client_test.go‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7160,11 +7160,13 @@ func testExportAttestations(t *testing.T, sb integration.Sandbox) {
71607160
requiresLinux(t)
71617161
c, err := New(sb.Context(), sb.Address())
71627162
require.NoError(t, err)
7163+
defer c.Close()
71637164

71647165
registry, err := sb.NewRegistry()
71657166
if errors.Is(err, integration.ErrRequirements) {
71667167
t.Skip(err.Error())
71677168
}
7169+
require.NoError(t, err)
71687170

71697171
ps := []ocispecs.Platform{
71707172
platforms.MustParse("linux/amd64"),
@@ -7480,11 +7482,13 @@ func testAttestationDefaultSubject(t *testing.T, sb integration.Sandbox) {
74807482
requiresLinux(t)
74817483
c, err := New(sb.Context(), sb.Address())
74827484
require.NoError(t, err)
7485+
defer c.Close()
74837486

74847487
registry, err := sb.NewRegistry()
74857488
if errors.Is(err, integration.ErrRequirements) {
74867489
t.Skip(err.Error())
74877490
}
7491+
require.NoError(t, err)
74887492

74897493
ps := []ocispecs.Platform{
74907494
platforms.MustParse("linux/amd64"),
@@ -7617,11 +7621,13 @@ func testAttestationBundle(t *testing.T, sb integration.Sandbox) {
76177621
requiresLinux(t)
76187622
c, err := New(sb.Context(), sb.Address())
76197623
require.NoError(t, err)
7624+
defer c.Close()
76207625

76217626
registry, err := sb.NewRegistry()
76227627
if errors.Is(err, integration.ErrRequirements) {
76237628
t.Skip(err.Error())
76247629
}
7630+
require.NoError(t, err)
76257631

76267632
ps := []ocispecs.Platform{
76277633
platforms.MustParse("linux/amd64"),
@@ -7766,11 +7772,13 @@ func testSBOMScan(t *testing.T, sb integration.Sandbox) {
77667772
requiresLinux(t)
77677773
c, err := New(sb.Context(), sb.Address())
77687774
require.NoError(t, err)
7775+
defer c.Close()
77697776

77707777
registry, err := sb.NewRegistry()
77717778
if errors.Is(err, integration.ErrRequirements) {
77727779
t.Skip(err.Error())
77737780
}
7781+
require.NoError(t, err)
77747782

77757783
p := platforms.MustParse("linux/amd64")
77767784
pk := platforms.Format(p)
@@ -8042,11 +8050,13 @@ func testSBOMScanSingleRef(t *testing.T, sb integration.Sandbox) {
80428050
requiresLinux(t)
80438051
c, err := New(sb.Context(), sb.Address())
80448052
require.NoError(t, err)
8053+
defer c.Close()
80458054

80468055
registry, err := sb.NewRegistry()
80478056
if errors.Is(err, integration.ErrRequirements) {
80488057
t.Skip(err.Error())
80498058
}
8059+
require.NoError(t, err)
80508060

80518061
p := platforms.DefaultSpec()
80528062
pk := platforms.Format(p)
@@ -8206,11 +8216,13 @@ func testSBOMSupplements(t *testing.T, sb integration.Sandbox) {
82068216
requiresLinux(t)
82078217
c, err := New(sb.Context(), sb.Address())
82088218
require.NoError(t, err)
8219+
defer c.Close()
82098220

82108221
registry, err := sb.NewRegistry()
82118222
if errors.Is(err, integration.ErrRequirements) {
82128223
t.Skip(err.Error())
82138224
}
8225+
require.NoError(t, err)
82148226

82158227
p := platforms.MustParse("linux/amd64")
82168228
pk := platforms.Format(p)

0 commit comments

Comments
 (0)