change if stmt logic to be easier to read

Author: sleepyStick

test/asynchronous/unified_format.py

@@ -1395,10 +1395,8 @@ async def run_scenario(self, spec, uri=None):
                 try:
                     return await self._run_scenario(spec, uri)
                 except (AssertionError, OperationFailure) as exc:
-                    if not (
-                        isinstance(exc, OperationFailure)
-                        and not _IS_SYNC
-                        and "failpoint" in exc._message
+                    if isinstance(exc, OperationFailure) and (
+                        _IS_SYNC or "failpoint" not in exc._message
                     ):
                         raise
                     if i < attempts - 1:

test/unified_format.py

@@ -1382,10 +1382,8 @@ def run_scenario(self, spec, uri=None):
                 try:
                     return self._run_scenario(spec, uri)
                 except (AssertionError, OperationFailure) as exc:
-                    if not (
-                        isinstance(exc, OperationFailure)
-                        and not _IS_SYNC
-                        and "failpoint" in exc._message
+                    if isinstance(exc, OperationFailure) and (
+                        _IS_SYNC or "failpoint" not in exc._message
                     ):
                         raise
                     if i < attempts - 1: