Skip to content

Commit 52d9242

Browse files
authored
refactor: drop MetricsConfig from ToolchainStatus type (#520)
remove the `Metrics` option from the helpers to generate `ToolchainConfig` in tests --------- Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
1 parent 79050e6 commit 52d9242

4 files changed

Lines changed: 5 additions & 23 deletions

File tree

codecov.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@ coverage:
1010
# solid green color.
1111
range: "20...60"
1212

13-
# See https://docs.codecov.com/docs/commit-status
13+
# See https://docs.codecov.com/docs/commit-status
1414
status:
1515
# project will give us the diff in the total code coverage between a commit
1616
# and its parent
1717
project:
18-
default:
18+
default:
1919
# Allow the coverage to drop by 1% and posting a success status.
2020
threshold: 1%
2121
# Patch gives just the coverage of the patch

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ require (
2626
)
2727

2828
require (
29-
github.com/codeready-toolchain/api v0.0.0-20260226033043-912fcbd23dc7
29+
github.com/codeready-toolchain/api v0.0.0-20260305144020-4ff0e6b6e174
3030
github.com/ghodss/yaml v1.0.0
3131
github.com/google/go-cmp v0.7.0
3232
github.com/google/go-github/v52 v52.0.0

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ github.com/cespare/xxhash/v2 v2.3.0/go.mod h1:VGX0DQ3Q6kWi7AoAeZDth3/j3BFtOZR5XL
2020
github.com/cloudflare/circl v1.1.0/go.mod h1:prBCrKB9DV4poKZY1l9zBXg2QJY7mvgRvtMxxK7fi4I=
2121
github.com/cloudflare/circl v1.6.3 h1:9GPOhQGF9MCYUeXyMYlqTR6a5gTrgR/fBLXvUgtVcg8=
2222
github.com/cloudflare/circl v1.6.3/go.mod h1:2eXP6Qfat4O/Yhh8BznvKnJ+uzEoTQ6jVKJRn81BiS4=
23-
github.com/codeready-toolchain/api v0.0.0-20260226033043-912fcbd23dc7 h1:1YCopXDF7WSjhM7sQAFoNeJvoIVPCHDY35jNVqc0Xh4=
24-
github.com/codeready-toolchain/api v0.0.0-20260226033043-912fcbd23dc7/go.mod h1:PMg6kNHuCGNlu3MOdrCisqGkBpvzB0qS1+E6nrXxPAc=
23+
github.com/codeready-toolchain/api v0.0.0-20260305144020-4ff0e6b6e174 h1:hed3ZyardxswS6yMB0ME9l3vEkO+pFouyk4dvIiAQOo=
24+
github.com/codeready-toolchain/api v0.0.0-20260305144020-4ff0e6b6e174/go.mod h1:PMg6kNHuCGNlu3MOdrCisqGkBpvzB0qS1+E6nrXxPAc=
2525
github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
2626
github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
2727
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=

pkg/test/config/toolchainconfig.go

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -145,24 +145,6 @@ func (o DeactivationOption) UserSignupUnverifiedRetentionDays(value int) Deactiv
145145
return o
146146
}
147147

148-
type MetricsOption struct {
149-
*ToolchainConfigOptionImpl
150-
}
151-
152-
func Metrics() *MetricsOption {
153-
o := &MetricsOption{
154-
ToolchainConfigOptionImpl: &ToolchainConfigOptionImpl{},
155-
}
156-
return o
157-
}
158-
159-
func (o MetricsOption) ForceSynchronization(value bool) MetricsOption {
160-
o.addFunction(func(config *toolchainv1alpha1.ToolchainConfig) {
161-
config.Spec.Host.Metrics.ForceSynchronization = &value
162-
})
163-
return o
164-
}
165-
166148
type NotificationsOption struct {
167149
*ToolchainConfigOptionImpl
168150
}

0 commit comments

Comments
 (0)