No, that test is sufficient I think.  I just meant that in `test_acquire_logs`, the first the the lock is tried and failed, the clock is at 0, so all time would be zero.  

Which means that nothing would be logged even if there were a mistake and it was checking for `self._acquire_start_seconds` instead of `self._lock_acquired_seconds`.  (because the reactor.seconds is 0).

So if the clock started at time 1, for example, then reactor.seconds would never be 0, and it seems like the test would be very slightly more accurate in testing?