Skip to content

Commit

Permalink
fix this test for spawn() vs fork()
Browse files Browse the repository at this point in the history
  • Loading branch information
Jacob Beck committed Jun 19, 2020
1 parent 8e63da2 commit db50880
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions test/integration/048_rpc_test/test_rpc.py
Original file line number Diff line number Diff line change
Expand Up @@ -768,11 +768,9 @@ def test_timeout_postgres(self):
self.assertIn('message', error_data)
self.assertEqual(error_data['message'], 'RPC timed out after 1.0s')
self.assertIn('logs', error_data)
if sys.platform == 'darwin':
# because fork() without exec() is broken on macos, we use 'spawn'
# so on macos we don't get logs back because we didn't fork()
return
self.assertTrue(len(error_data['logs']) > 0)
# because fork() without exec() is broken, we use 'spawn' so we don't
# get logs back because we didn't fork()
return


@mark.flaky(rerun_filter=addr_in_use)
Expand Down

0 comments on commit db50880

Please sign in to comment.