Skip to content

Commit b10eee5

Browse files
committed
fixup! Add tests that fail when a worker topology name causes invalid Kubernetes resource name
Test for max length should fail due to max length, not due to uppercase characters
1 parent c544de1 commit b10eee5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

internal/webhooks/cluster_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -1913,7 +1913,7 @@ func TestClusterTopologyValidation(t *testing.T) {
19131913
WithClass("foo").
19141914
WithVersion("v1.19.1").
19151915
WithMachineDeployment(
1916-
builder.MachineDeploymentTopology("thisNameIsReallyMuchLongerThanTheMaximumLengthOfSixtyThreeCharacters").
1916+
builder.MachineDeploymentTopology("thisnameisreallymuchlongerthanthemaximumlengthofsixtythreecharacters").
19171917
WithClass("aa").
19181918
Build()).
19191919
Build()).
@@ -1927,7 +1927,7 @@ func TestClusterTopologyValidation(t *testing.T) {
19271927
WithClass("foo").
19281928
WithVersion("v1.19.1").
19291929
WithMachinePool(
1930-
builder.MachinePoolTopology("thisNameIsReallyMuchLongerThanTheMaximumLengthOfSixtyThreeCharacters").
1930+
builder.MachinePoolTopology("thisnameisreallymuchlongerthanthemaximumlengthofsixtythreecharacters").
19311931
WithClass("aa").
19321932
Build()).
19331933
Build()).

0 commit comments

Comments
 (0)