Skip to content

Commit 3b0aa69

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]>
1 parent db6342f commit 3b0aa69

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
@@ -7356,11 +7356,13 @@ func testExportAttestations(t *testing.T, sb integration.Sandbox) {
73567356
requiresLinux(t)
73577357
c, err := New(sb.Context(), sb.Address())
73587358
require.NoError(t, err)
7359+
defer c.Close()
73597360

73607361
registry, err := sb.NewRegistry()
73617362
if errors.Is(err, integration.ErrRequirements) {
73627363
t.Skip(err.Error())
73637364
}
7365+
require.NoError(t, err)
73647366

73657367
ps := []ocispecs.Platform{
73667368
platforms.MustParse("linux/amd64"),
@@ -7676,11 +7678,13 @@ func testAttestationDefaultSubject(t *testing.T, sb integration.Sandbox) {
76767678
requiresLinux(t)
76777679
c, err := New(sb.Context(), sb.Address())
76787680
require.NoError(t, err)
7681+
defer c.Close()
76797682

76807683
registry, err := sb.NewRegistry()
76817684
if errors.Is(err, integration.ErrRequirements) {
76827685
t.Skip(err.Error())
76837686
}
7687+
require.NoError(t, err)
76847688

76857689
ps := []ocispecs.Platform{
76867690
platforms.MustParse("linux/amd64"),
@@ -7813,11 +7817,13 @@ func testAttestationBundle(t *testing.T, sb integration.Sandbox) {
78137817
requiresLinux(t)
78147818
c, err := New(sb.Context(), sb.Address())
78157819
require.NoError(t, err)
7820+
defer c.Close()
78167821

78177822
registry, err := sb.NewRegistry()
78187823
if errors.Is(err, integration.ErrRequirements) {
78197824
t.Skip(err.Error())
78207825
}
7826+
require.NoError(t, err)
78217827

78227828
ps := []ocispecs.Platform{
78237829
platforms.MustParse("linux/amd64"),
@@ -7962,11 +7968,13 @@ func testSBOMScan(t *testing.T, sb integration.Sandbox) {
79627968
requiresLinux(t)
79637969
c, err := New(sb.Context(), sb.Address())
79647970
require.NoError(t, err)
7971+
defer c.Close()
79657972

79667973
registry, err := sb.NewRegistry()
79677974
if errors.Is(err, integration.ErrRequirements) {
79687975
t.Skip(err.Error())
79697976
}
7977+
require.NoError(t, err)
79707978

79717979
p := platforms.MustParse("linux/amd64")
79727980
pk := platforms.Format(p)
@@ -8238,11 +8246,13 @@ func testSBOMScanSingleRef(t *testing.T, sb integration.Sandbox) {
82388246
requiresLinux(t)
82398247
c, err := New(sb.Context(), sb.Address())
82408248
require.NoError(t, err)
8249+
defer c.Close()
82418250

82428251
registry, err := sb.NewRegistry()
82438252
if errors.Is(err, integration.ErrRequirements) {
82448253
t.Skip(err.Error())
82458254
}
8255+
require.NoError(t, err)
82468256

82478257
p := platforms.DefaultSpec()
82488258
pk := platforms.Format(p)
@@ -8402,11 +8412,13 @@ func testSBOMSupplements(t *testing.T, sb integration.Sandbox) {
84028412
requiresLinux(t)
84038413
c, err := New(sb.Context(), sb.Address())
84048414
require.NoError(t, err)
8415+
defer c.Close()
84058416

84068417
registry, err := sb.NewRegistry()
84078418
if errors.Is(err, integration.ErrRequirements) {
84088419
t.Skip(err.Error())
84098420
}
8421+
require.NoError(t, err)
84108422

84118423
p := platforms.MustParse("linux/amd64")
84128424
pk := platforms.Format(p)

0 commit comments

Comments
 (0)