fixes

database/integration_test.go

@@ -910,7 +910,7 @@ func testJWKs(t *testing.T, db Interface, resources *Resources) {
 			t.Errorf("unable to get jwk %s: %v", jkPub.KeyID(), err)
 		}
 
-		if !cmp.Equal(jkPub, got) {
+		if !cmp.Equal(jkPub, got, testutils.JwkOpts) {
 			t.Errorf("GetJWK() is %v, want %v", got, jkPub)
 		}
 	}

database/jwk/get.go

@@ -5,9 +5,10 @@ package jwk
 import (
 	"context"
 
+	"github.com/lestrrat-go/jwx/jwk"
+
 	"github.com/go-vela/server/constants"
 	"github.com/go-vela/server/database/types"
-	"github.com/lestrrat-go/jwx/jwk"
 )
 
 // GetActiveJWK gets a JWK by UUID (kid) from the database if active.

database/jwk/get_test.go

@@ -79,7 +79,7 @@ func TestJWK_Engine_GetJWK(t *testing.T) {
 				t.Errorf("GetActiveJWK for %s returned err: %v", test.name, err)
 			}
 
-			if diff := cmp.Diff(test.want, got, jwkOpts); diff != "" {
+			if diff := cmp.Diff(test.want, got, testutils.JwkOpts); diff != "" {
 				t.Errorf("GetActiveJWK mismatch (-want +got):\n%s", diff)
 			}
 		})

database/jwk/jwk_test.go

@@ -7,8 +7,6 @@ import (
 	"testing"
 
 	"github.com/DATA-DOG/go-sqlmock"
-	"github.com/google/go-cmp/cmp"
-	"github.com/lestrrat-go/jwx/jwk"
 	"github.com/sirupsen/logrus"
 	"gorm.io/driver/postgres"
 	"gorm.io/driver/sqlite"
@@ -163,26 +161,3 @@ func testSqlite(t *testing.T) *engine {
 
 	return _engine
 }
-
-var jwkOpts = cmp.Options{
-	cmp.FilterValues(func(x, y interface{}) bool {
-		_, xOk := x.(jwk.RSAPublicKey)
-		_, yOk := y.(jwk.RSAPublicKey)
-		return xOk && yOk
-	}, cmp.Comparer(func(x, y interface{}) bool {
-		xJWK := x.(jwk.RSAPublicKey)
-		yJWK := y.(jwk.RSAPublicKey)
-
-		var rawXKey, rawYKey interface{}
-
-		if err := xJWK.Raw(&rawXKey); err != nil {
-			return false
-		}
-
-		if err := yJWK.Raw(&rawYKey); err != nil {
-			return false
-		}
-
-		return reflect.DeepEqual(rawXKey, rawYKey) && xJWK.KeyID() == yJWK.KeyID()
-	})),
-}

database/jwk/list.go

@@ -5,9 +5,10 @@ package jwk
 import (
 	"context"
 
+	"github.com/lestrrat-go/jwx/jwk"
+
 	"github.com/go-vela/server/constants"
 	"github.com/go-vela/server/database/types"
-	"github.com/lestrrat-go/jwx/jwk"
 )
 
 // ListJWKs gets a list of all configured JWKs from the database.

database/testutils/api_resources.go

@@ -6,12 +6,13 @@ import (
 	"crypto/rand"
 	"crypto/rsa"
 
+	"github.com/google/uuid"
+	"github.com/lestrrat-go/jwx/jwk"
+
 	api "github.com/go-vela/server/api/types"
 	"github.com/go-vela/server/api/types/actions"
 	"github.com/go-vela/types/library"
 	"github.com/go-vela/types/raw"
-	"github.com/google/uuid"
-	"github.com/lestrrat-go/jwx/jwk"
 )
 
 // API TEST RESOURCES

database/testutils/mock_args.go

@@ -4,7 +4,11 @@ package testutils
 
 import (
 	"database/sql/driver"
+	"reflect"
 	"time"
+
+	"github.com/google/go-cmp/cmp"
+	"github.com/lestrrat-go/jwx/jwk"
 )
 
 // This will be used with the github.com/DATA-DOG/go-sqlmock library to compare values
@@ -33,3 +37,26 @@ func (t NowTimestamp) Match(v driver.Value) bool {
 
 	return now-ts < 10
 }
+
+var JwkOpts = cmp.Options{
+	cmp.FilterValues(func(x, y interface{}) bool {
+		_, xOk := x.(jwk.RSAPublicKey)
+		_, yOk := y.(jwk.RSAPublicKey)
+		return xOk && yOk
+	}, cmp.Comparer(func(x, y interface{}) bool {
+		xJWK := x.(jwk.RSAPublicKey)
+		yJWK := y.(jwk.RSAPublicKey)
+
+		var rawXKey, rawYKey interface{}
+
+		if err := xJWK.Raw(&rawXKey); err != nil {
+			return false
+		}
+
+		if err := yJWK.Raw(&rawYKey); err != nil {
+			return false
+		}
+
+		return reflect.DeepEqual(rawXKey, rawYKey) && xJWK.KeyID() == yJWK.KeyID()
+	})),
+}