Skip to content

Commit

Permalink
Merge #1468: tests: notifyAll() is deprecated, replace w/ notify_all()
Browse files Browse the repository at this point in the history
5c088b2 tests: notifyAll() is deprecated, replace w/ notify_all() (pythcoiner)

Pull request description:

  closes #1253

ACKs for top commit:
  edouardparis:
    ACK 5c088b2

Tree-SHA512: 86e87277b77aa3be3159cb4337bedaf5206cac6d1610e93c9ec0c4223e00791d121bdd49fc60a08e18c0d983e6c4e8efd76046bcab78459073101ae775f9c54e
  • Loading branch information
edouardparis committed Nov 18, 2024
2 parents 1b00cdf + 5c088b2 commit d83f34c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/test_framework/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -372,7 +372,7 @@ def tail(self):
logging.debug(f"{self.prefix}: {line.decode().rstrip()}")
with self.logs_cond:
self.logs.append(str(line.rstrip()))
self.logs_cond.notifyAll()
self.logs_cond.notify_all()
self.running = False
self.proc.stdout.close()
if self.proc.stderr is not None:
Expand Down

0 comments on commit d83f34c

Please sign in to comment.