Skip to content

Commit 77c7489

Browse files
authored
Merge pull request #5656 from cpuguy83/platform_format_add_config_check
tests: add validation that config has OSVersion set
2 parents e7185ac + d0d143b commit 77c7489

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

frontend/dockerfile/dockerfile_test.go

+9
Original file line numberDiff line numberDiff line change
@@ -9511,6 +9511,15 @@ EOF
95119511

95129512
require.NoError(t, err)
95139513

9514+
desc, provider, err := contentutil.ProviderFromRef(target)
9515+
require.NoError(t, err)
9516+
9517+
info, err := testutil.ReadImages(ctx, provider, desc)
9518+
require.NoError(t, err)
9519+
require.Len(t, info.Images, 2)
9520+
require.Equal(t, info.Images[0].Img.Platform.OSVersion, p1.OSVersion)
9521+
require.Equal(t, info.Images[1].Img.Platform.OSVersion, p2.OSVersion)
9522+
95149523
dt, err := os.ReadFile(filepath.Join(destDir, strings.Replace(p1Str, "/", "_", 1), "osversion"))
95159524
require.NoError(t, err)
95169525
require.Equal(t, p1.OSVersion+"\n", string(dt))

0 commit comments

Comments
 (0)