fix: go validator (#93)

* fix: go validator

* fix: remove pin on golang

Author: sriram-mv

.appveyor.yml

@@ -30,7 +30,7 @@ install:
 
 # setup go
 - rmdir c:\go /s /q
-- "choco install golang --version 1.11.5"
+- "choco install golang"
 - "choco install bzr"
 - "choco install dep"
 - setx PATH "C:\go\bin;C:\gopath\bin;C:\Program Files (x86)\Bazaar\;C:\Program Files\Mercurial;%PATH%;"

aws_lambda_builders/workflows/go_modules/validator.py

@@ -53,7 +53,7 @@ def validate(self, runtime_path):
             out_parts = out.decode().split()
             if len(out_parts) >= 3:
                 version_parts = [int(x) for x in out_parts[2].replace(self.LANGUAGE, "").split('.')]
-                if len(version_parts) == 3:
+                if len(version_parts) >= 2:
                     if version_parts[0] == expected_major_version and version_parts[1] >= min_expected_minor_version:
                         self._valid_runtime_path = runtime_path
                         return self._valid_runtime_path

tests/unit/workflows/go_modules/test_validator.py

@@ -40,6 +40,12 @@ def test_runtime_validate_supported_version_runtime(self):
             self.validator.validate(runtime_path="/usr/bin/go")
             self.assertTrue(mock_subprocess.call_count, 1)
 
+    def test_runtime_validate_supported_higher_than_min_version_runtime(self):
+        with mock.patch("subprocess.Popen") as mock_subprocess:
+            mock_subprocess.return_value = MockSubProcess(0, out=b"go version go1.12 test")
+            self.validator.validate(runtime_path="/usr/bin/go")
+            self.assertTrue(mock_subprocess.call_count, 1)
+
     def test_runtime_validate_mismatch_nonzero_exit(self):
         with mock.patch("subprocess.Popen") as mock_subprocess:
             mock_subprocess.return_value = MockSubProcess(1)