Skip to content

Commit

Permalink
upgrades: unskip TestIsAtLeastVersionBuiltin
Browse files Browse the repository at this point in the history
Release note: None
  • Loading branch information
rafiss committed Mar 31, 2023
1 parent b94053b commit 7aa3c9d
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions pkg/upgrade/upgrades/builtins_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ import (
"github.com/cockroachdb/cockroach/pkg/base"
"github.com/cockroachdb/cockroach/pkg/clusterversion"
"github.com/cockroachdb/cockroach/pkg/server"
"github.com/cockroachdb/cockroach/pkg/testutils/skip"
"github.com/cockroachdb/cockroach/pkg/testutils/sqlutils"
"github.com/cockroachdb/cockroach/pkg/testutils/testcluster"
"github.com/cockroachdb/cockroach/pkg/util/leaktest"
Expand All @@ -28,14 +27,12 @@ func TestIsAtLeastVersionBuiltin(t *testing.T) {
defer leaktest.AfterTest(t)()
defer log.Scope(t).Close(t)

skip.WithIssue(t, 95530, "bump minBinary to 22.2. Skip 22.2 mixed-version tests for future cleanup")

clusterArgs := base.TestClusterArgs{
ServerArgs: base.TestServerArgs{
Knobs: base.TestingKnobs{
Server: &server.TestingKnobs{
DisableAutomaticVersionUpgrade: make(chan struct{}),
BinaryVersionOverride: clusterversion.ByKey(clusterversion.TODODelete_V22_1),
BinaryVersionOverride: clusterversion.ByKey(clusterversion.V22_2),
},
},
},
Expand All @@ -49,7 +46,7 @@ func TestIsAtLeastVersionBuiltin(t *testing.T) {
)
defer tc.Stopper().Stop(ctx)

v := clusterversion.ByKey(clusterversion.TODODelete_V22_2Start).String()
v := clusterversion.ByKey(clusterversion.V23_1Start).String()
// Check that the builtin returns false when comparing against 22.1-2
// version because we are still on 22.1-0.
sqlDB.CheckQueryResults(t, "SELECT crdb_internal.is_at_least_version('"+v+"')", [][]string{{"false"}})
Expand Down

0 comments on commit 7aa3c9d

Please sign in to comment.