From c0ef7a69f5e937b600d6038a2e62066b7f2d5cb5 Mon Sep 17 00:00:00 2001
From: Tarun Chinmai Sekar <schinmai@akamai.com>
Date: Thu, 31 Aug 2023 12:32:10 -0700
Subject: [PATCH] Add a unit-test

---
 .github/workflows/test.yml |  4 ++++
 Makefile                   | 32 ++++++++++++++++++++++++++++++--
 2 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 47df478e..fe60f0a4 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -16,3 +16,7 @@ jobs:
         run: make docker-build
       - name: unit test
         run: make test
+      - name: helm lint
+        run: make helm-lint
+      - name: helm template
+        run: make helm-template
\ No newline at end of file
diff --git a/Makefile b/Makefile
index 5bf19fff..126a0a2d 100644
--- a/Makefile
+++ b/Makefile
@@ -74,5 +74,33 @@ run-debug: build
 		--stderrthreshold=INFO \
 		--kubeconfig=${KUBECONFIG} \
 		--linodego-debug
-
-
+# Set the host's OS. Only linux and darwin supported for now
+HOSTOS := $(shell uname -s | tr '[:upper:]' '[:lower:]')
+ifeq ($(filter darwin linux,$(HOSTOS)),)
+$(error build only supported on linux and darwin host currently)
+endif
+
+HELM_VERSION ?= v3.9.1
+TOOLS_HOST_DIR ?= .tmp/tools
+HELM := $(TOOLS_HOST_DIR)/helm-$(HELM_VERSION)
+
+.PHONY: $(HELM)
+$(HELM):
+	@echo installing helm $(HELM_VERSION)
+	@mkdir -p $(TOOLS_HOST_DIR)/tmp-helm
+	@curl -fsSL https://get.helm.sh/helm-$(HELM_VERSION)-$(HOSTOS)-amd64.tar.gz | tar -xz -C $(TOOLS_HOST_DIR)/tmp-helm
+	@mv $(TOOLS_HOST_DIR)/tmp-helm/$(HOSTOS)-amd64/helm $(HELM)
+	@rm -fr $(TOOLS_HOST_DIR)/tmp-helm
+	@echo installing helm $(HELM_VERSION)
+
+.PHONY: helm-lint
+helm-lint: $(HELM)
+#Verify lint works when region and apiToken are passed, and when it is passed as reference.
+	@$(HELM) lint deploy/chart --set apiToken="apiToken",region="us-east"
+	@$(HELM) lint deploy/chart --set secretRef.apiTokenRef="apiToken",secretRef.name="api",secretRef.regionRef="us-east"
+
+.PHONY: helm-template
+helm-template: $(HELM)
+#Verify template works when region and apiToken are passed, and when it is passed as reference.
+	@$(HELM) template foo deploy/chart --set apiToken="apiToken",region="us-east" > /dev/null
+	@$(HELM) template foo deploy/chart --set secretRef.apiTokenRef="apiToken",secretRef.name="api",secretRef.regionRef="us-east" > /dev/null