Merge pull request #279 from bricerisingalgorand/feature/circleciPipe…

…line

Updates to pipeline

.circleci/config.yml

@@ -1,14 +1,24 @@
 version: 2.1
+
+workflows:
+  version: 2
+  test:
+    jobs:
+      - unit-test
+      - integration-test
+
 jobs:
-  build:
-    machine: 
-      image: "ubuntu-2004:202104-01"
+  unit-test:
+    docker:
+      - image: python:3.7.9
     steps:
       - checkout
-      - run:
-          command: |
-            pip3 install -r requirements.txt
-            black --check .
-            set -e
-            python3 test_unit.py
-            make docker-test
+      - run: pip install -r requirements.txt
+      - run: black --check .
+      - run: python3 test_unit.py
+  integration-test:
+    machine:
+     image: "ubuntu-2004:202104-01"
+    steps:
+      - checkout
+      - run: make docker-test

.gitignore

@@ -113,3 +113,9 @@ venv.bak/
 *.feature
 test/features
 test-harness
+
+# Build files
+py-algorand-sdk-*
+
+# Pycharm
+.idea

algosdk/future/transaction.py

@@ -2533,7 +2533,7 @@ def verify(self, public_key):
             try:
                 verify_key.verify(to_sign, base64.b64decode(self.sig))
                 return True
-            except BadSignatureError:
+            except (BadSignatureError, ValueError):
                 return False
 
         return self.msig.verify(to_sign)

test_unit.py

@@ -3,6 +3,7 @@
 import os
 import random
 import string
+import sys
 import unittest
 import uuid
 from unittest.mock import Mock
@@ -4317,3 +4318,5 @@ def test_contract(self):
     suite = unittest.TestSuite(suites)
     runner = unittest.TextTestRunner(verbosity=2)
     results = runner.run(suite)
+    ret = not results.wasSuccessful()
+    sys.exit(ret)