mirror of
https://github.com/jesseduffield/lazygit.git
synced 2026-09-10 15:46:26 -04:00
5761f92e16
The "custom time format" test case of TestGetCommitListDisplayStrings failed on
Windows. Root cause: UnixToDateSmart used time.Unix(timestamp, 0), which formats
in process-local time. In tests we pass an explicit 'now' (often UTC), but the
timestamp was rendered in Local, causing one-hour drift on non-UTC machines.
The test tried to work around this by doing os.Setenv("TZ", "UTC"); however,
this only worked on Mac and Linux. On Windows, it has no effect because Windows
uses registry-based timezone configuration, not TZ environment variables.
Change: convert the timestamp into now.Location() before both the same-day
comparison and formatting.
Why this is safe: callers already define the presentation timezone via the 'now'
argument (typically time.Now() in app code, controlled time in tests). This
aligns timestamp rendering with that caller intent and removes dependence on
global TZ state or OS-specific environment variable behavior.
Added regression tests for UTC and UTC+1 to ensure deterministic behavior across
all platforms.
|
||
|---|---|---|
| .. | ||
| yaml_utils | ||
| color.go | ||
| color_test.go | ||
| date.go | ||
| date_test.go | ||
| dummies.go | ||
| errors.go | ||
| formatting.go | ||
| formatting_test.go | ||
| history_buffer.go | ||
| history_buffer_test.go | ||
| io.go | ||
| io_test.go | ||
| lines.go | ||
| lines_test.go | ||
| once_writer.go | ||
| once_writer_test.go | ||
| rebase_todo.go | ||
| rebase_todo_test.go | ||
| regexp.go | ||
| regexp_test.go | ||
| search.go | ||
| search_test.go | ||
| slice.go | ||
| slice_test.go | ||
| string_pool.go | ||
| string_stack.go | ||
| template.go | ||
| template_test.go | ||
| thread_safe_map.go | ||
| thread_safe_map_test.go | ||
| utils.go | ||
| utils_test.go | ||