Skip to content

Commit

Permalink
rename test cases
Browse files Browse the repository at this point in the history
  • Loading branch information
izabelacg committed Jun 5, 2024
1 parent 7dec43d commit 6c9252f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
10 changes: 5 additions & 5 deletions pkg/deployment/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ func TestControllerConfiguration(t *testing.T) {
wantConfig *Config
data map[string]string
}{{
name: "controller configuration with no affinity rule specified",
name: "controller configuration with no default affinity type specified",
wantConfig: &Config{
RegistriesSkippingTagResolving: sets.New("kind.local", "ko.local", "dev.local"),
DigestResolutionTimeout: digestResolutionTimeoutDefault,
Expand All @@ -95,21 +95,21 @@ func TestControllerConfiguration(t *testing.T) {
QueueSidecarImageKey: defaultSidecarImage,
},
}, {
name: "controller configuration with empty string for the affinity rule",
name: "controller configuration with empty string set for the default affinity type",
wantErr: true,
data: map[string]string{
QueueSidecarImageKey: defaultSidecarImage,
defaultAffinityTypeKey: "",
},
}, {
name: "controller configuration with unsupported affinity value",
name: "controller configuration with unsupported value for default affinity type",
wantErr: true,
data: map[string]string{
QueueSidecarImageKey: defaultSidecarImage,
defaultAffinityTypeKey: "coconut",
},
}, {
name: "controller configuration with the default affinity rule set",
name: "controller configuration with the default affinity type set",
wantConfig: &Config{
RegistriesSkippingTagResolving: sets.New("kind.local", "ko.local", "dev.local"),
DigestResolutionTimeout: digestResolutionTimeoutDefault,
Expand All @@ -124,7 +124,7 @@ func TestControllerConfiguration(t *testing.T) {
defaultAffinityTypeKey: string(PreferSpreadRevisionOverNodes),
},
}, {
name: "controller configuration with affinity deactivated",
name: "controller configuration with default affinity type deactivated",
wantConfig: &Config{
RegistriesSkippingTagResolving: sets.New("kind.local", "ko.local", "dev.local"),
DigestResolutionTimeout: digestResolutionTimeoutDefault,
Expand Down
6 changes: 3 additions & 3 deletions pkg/reconciler/revision/resources/deploy_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1435,7 +1435,7 @@ func TestMakePodSpec(t *testing.T) {
),
}),
}, {
name: "with default affinity rules",
name: "with default affinity type set",
rev: revision("bar", "foo",
withContainers([]corev1.Container{{
Name: servingContainerName,
Expand Down Expand Up @@ -1466,7 +1466,7 @@ func TestMakePodSpec(t *testing.T) {
},
),
}, {
name: "with affinity rules deactivated",
name: "with default affinity type deactivated",
rev: revision("bar", "foo",
withContainers([]corev1.Container{{
Name: servingContainerName,
Expand All @@ -1492,7 +1492,7 @@ func TestMakePodSpec(t *testing.T) {
},
),
}, {
name: "with affinity rules set by the user and the operator",
name: "with affinity rules set by both the user and the operator",
rev: revision("bar", "foo",
withContainers([]corev1.Container{{
Name: servingContainerName,
Expand Down

0 comments on commit 6c9252f

Please sign in to comment.