feat: update style to match github

api/buildstatus/build_status_controller_test.go

@@ -102,7 +102,7 @@ func TestGetBuildStatus(t *testing.T) {
 		expected := []byte("badge")
 
 		fakeBuildStatus.EXPECT().
-			GetBadge(gomock.Any(), gomock.Any()).
+			GetBadge(gomock.Any(), gomock.Any(), gomock.Any()).
 			Return(expected, nil).
 			Times(1)
 
@@ -129,7 +129,7 @@ func TestGetBuildStatus(t *testing.T) {
 		var actualPipeline v1.RadixPipelineType
 
 		fakeBuildStatus.EXPECT().
-			GetBadge(gomock.Any(), gomock.Any()).
+			GetBadge(gomock.Any(), gomock.Any(), gomock.Any()).
 			DoAndReturn(func(c v1.RadixJobCondition, p v1.RadixPipelineType) ([]byte, error) {
 				actualCondition = c
 				actualPipeline = p
@@ -159,7 +159,7 @@ func TestGetBuildStatus(t *testing.T) {
 		var actualPipeline v1.RadixPipelineType
 
 		fakeBuildStatus.EXPECT().
-			GetBadge(gomock.Any(), gomock.Any()).
+			GetBadge(gomock.Any(), gomock.Any(), gomock.Any()).
 			DoAndReturn(func(c v1.RadixJobCondition, p v1.RadixPipelineType) ([]byte, error) {
 				actualCondition = c
 				actualPipeline = p
@@ -189,7 +189,7 @@ func TestGetBuildStatus(t *testing.T) {
 		var actualPipeline v1.RadixPipelineType
 
 		fakeBuildStatus.EXPECT().
-			GetBadge(gomock.Any(), gomock.Any()).
+			GetBadge(gomock.Any(), gomock.Any(), gomock.Any()).
 			DoAndReturn(func(c v1.RadixJobCondition, p v1.RadixPipelineType) ([]byte, error) {
 				actualCondition = c
 				actualPipeline = p
@@ -216,7 +216,7 @@ func TestGetBuildStatus(t *testing.T) {
 		fakeBuildStatus := mock.NewMockPipelineBadge(ctrl)
 
 		fakeBuildStatus.EXPECT().
-			GetBadge(gomock.Any(), gomock.Any()).
+			GetBadge(gomock.Any(), gomock.Any(), gomock.Any()).
 			Return(nil, errors.New("error")).
 			Times(1)
 

api/buildstatus/buildstatus_handler.go

@@ -40,7 +40,7 @@ func (handler BuildStatusHandler) GetBuildStatusForApplication(ctx context.Conte
 		buildCondition = latestPipelineJob.Status.Condition
 	}
 
-	output, err = handler.pipelineBadge.GetBadge(buildCondition, v1.RadixPipelineType(pipeline))
+	output, err = handler.pipelineBadge.GetBadge(ctx, buildCondition, v1.RadixPipelineType(pipeline))
 	if err != nil {
 		return nil, err
 	}

api/buildstatus/models/buildstatus.go

@@ -2,12 +2,14 @@ package models
 
 import (
 	"bytes"
+	"context"
 	_ "embed"
 	"errors"
 	"html/template"
 	"strings"
 
 	v1 "github.com/equinor/radix-operator/pkg/apis/radix/v1"
+	"github.com/rs/zerolog/log"
 )
 
 // embed https://golang.org/pkg/embed/ - For embedding a single file, a variable of type []byte or string is often best
@@ -33,7 +35,7 @@ const (
 )
 
 type PipelineBadge interface {
-	GetBadge(condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error)
+	GetBadge(ctx context.Context, condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error)
 }
 
 func NewPipelineBadge() PipelineBadge {
@@ -54,16 +56,16 @@ type pipelineBadge struct {
 	badgeTemplate string
 }
 
-func (rbs *pipelineBadge) GetBadge(condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error) {
-	return rbs.getBadge(condition, pipeline)
+func (rbs *pipelineBadge) GetBadge(ctx context.Context, condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error) {
+	return rbs.getBadge(ctx, condition, pipeline)
 }
 
-func (rbs *pipelineBadge) getBadge(condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error) {
+func (rbs *pipelineBadge) getBadge(ctx context.Context, condition v1.RadixJobCondition, pipeline v1.RadixPipelineType) ([]byte, error) {
 	operation := translatePipeline(pipeline)
 	status := translateCondition(condition)
 	color := getColor(condition)
-	operationWidth := calculateWidth(10, operation)
-	statusWidth := calculateWidth(10, status)
+	operationWidth := calculateWidth(6, operation)
+	statusWidth := calculateWidth(6, status)
 	badgeData := pipelineBadgeData{
 		Operation:      operation,
 		OperationWidth: operationWidth,
@@ -72,6 +74,8 @@ func (rbs *pipelineBadge) getBadge(condition v1.RadixJobCondition, pipeline v1.R
 		StatusWidth:    statusWidth,
 	}
 
+	log.Ctx(ctx).Trace().Interface("badge", badgeData).Msg("Rendering badge")
+
 	funcMap := template.FuncMap{"sum": TemplateSum}
 	svgTemplate := template.New("status-badge.svg").Funcs(funcMap)
 	_, err := svgTemplate.Parse(rbs.badgeTemplate)