fix: replace time.sleep with GIT_AUTHOR_DATE in test #165

Merged
fuzzy merged 1 commit from fix/130-time-sleep-test into main 2026-07-06 08:38:15 +00:00
Owner

Replaces time.sleep(1.1) calls in test_returns_newest_first with explicit date_str parameter using GIT_AUTHOR_DATE. Speeds up test execution by ~2.2s and removes the time import.

Closes #130

Replaces `time.sleep(1.1)` calls in `test_returns_newest_first` with explicit `date_str` parameter using `GIT_AUTHOR_DATE`. Speeds up test execution by ~2.2s and removes the `time` import. Closes #130
fix(tests): replace time.sleep with GIT_AUTHOR_DATE in test
All checks were successful
CI / lint-and-test (pull_request) Successful in 1m33s
92ab190596
Removes time.sleep(1.1) calls and uses explicit date_str
parameter instead. Speeds up test by ~2.2s.

Closes #130
the.auditor left a comment

Audit Review — PR #165

Clean fix for #130. time.sleep(1.1) replaced with explicit date_str dates via GIT_AUTHOR_DATE. Test now deterministic and ~2.2s faster.

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #165 Clean fix for #130. `time.sleep(1.1)` replaced with explicit `date_str` dates via `GIT_AUTHOR_DATE`. Test now deterministic and ~2.2s faster. No suggestions. ## Summary **Result:** COMMENT — no issues found.
the.auditor left a comment

APPROVED. No more time.sleep in tests, closes #130. Merge when ready.

APPROVED. No more time.sleep in tests, closes #130. Merge when ready.
fuzzy force-pushed fix/130-time-sleep-test from 92ab190596
All checks were successful
CI / lint-and-test (pull_request) Successful in 1m33s
to bdb5381da1
All checks were successful
CI / lint-and-test (pull_request) Successful in 1m31s
2026-07-06 08:36:03 +00:00
Compare
fuzzy scheduled this pull request to auto merge when all checks succeed 2026-07-06 08:36:13 +00:00
fuzzy merged commit bdb5381da1 into main 2026-07-06 08:38:15 +00:00
fuzzy deleted branch fix/130-time-sleep-test 2026-07-06 08:38:15 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
thwap/hottea!165
No description provided.