Merge pull request #2250 from CiiDyR/fix/constraint-comparisons

fix: correctly compare semver according to spec 2.0.0

Author: FabianKramm

pkg/util/constraint/constraint.go

@@ -2,7 +2,6 @@ package constraint
 
 import (
 	"fmt"
-	"reflect"
 	"regexp"
 	"sort"
 	"strings"
@@ -180,11 +179,6 @@ func parseSingle(v string) (*Constraint, error) {
 
 func prereleaseCheck(v, c *Version) bool {
 	switch vPre, cPre := v.Prerelease() != "", c.Prerelease() != ""; {
-	case cPre && vPre:
-		// A constraint with a pre-release can only match a pre-release version
-		// with the same base segments.
-		return reflect.DeepEqual(c.Segments64(), v.Segments64())
-
 	case !cPre && vPre:
 		// OK
 	case cPre && !vPre:

pkg/util/constraint/constraint_test.go

@@ -72,10 +72,12 @@ func TestConstraintCheck(t *testing.T) {
 		{"~> 2.1.0-a", "2.2.0-alpha", false},
 		{"> 2.0", "2.1.0-beta", true},
 		{">= 2.1.0-a", "2.1.0-beta", true},
-		{">= 2.1.0-a", "2.1.1-beta", false},
+		{">= 2.1.0-a", "2.1.1-beta", true},
 		{">= 2.0.0", "2.1.0-beta", true},
+		{">= 2.0.0-beta.0", "2.0.0-beta.2", true},
+		{">= 2.0.0-beta.0", "2.0.1-beta.0", true},
+		{">= 2.0.1-beta.0", "2.0.0-beta.0", false},
 		{">= 2.1.0-a", "2.1.1", true},
-		{">= 2.1.0-a", "2.1.1-beta", false},
 		{">= 2.1.0-a", "2.1.0", true},
 		{"<= 2.1.0-a", "2.0.0", true},
 	}