actually set exit value

Also, apparently the formatting rules changed. OK...

Author: cecille

src/python_testing/test_testing/test_TC_DA_1_2.py

@@ -19,8 +19,10 @@
 import json
 import os
 import subprocess
+import sys
 
-CHIP_ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), '../../..'))
+CHIP_ROOT = os.path.abspath(os.path.join(
+    os.path.dirname(__file__), '../../..'))
 RUNNER_SCRIPT_DIR = os.path.join(CHIP_ROOT, 'scripts/tests')
 
 
@@ -30,13 +32,15 @@ def run_single_test(dac_provider: str, product_id: int, factory_reset: bool = Fa
     if factory_reset:
         reset = ' --factoryreset'
 
-    app = os.path.join(CHIP_ROOT, 'out/linux-x64-all-clusters-ipv6only-no-ble-no-wifi-tsan-clang-test/chip-all-clusters-app')
+    app = os.path.join(
+        CHIP_ROOT, 'out/linux-x64-all-clusters-ipv6only-no-ble-no-wifi-tsan-clang-test/chip-all-clusters-app')
 
     # Certs in the commissioner_dut directory use 0x8000 as the PID
     app_args = '--discriminator 1234 --KVS kvs1 --product-id ' + \
         str(product_id) + ' --vendor-id 65521 --dac_provider ' + dac_provider
 
-    ci_pics_values = os.path.abspath(os.path.join(CHIP_ROOT, 'src/app/tests/suites/certification/ci-pics-values'))
+    ci_pics_values = os.path.abspath(os.path.join(
+        CHIP_ROOT, 'src/app/tests/suites/certification/ci-pics-values'))
     script_args = '--storage-path admin_storage.json --discriminator 1234 --passcode 20202021 --dut-node-id 1 --PICS ' + \
         str(ci_pics_values)
 
@@ -45,28 +49,34 @@ def run_single_test(dac_provider: str, product_id: int, factory_reset: bool = Fa
     if factory_reset:
         script_args = script_args + ' --commissioning-method on-network'
 
-    script = os.path.abspath(os.path.join(CHIP_ROOT, 'src/python_testing/TC_DA_1_2.py'))
+    script = os.path.abspath(os.path.join(
+        CHIP_ROOT, 'src/python_testing/TC_DA_1_2.py'))
 
     # run_python_test uses click so call as a command
-    run_python_test = os.path.abspath(os.path.join(RUNNER_SCRIPT_DIR, 'run_python_test.py'))
+    run_python_test = os.path.abspath(os.path.join(
+        RUNNER_SCRIPT_DIR, 'run_python_test.py'))
     cmd = str(run_python_test) + reset + ' --app ' + str(app) + ' --app-args "' + \
-        app_args + '" --script ' + str(script) + ' --script-args "' + script_args + '"'
+        app_args + '" --script ' + \
+        str(script) + ' --script-args "' + script_args + '"'
 
     return subprocess.call(cmd, shell=True)
 
 
 def main():
-    cert_path = os.path.abspath(os.path.join(CHIP_ROOT, 'credentials/development/commissioner_dut'))
+    cert_path = os.path.abspath(os.path.join(
+        CHIP_ROOT, 'credentials/development/commissioner_dut'))
 
     # Commission first using a known good set, then run the rest of the tests without recommissioning
-    path = str(os.path.join(cert_path, "struct_cd_authorized_paa_list_count1_valid/test_case_vector.json"))
+    path = str(os.path.join(
+        cert_path, "struct_cd_authorized_paa_list_count1_valid/test_case_vector.json"))
     run_single_test(path, 32768, factory_reset=True)
 
     test_cases = {'struct_cd': 32768, 'fallback_encoding': 177}
 
     # struct_cd_version_number_wrong - excluded because this is a DCL test not covered by cert
     # struct_cd_cert_id_mismatch - excluded because this is a DCL test not covered by cert
-    exclude_cases = ['struct_cd_version_number_wrong', 'struct_cd_cert_id_mismatch']
+    exclude_cases = ['struct_cd_version_number_wrong',
+                     'struct_cd_cert_id_mismatch']
 
     passes = []
     for p in os.listdir(cert_path):
@@ -92,7 +102,7 @@ def main():
             print('INCORRECT: ' + p[0])
             retval = 1
 
-    return retval
+    sys.exit(retval)
 
 
 if __name__ == '__main__':