fix: fix flake8 errors and add flake8 run in buildspec.yml (#200)

Author: icywang86rui

buildspec.yml

@@ -29,8 +29,11 @@ phases:
       - pip3 install -U -e .
       - pip3 install -U -e .[test]
 
+      # run flake8
+      - tox -e flake8
+
       # run unit tests
-      - pytest test/unit
+      - tox -e py36,py27 test/unit
 
       # Create pip archive
       - build_dir="docker/$FRAMEWORK_VERSION"

test/conftest.py

@@ -107,8 +107,8 @@ def skip_by_device_type(request, processor):
 
 @pytest.fixture(autouse=True)
 def skip_gpu_instance_restricted_regions(region, instance_type):
-    if((region in NO_P2_REGIONS and instance_type.startswith('ml.p2')) or
-       (region in NO_P3_REGIONS and instance_type.startswith('ml.p3'))):
+    if (region in NO_P2_REGIONS and instance_type.startswith('ml.p2')) or \
+            (region in NO_P3_REGIONS and instance_type.startswith('ml.p3')):
         pytest.skip('Skipping GPU test in region {}'.format(region))