From 475e45b8484ac7d0d1617f8f88200402fae7db85 Mon Sep 17 00:00:00 2001
From: Bezalel Brandwine <bbrandwine@armosec.io>
Date: Thu, 14 Oct 2021 12:07:51 +0300
Subject: [PATCH] add CLI prints checks skeleton

---
 .github/workflows/build_dev.yaml | 5 +----
 build.py                         | 8 ++++++++
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/build_dev.yaml b/.github/workflows/build_dev.yaml
index c3b270da..7638ac5e 100644
--- a/.github/workflows/build_dev.yaml
+++ b/.github/workflows/build_dev.yaml
@@ -31,10 +31,7 @@ jobs:
           ArmoERServer: report.euprod1.cyberarmorsoft.com
           ArmoWebsite: portal.armo.cloud
           CGO_ENABLED: 0
-        run: mkdir -p build/${{ matrix.os }}  && go mod tidy && go build -ldflags "-w -s -X github.com/armosec/kubescape/cmd.BuildNumber=$RELEASE -X github.com/armosec/kubescape/cautils/getter.ArmoBEURL=$ArmoBEServer -X github.com/armosec/kubescape/cautils/getter.ArmoERURL=$ArmoERServer -X github.com/armosec/kubescape/cautils/getter.ArmoFEURL=$ArmoWebsite" -o build/${{ matrix.os }}/kubescape # && md5sum build/${{ matrix.os }}/kubescape > build/${{ matrix.os }}/kubescape.md5
-      
-      - name: TestHelpCMD
-        run: ./build/${{ matrix.os }}/kubescape -h | grep "download"
+        run: python build.py
       
       - name: Upload build artifacts 
         uses: actions/upload-artifact@v2
diff --git a/build.py b/build.py
index 2687e0ef..a8cbe021 100644
--- a/build.py
+++ b/build.py
@@ -59,6 +59,8 @@ def main():
            ER_SERVER_CONST, ArmoERServer, WEBSITE_CONST, ArmoWebsite)
     status = subprocess.call(["go", "build", "-o", "%s/%s" % (buildDir, packageName), "-ldflags" ,ldflags])
     checkStatus(status, "Failed to build kubescape")
+    
+    test_cli_prints(buildDir,packageName)
 
 
     sha1 = hashlib.sha1()
@@ -69,5 +71,11 @@ def main():
 
     print("Build Done")
 
+def test_cli_prints(buildDir,packageName):
+    print("testing CLI prints")
+    bin_cli = os.path.join(buildDir,packageName)
+    status = str(subprocess.check_output([bin_cli, "-h"]))
+    assert "download" in status, "download is missing: " + status    
+
 if __name__ == "__main__":
     main()