[Lldb-commits] [lldb] 8fb8601 - Revert "[lldb] Skip TestAppleSimulatorOSType if json parsing fails"

Augusto Noronha via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 16 13:50:38 PDT 2021


Author: Augusto Noronha
Date: 2021-09-16T17:49:59-03:00
New Revision: 8fb86011026c56204e54a2806d598249c6297d08

URL: https://github.com/llvm/llvm-project/commit/8fb86011026c56204e54a2806d598249c6297d08
DIFF: https://github.com/llvm/llvm-project/commit/8fb86011026c56204e54a2806d598249c6297d08.diff

LOG: Revert "[lldb] Skip TestAppleSimulatorOSType if json parsing fails"

This reverts commit 47dd1f642846d476e3d789f4aa941699dc0ed3fe.
After discussing with Jim Ingham, we agreed to leave the test as-is
so we can catch any CI problems instead of silently skipping the test.

Added: 
    

Modified: 
    lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py b/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
index 1230af95a9a33..01942344e0e10 100644
--- a/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
+++ b/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
@@ -20,16 +20,7 @@ def check_simulator_ostype(self, sdk, platform_name, arch=platform.machine()):
         cmd = ['xcrun', 'simctl', 'list', '-j', 'devices']
         self.trace(' '.join(cmd))
         sim_devices_str = subprocess.check_output(cmd).decode("utf-8")
-
-        # xcodebuild, which is invoked by the apple_simulator_test decorator,
-        # may return a successful status even if it was unable to run due to 
-        # the authorization agent denying it. Try to parse the json that lists 
-        # the simulators but if that fails skip the test.
-        try:
-            sim_devices = json.loads(sim_devices_str)['devices']
-        except json.decoder.JSONDecodeError:
-            self.skipTest("Could not parse JSON of simulators available")
-
+        sim_devices = json.loads(sim_devices_str)['devices']
         # Find an available simulator for the requested platform
         deviceUDID = None
         deviceRuntime = None


        


More information about the lldb-commits mailing list