Skip to content

Conversation

@eviljeff
Copy link
Member

@eviljeff eviljeff commented Oct 14, 2025

Fixes: mozilla/addons#15892

Description

Replaces freezegun with time-machine.

Context

time machine defaults to tick=True, which is probably perfectly fine - and preferable - in most cases, but for ease of migration I added tick=False everywhere. (Though the one test class I thought it'd be fine in - TestReviewReports - was broken without it).

Testing

N/A - it only affects tests

Checklist

  • Add #ISSUENUM at the top of your PR to an existing open issue in the mozilla/addons repository.
  • Successfully verified the change locally.
  • The change is covered by automated tests, or otherwise indicated why doing so is unnecessary/impossible.
  • Add before and after screenshots (Only for changes that impact the UI).
  • Add or update relevant docs reflecting the changes made.

@eviljeff eviljeff force-pushed the 15892-replace-freezegun-with-timemachine branch 4 times, most recently from cf80bc4 to 24beb46 Compare October 23, 2025 12:18
@eviljeff eviljeff force-pushed the 15892-replace-freezegun-with-timemachine branch from 24beb46 to 6d597d3 Compare October 23, 2025 12:29
@eviljeff eviljeff marked this pull request as ready for review October 23, 2025 12:43
@eviljeff eviljeff requested a review from willdurand October 23, 2025 12:43
@eviljeff eviljeff merged commit a3b191c into mozilla:master Oct 23, 2025
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task]: Replace freezegun with time-machine

2 participants