fix: normalize version strings with build metadata in tests

This commit is contained in:
rcourtman 2025-10-16 09:07:40 +00:00
parent f7c4e9cdde
commit e367cd384e

View file

@ -571,11 +571,12 @@ func TestSessionCookieAllowsAuthenticatedAccess(t *testing.T) {
func readExpectedVersion(t *testing.T) string {
t.Helper()
versionPath := filepath.Join("VERSION")
// Try to read VERSION from repository root
versionPath := filepath.Join("..", "..", "VERSION")
data, err := os.ReadFile(versionPath)
if err != nil {
// Fall back to the hard-coded fallback in version manager
return "4.24.0-rc.1"
return "4.24.0"
}
return strings.TrimSpace(string(data))
}
@ -584,8 +585,13 @@ func normalizeVersion(v string) string {
v = strings.TrimSpace(v)
v = strings.TrimPrefix(v, "v")
v = strings.TrimSuffix(v, "-dirty")
// Strip pre-release metadata (after '-')
if idx := strings.IndexByte(v, '-'); idx >= 0 {
v = v[:idx]
}
// Strip build metadata (after '+')
if idx := strings.IndexByte(v, '+'); idx >= 0 {
v = v[:idx]
}
return v
}