Skip to content

Conversation

@jezdez
Copy link
Member

@jezdez jezdez commented Dec 16, 2025

  • Replace patch context managers with mocker.patch() in all examples
  • Add mocker.patch() and monkeypatch to recommended conventions
  • Add with patch() context managers to 'Don't Do This' section
  • Update complete example to use mocker.patch() pattern

…Admin

- Add custom UUID type formatter to display UUIDs as hex strings
- Add ProjectCredentialInlineForm for inline credential editing with readonly key
- Add ProjectCredentialAdmin with readonly key field and detail view
- Auto-generate UUID key when creating new credentials
- Use form_widget_args for proper readonly styling
- Add tests for UUID formatter function
- Add tests for ProjectCredentialInlineForm configuration and behavior
- Add tests for ProjectCredentialAdmin configuration and key generation
- Add tests for JazzbandModelView access control
- Add tests for JazzbandAdminIndexView authentication
- Add tests for init_app function
- Achieve 100% test coverage for admin.py
- Replace patch context managers with mocker.patch() in all examples
- Add mocker.patch() and monkeypatch to recommended conventions
- Add with patch() context managers to 'Don't Do This' section
- Update complete example to use mocker.patch() pattern
@jezdez jezdez enabled auto-merge December 16, 2025 09:03
@jezdez jezdez added this pull request to the merge queue Dec 16, 2025
@codecov
Copy link

codecov bot commented Dec 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.75%. Comparing base (25eb362) to head (f3a363b).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1245      +/-   ##
==========================================
+ Coverage   64.05%   64.75%   +0.69%     
==========================================
  Files          40       40              
  Lines        2462     2491      +29     
==========================================
+ Hits         1577     1613      +36     
+ Misses        885      878       -7     
Files with missing lines Coverage Δ
jazzband/admin.py 100.00% <100.00%> (+14.58%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25eb362...f3a363b. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Merged via the queue into main with commit 1f15548 Dec 16, 2025
6 checks passed
@jezdez jezdez deleted the fix/admin-credential-key-display branch December 16, 2025 09:07
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.

2 participants