Skip to content

Commit 65f123c

Browse files
committed
Extend e2e tests
1 parent 206cda8 commit 65f123c

File tree

1 file changed

+34
-8
lines changed

1 file changed

+34
-8
lines changed

cmd/e2e/basic_test.go

+34-8
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ var (
2828

2929
type TestStacksetSpecFactory struct {
3030
stacksetName string
31+
configMap bool
3132
hpaBehavior bool
3233
ingress bool
3334
routegroup bool
@@ -48,6 +49,7 @@ type TestStacksetSpecFactory struct {
4849
func NewTestStacksetSpecFactory(stacksetName string) *TestStacksetSpecFactory {
4950
return &TestStacksetSpecFactory{
5051
stacksetName: stacksetName,
52+
configMap: false,
5153
ingress: false,
5254
externalIngress: false,
5355
limit: 4,
@@ -59,6 +61,11 @@ func NewTestStacksetSpecFactory(stacksetName string) *TestStacksetSpecFactory {
5961
}
6062
}
6163

64+
func (f *TestStacksetSpecFactory) ConfigMap() *TestStacksetSpecFactory {
65+
f.configMap = true
66+
return f
67+
}
68+
6269
func (f *TestStacksetSpecFactory) Behavior(stabilizationWindowSeconds int32) *TestStacksetSpecFactory {
6370
f.hpaBehavior = true
6471
f.hpaStabilizationWindowSeconds = stabilizationWindowSeconds
@@ -347,11 +354,13 @@ func verifyStack(t *testing.T, stacksetName, currentVersion string, stacksetSpec
347354
require.EqualValues(t, stackRGHosts, stackRG.Spec.Hosts)
348355
}
349356

350-
// Verify the ConfigMaps
357+
// Verify ConfigMaps
351358
if stacksetSpec.StackTemplate.Spec.ConfigurationResources != nil {
352359
configMap, err := waitForConfigMap(t, stack.Name)
353360
require.NoError(t, err)
354361
require.EqualValues(t, stackResourceLabels, configMap.Labels)
362+
require.Contains(t, configMap.Name, stack.Name)
363+
require.NotEmpty(t, configMap.Data)
355364
}
356365
}
357366

@@ -496,12 +505,25 @@ func verifyStacksetRouteGroup(t *testing.T, stacksetName string, stacksetSpec zv
496505
require.NoError(t, err)
497506
}
498507

499-
func testStacksetCreate(t *testing.T, testName string, hpa, ingress, routegroup, externalIngress bool, updateStrategy bool, subResourceAnnotations map[string]string) {
508+
func testStacksetCreate(
509+
t *testing.T,
510+
testName string,
511+
configmap bool,
512+
hpa,
513+
ingress,
514+
routegroup,
515+
externalIngress bool,
516+
updateStrategy bool,
517+
subResourceAnnotations map[string]string,
518+
) {
500519
t.Parallel()
501520

502521
stacksetName := fmt.Sprintf("stackset-create-%s", testName)
503522
stackVersion := "v1"
504523
stacksetSpecFactory := NewTestStacksetSpecFactory(stacksetName)
524+
if configmap {
525+
stacksetSpecFactory.ConfigMap()
526+
}
505527
if hpa {
506528
stacksetSpecFactory.Autoscaler(1, 3, []zv1.AutoscalerMetrics{makeCPUAutoscalerMetrics(50)})
507529
}
@@ -696,27 +718,31 @@ func testStacksetUpdate(
696718
}
697719

698720
func TestStacksetCreateBasic(t *testing.T) {
699-
testStacksetCreate(t, "basic", false, false, false, false, false, testAnnotationsCreate)
721+
testStacksetCreate(t, "basic", false, false, false, false, false, false, testAnnotationsCreate)
722+
}
723+
724+
func TestStacksetCreateConfigMap(t *testing.T) {
725+
testStacksetCreate(t, "configmap", true, false, false, false, false, false, testAnnotationsCreate)
700726
}
701727

702728
func TestStacksetCreateHPA(t *testing.T) {
703-
testStacksetCreate(t, "hpa", true, false, false, false, false, testAnnotationsCreate)
729+
testStacksetCreate(t, "hpa", false, true, false, false, false, false, testAnnotationsCreate)
704730
}
705731

706732
func TestStacksetCreateIngress(t *testing.T) {
707-
testStacksetCreate(t, "ingress", false, true, false, false, false, testAnnotationsCreate)
733+
testStacksetCreate(t, "ingress", false, false, true, false, false, false, testAnnotationsCreate)
708734
}
709735

710736
func TestStacksetCreateRouteGroup(t *testing.T) {
711-
testStacksetCreate(t, "routegroup", false, false, true, false, false, testAnnotationsCreate)
737+
testStacksetCreate(t, "routegroup", false, false, false, true, false, false, testAnnotationsCreate)
712738
}
713739

714740
func TestStacksetCreateExternalIngress(t *testing.T) {
715-
testStacksetCreate(t, "externalingress", false, false, false, true, false, testAnnotationsCreate)
741+
testStacksetCreate(t, "externalingress", false, false, false, false, true, false, testAnnotationsCreate)
716742
}
717743

718744
func TestStacksetCreateUpdateStrategy(t *testing.T) {
719-
testStacksetCreate(t, "updatestrategy", false, false, false, false, true, testAnnotationsCreate)
745+
testStacksetCreate(t, "updatestrategy", false, false, false, false, false, true, testAnnotationsCreate)
720746
}
721747

722748
func TestStacksetUpdateBasic(t *testing.T) {

0 commit comments

Comments
 (0)