Add failing test cases for current

jsturtevant · jsturtevant · commit 5ae90c60c7c5 · 2022-02-01T08:21:13.000-08:00
diff --git a/platforms/compare_test.go b/platforms/compare_test.go
@@ -17,10 +17,16 @@
 package platforms
 
 import (
+	"fmt"
+	"golang.org/x/sys/windows"
 	"testing"
 )
 
 func TestOnly(t *testing.T) {
+	// will not work on linux. Demonstrates that build strings don't work with Only() and other variants
+    major, minor, build := windows.RtlGetNtVersionNumbers()
+	buildStr := fmt.Sprintf("%d.%d.%d", major, minor, build)
+
 	for _, tc := range []struct {
 		platform string
 		matches  map[bool][]string
@@ -75,6 +81,18 @@ func TestOnly(t *testing.T) {
 				},
 			},
 		},
+		{
+			platform: fmt.Sprintf("windows/amd64//%s", buildStr+".1"),
+			matches: map[bool][]string{
+				true: { fmt.Sprintf("windows/amd64//%s", buildStr)},
+				false: {
+					"linux/amd64",
+					"linux/arm/v7",
+					"linux/arm64",
+					"windows/arm",
+				},
+			},
+		},
 		{
 			platform: "windows/amd64",
 			matches: map[bool][]string{
diff --git a/platforms/defaults_windows_test.go b/platforms/defaults_windows_test.go
@@ -47,6 +47,30 @@ func TestDefault(t *testing.T) {
 	}
 }
 
+func TestDefaultMatchComparer(t *testing.T) {
+	defaultMatcher := Default()
+
+	for _, test := range []struct {
+		platform imagespec.Platform
+		match    bool
+	}{
+		{
+			platform: DefaultSpec(),
+			match:    true,
+		},
+		{
+			platform: imagespec.Platform{
+				OS: "linux",
+				Architecture: runtime.GOARCH,
+			},
+			match:    true,
+		},
+	} {
+		assert.Equal(t, test.match, defaultMatcher.Match(test.platform))
+	}
+
+}
+
 func TestMatchComparerMatch(t *testing.T) {
 	major, minor, build := windows.RtlGetNtVersionNumbers()
 	buildStr := fmt.Sprintf("%d.%d.%d", major, minor, build)
