Skip to content

Commit

Permalink
chore: update unit tests for new max versions
Browse files Browse the repository at this point in the history
  • Loading branch information
ralgozino committed Aug 23, 2024
1 parent 740e14c commit 940d32f
Showing 1 changed file with 32 additions and 32 deletions.
64 changes: 32 additions & 32 deletions internal/distribution/compatibility_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,28 +51,28 @@ func TestEKSClusterCheckIsCompatible(t *testing.T) {
expected: false,
},
{
name: "should return false if distribution version is greater than 1.27.7",
distributionVersion: "v1.27.8",
name: "should return false if distribution version is greater than 1.27.8",
distributionVersion: "v1.27.9",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.3",
distributionVersion: "v1.28.2",
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.4",
distributionVersion: "v1.28.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.28.2",
distributionVersion: "v1.28.3",
name: "should return false if distribution version is greater than 1.28.3",
distributionVersion: "v1.28.4",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.29.0 and less than 1.29.3",
distributionVersion: "v1.29.2",
name: "should return true if distribution version is greater than 1.29.0 and less than 1.29.4",
distributionVersion: "v1.29.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.29.2",
distributionVersion: "v1.29.3",
name: "should return false if distribution version is greater than 1.29.3",
distributionVersion: "v1.29.4",
expected: false,
},
}
Expand Down Expand Up @@ -133,33 +133,33 @@ func TestKFDDistributionCheckIsCompatible(t *testing.T) {
expected: false,
},
{
name: "should return true if distribution version is greater than 1.27.0 and less than 1.27.5",
name: "should return true if distribution version is greater than 1.27.0 and less than 1.27.8",
distributionVersion: "v1.27.2",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.27.7",
distributionVersion: "v1.27.8",
name: "should return false if distribution version is greater than 1.27.8",
distributionVersion: "v1.27.9",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.3",
distributionVersion: "v1.28.2",
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.4",
distributionVersion: "v1.28.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.28.2",
distributionVersion: "v1.28.3",
name: "should return false if distribution version is greater than 1.28.3",
distributionVersion: "v1.28.4",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.29.0 and less than 1.29.3",
distributionVersion: "v1.29.2",
name: "should return true if distribution version is greater than 1.29.0 and less than 1.29.4",
distributionVersion: "v1.29.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.29.2",
distributionVersion: "v1.29.3",
name: "should return false if distribution version is greater than 1.29.3",
distributionVersion: "v1.29.4",
expected: false,
},
}
Expand Down Expand Up @@ -220,33 +220,33 @@ func TestOnPremisesCheckIsCompatible(t *testing.T) {
expected: false,
},
{
name: "should return true if distribution version is greater than 1.27.0 and less than 1.27.5",
name: "should return true if distribution version is greater than 1.27.0 and less than 1.27.9",
distributionVersion: "v1.27.2",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.27.7",
distributionVersion: "v1.27.8",
name: "should return false if distribution version is greater than 1.27.8",
distributionVersion: "v1.27.9",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.3",
distributionVersion: "v1.28.2",
name: "should return true if distribution version is greater than 1.28.0 and less than 1.28.4",
distributionVersion: "v1.28.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.28.2",
distributionVersion: "v1.28.3",
name: "should return false if distribution version is greater than 1.28.3",
distributionVersion: "v1.28.4",
expected: false,
},
{
name: "should return true if distribution version is greater than 1.29.0 and less than 1.29.3",
distributionVersion: "v1.29.2",
name: "should return true if distribution version is greater than 1.29.1 and less than 1.29.4",
distributionVersion: "v1.29.3",
expected: true,
},
{
name: "should return false if distribution version is greater than 1.29.2",
distributionVersion: "v1.29.3",
name: "should return false if distribution version is greater than 1.29.3",
distributionVersion: "v1.29.4",
expected: false,
},
}
Expand Down

0 comments on commit 940d32f

Please sign in to comment.