Transaction API unit test DoesNotWaitForBestEffortAbortIfCancelled accesses an unsynchronized member variable in the test's mocked transaction API client, which triggered a TSAN failure. The access happens when the transaction API using the mocked client should be blocked on a condition variable, so there shouldn't be a problem in practice, but we should add synchronization to the mock class to silence TSAN and follow best concurrency practices.