chore: fixing compatibility tests (#4876)

cosmos · Oct 16, 2023 · 830982a · 830982a
1 parent e51b603
commit 830982a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/e2e/semverutil/semver.go b/e2e/semverutil/semver.go
@@ -25,7 +25,8 @@ func (fr FeatureReleases) IsSupported(versionStr string) bool {
 	const releasePrefix = "release-"
 	if strings.HasPrefix(versionStr, releasePrefix) {
 		versionStr = versionStr[len(releasePrefix):]
-		versionStr = strings.ReplaceAll(versionStr, "x", "0")
+		// replace x with 999 so the release version is always larger than the others in the release line.
+		versionStr = strings.ReplaceAll(versionStr, "x", "999")
 	}
 
 	// assume any non-semantic version formatted version supports the feature

diff --git a/e2e/testvalues/values.go b/e2e/testvalues/values.go
@@ -77,18 +77,20 @@ var MemoFeatureReleases = semverutil.FeatureReleases{
 
 // TotalEscrowFeatureReleases represents the releases the total escrow state entry was released in.
 var TotalEscrowFeatureReleases = semverutil.FeatureReleases{
+	MajorVersion: "v8",
 	MinorVersions: []string{
 		"v7.1",
 	},
 }
 
 // IbcErrorsFeatureReleases represents the releases the IBC module level errors was released in.
 var IbcErrorsFeatureReleases = semverutil.FeatureReleases{
-	MajorVersion: "v8.0",
+	MajorVersion: "v8",
 }
 
 // LocalhostClientFeatureReleases represents the releases the localhost client was released in.
 var LocalhostClientFeatureReleases = semverutil.FeatureReleases{
+	MajorVersion: "v8",
 	MinorVersions: []string{
 		"v7.1",
 	},