Test for log_heartbeat fails in timezones with non-integer offsets
The test for log_heartbeat contains the check
```
/* format_local_iso_time uses local tz, just check mins and secs. */
tt_ptr_op(strstr(va_arg(ap, char *), ":01:00"),
OP_NE, NULL); /* end_buf */
```
which assumes that the timezone will only affect the hours. In timezones with half-hour or 15 minute offsets this test will always (incorrectly) fail.
I have a fix at https://github.com/fergus-dall/tor/tree/log_heartbeat_test which calls format_local_iso_time to figure out what to compare against.
**Trac**:
**Username**: fergus
issue