From 03dbd7dc3767d26d179f7eb35af1ef7da65a9a58 Mon Sep 17 00:00:00 2001
From: Marco Vito Moscaritolo <mavimo@gmail.com>
Date: Mon, 16 Oct 2023 14:39:49 +0200
Subject: [PATCH] Add tests for pkg/store (#147)

---
 pkg/store/store_test.go | 200 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 200 insertions(+)
 create mode 100644 pkg/store/store_test.go

diff --git a/pkg/store/store_test.go b/pkg/store/store_test.go
new file mode 100644
index 0000000..36eb05b
--- /dev/null
+++ b/pkg/store/store_test.go
@@ -0,0 +1,200 @@
+package store
+
+import (
+	"testing"
+
+	v1 "k8s.io/api/networking/v1"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	typev1 "k8s.io/apimachinery/pkg/types"
+)
+
+func TestStoreIngresses(t *testing.T) {
+	tests := []struct {
+		name            string
+		addIngresses    []string
+		removeIngresses []string
+		expectCount     int
+	}{
+		{
+			name:            "No ingress added nor removed",
+			addIngresses:    []string{},
+			removeIngresses: []string{},
+			expectCount:     0,
+		},
+		{
+			name:            "One ingress added, no ingress removed",
+			addIngresses:    []string{"first"},
+			removeIngresses: []string{},
+			expectCount:     1,
+		},
+		{
+			name:            "Two ingresses added, no ingress removed",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{},
+			expectCount:     2,
+		},
+		{
+			name:            "Two ingresses added, first ingress removed",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"first"},
+			expectCount:     1,
+		},
+		{
+			name:            "Two ingresses added, second ingress removed",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"second"},
+			expectCount:     1,
+		},
+		{
+			name:            "Two ingresses added, both ingresses removed",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"second", "first"},
+			expectCount:     0,
+		},
+		{
+			name:            "Two ingresses added, one existing and one non existing ingress removed",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"second", "third"},
+			expectCount:     1,
+		},
+		{
+			name:            "Remove non existing ingresses",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"third", "forth"},
+			expectCount:     2,
+		},
+		{
+			name:            "Remove non existing ingresses from an empty list",
+			addIngresses:    []string{},
+			removeIngresses: []string{"first", "second"},
+			expectCount:     0,
+		},
+		{
+			name:            "Add the same ingress multiple time",
+			addIngresses:    []string{"first", "first", "first"},
+			removeIngresses: []string{},
+			expectCount:     1,
+		},
+		{
+			name:            "Remove the same ingress multiple time",
+			addIngresses:    []string{"first", "second"},
+			removeIngresses: []string{"second", "second", "second"},
+			expectCount:     1,
+		},
+	}
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			s := NewStore(Options{}, &PodInfo{})
+			for _, uid := range test.addIngresses {
+				i := createIngress(uid)
+				s.AddIngress(&i)
+			}
+
+			for _, uid := range test.removeIngresses {
+				i := createIngress(uid)
+				s.PluckIngress(&i)
+			}
+
+			if test.expectCount != len(s.Ingresses) {
+				t.Errorf("Number of ingresses do not match expectation in %s: got %v, expected %v", test.name, len(s.Ingresses), test.expectCount)
+			}
+		})
+	}
+}
+
+func TestStoreReturnIfHasManagedTLS(t *testing.T) {
+	tests := []struct {
+		name      string
+		ingresses []v1.Ingress
+		expect    bool
+	}{
+		{
+			name:      "No ingress",
+			ingresses: []v1.Ingress{},
+			expect:    false,
+		},
+		{
+			name: "One ingress without certificate",
+			ingresses: []v1.Ingress{
+				createIngress("first"),
+			},
+			expect: false,
+		},
+		{
+			name: "One ingress with certificate",
+			ingresses: []v1.Ingress{
+				createIngressTLS("first", []string{"host1"}, "mysecret1"),
+			},
+			expect: true,
+		},
+		{
+			name: "Multiple ingresses without certificate",
+			ingresses: []v1.Ingress{
+				createIngress("first"),
+				createIngress("second"),
+				createIngress("third"),
+			},
+			expect: false,
+		},
+		{
+			name: "Three ingresses mixed with and without certificate",
+			ingresses: []v1.Ingress{
+				createIngressTLS("first", []string{"host1a", "host1b"}, "mysecret1"),
+				createIngress("second"),
+				createIngressTLS("third", []string{"host3"}, "mysecret3"),
+			},
+			expect: true,
+		},
+		{
+			name: "Ingress replaced without a certificate",
+			ingresses: []v1.Ingress{
+				createIngressTLS("first", []string{"host1a", "host1b"}, "mysecret1"),
+				createIngress("first"),
+			},
+			expect: false,
+		},
+		{
+			name: "Ingress replaced with a certificate",
+			ingresses: []v1.Ingress{
+				createIngress("first"),
+				createIngressTLS("first", []string{"host1a", "host1b"}, "mysecret1"),
+			},
+			expect: true,
+		},
+	}
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			s := NewStore(Options{}, &PodInfo{})
+			for _, i := range test.ingresses {
+				s.AddIngress(&i)
+			}
+
+			if test.expect != s.HasManagedTLS() {
+				t.Errorf("managed TLS do not match expectation in %s: got %v, expected %v", test.name, s.HasManagedTLS(), test.expect)
+			}
+		})
+	}
+}
+
+func createIngressTLS(uid string, hosts []string, secret string) v1.Ingress {
+	i := createIngress(uid)
+
+	i.Spec = v1.IngressSpec{
+		TLS: []v1.IngressTLS{
+			{
+				Hosts:      hosts,
+				SecretName: secret,
+			},
+		},
+	}
+
+	return i
+}
+
+func createIngress(uid string) v1.Ingress {
+	return v1.Ingress{
+		ObjectMeta: metav1.ObjectMeta{
+			UID: typev1.UID(uid),
+		},
+	}
+}