chore: Use pytest-mock in tests

[RisingOrange]

Replace usages of monkeypatch (+ unittest.mock) or our custom mock_function fixture with pytest-mock. This simplifies the test code.

Related issues

Proposed changes

• Add pytest-mock to requirements
• Replace usages of monkeypatch with mocker.patch
• Replace usages of our mock_function fixture with mocker.patch
• Replace usages of unittest.Mock() with mocker.stub() or mocker.Mock()

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ad16933) 85.12% compared to head (472b3e0) 85.06%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   85.12%   85.06%   -0.07%     
==========================================
  Files          65       65              
  Lines        5931     5931              
==========================================
- Hits         5049     5045       -4     
- Misses        882      886       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[treyhunner]

This is looking great.

All the repetition of various mocker.patch.object(AnkiHubClient, "some_method") usages make me think a custom fixture to assist with patching the client in various ways might be useful after this first wave of monkey patch refactoring.

[RisingOrange]

@treyhunner

All the repetition of various mocker.patch.object(AnkiHubClient, "some_method") usages make me think a custom fixture to assist with patching the client in various ways might be useful after this first wave of monkey patch refactoring.

Sounds good. I'm curious how this would look like.

[treyhunner]

This looks great! I think #849 should be merged into this first and then I think this is ready to go. It looks like there's an minor import difference that needs resolving before merging to main.