Skip to content

Commit 1443ad1

Browse files
committed
Do not skip if exited properly
1 parent fcc90c8 commit 1443ad1

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

mmengine/testing/_internal/distributed.py

+7-6
Original file line numberDiff line numberDiff line change
@@ -351,12 +351,13 @@ def _check_return_codes(self, elapsed_time) -> None:
351351
if first_process.exitcode == skip.exit_code:
352352
raise unittest.SkipTest(skip.message)
353353

354-
# Skip the unittest since the raised error maybe not caused by
355-
# the tested function. For example, in CI environment, the tested
356-
# method could be terminated by system signal for the limited
357-
# resources.
358-
self.skipTest(f'Skip test {self._testMethodName} due to '
359-
'the program abort')
354+
if first_process.exitcode != 0:
355+
# Skip the unittest since the raised error maybe not caused by
356+
# the tested function. For example, in CI environment, the tested
357+
# method could be terminated by system signal for the limited
358+
# resources.
359+
self.skipTest(f'Skip test {self._testMethodName} due to '
360+
'the program abort')
360361

361362
@property
362363
def is_master(self) -> bool:

0 commit comments

Comments
 (0)