setup-envtest: select the newest Kubernetes by default

The order inadvertently changed in 0ddbc52.

Fixes: #3379
Signed-off-by: Chris Bandy <bandy.chris@gmail.com>

Author: cbandy

tools/setup-envtest/remote/http_client.go

@@ -88,7 +88,7 @@ func (c *HTTPClient) ListVersions(ctx context.Context) ([]versions.Set, error) {
 	}
 	// sort in inverse order so that the newest one is first
 	slices.SortStableFunc(res, func(i, j versions.Set) int {
-		return i.Version.Compare(j.Version)
+		return j.Version.Compare(i.Version)
 	})
 
 	return res, nil

tools/setup-envtest/store/store.go

@@ -105,7 +105,8 @@ func (s *Store) List(ctx context.Context, matching Filter) ([]Item, error) {
 
 	slices.SortStableFunc(res, func(i, j Item) int {
 		if !i.Version.Matches(j.Version) {
-			return i.Version.Compare(j.Version)
+			// sort in inverse order so that the newest one is first
+			return j.Version.Compare(i.Version)
 		}
 		return orderPlatforms(i.Platform, j.Platform)
 	})

tools/setup-envtest/versions/misc_test.go

@@ -17,6 +17,9 @@ limitations under the License.
 package versions_test
 
 import (
+	"math/rand/v2"
+	"slices"
+
 	. "github.com/onsi/ginkgo/v2"
 	. "github.com/onsi/gomega"
 
@@ -44,6 +47,23 @@ var _ = Describe("Concrete", func() {
 		Specify("newer major should be newer", func() {
 			Expect(ver1163.Compare(Concrete{Major: 0, Minor: 16, Patch: 3})).To(Equal(1))
 		})
+
+		Describe("sorting", func() {
+			ver16 := Concrete{Major: 1, Minor: 16}
+			ver17 := Concrete{Major: 1, Minor: 17}
+			many := []Concrete{ver16, ver17, ver1163}
+
+			BeforeEach(func() {
+				rand.Shuffle(len(many), func(i, j int) {
+					many[i], many[j] = many[j], many[i]
+				})
+			})
+
+			Specify("newer versions are later", func() {
+				slices.SortStableFunc(many, Concrete.Compare)
+				Expect(many).To(Equal([]Concrete{ver16, ver1163, ver17}))
+			})
+		})
 	})
 })