feat: create django version of the repository and bots services

[joseph-sentry]

This PR creates copies of the repository and bots services that use the Django models instead of the SQLAlchemy models. This is so that the test results finisher can use the django models instead of the SQLAlchemy models.

Note: there is a difference in the behaviour we're verifying between the regular repository service tests and the django repository service tests. One example is test_fetch_and_update_pull_request_information_no_compared_to

[codecov-qa]

Codecov Report

Attention: Patch coverage is 98.65841% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 97.30%. Comparing base (5285a97) to head (120cf3c).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
+ Coverage   97.26%   97.30%   +0.03%     
==========================================
  Files         411      421      +10     
  Lines       34146    35121     +975     
==========================================
+ Hits        33211    34173     +962     
- Misses        935      948      +13     

Flag	Coverage Δ
integration	97.30% <98.65%> (+0.03%)	⬆️
latest-uploader-overall	97.30% <98.65%> (+0.03%)	⬆️
unit	97.30% <98.65%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.43% <96.81%> (+0.06%)	⬆️
OutsideTasks	97.56% <98.65%> (+0.04%)	⬆️

Files	Coverage Δ
helpers/token_refresh_django.py	100.00% <100.00%> (ø)
services/bots_django/__init__.py	100.00% <100.00%> (ø)
services/bots_django/owner_bots.py	100.00% <100.00%> (ø)
services/bots_django/tests/test_bots.py	100.00% <100.00%> (ø)
services/bots_django/types.py	100.00% <100.00%> (ø)
services/bots_django/public_bots.py	91.30% <91.30%> (ø)
services/repository_django.py	98.33% <98.33%> (ø)
services/tests/test_repository_django.py	99.58% <99.58%> (ø)
services/bots_django/repo_bots.py	93.33% <93.33%> (ø)
services/bots_django/github_apps.py	94.59% <94.59%> (ø)

... and 1 file with indirect coverage changes

[codecov-notifications]

Codecov Report

Attention: Patch coverage is 98.65841% with 13 lines in your changes are missing coverage. Please review.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
+ Coverage   97.26%   97.30%   +0.03%     
==========================================
  Files         411      421      +10     
  Lines       34146    35121     +975     
==========================================
+ Hits        33211    34173     +962     
- Misses        935      948      +13     

Flag	Coverage Δ
integration	97.30% <98.65%> (+0.03%)	⬆️
latest-uploader-overall	97.30% <98.65%> (+0.03%)	⬆️
unit	97.30% <98.65%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.43% <96.81%> (+0.06%)	⬆️
OutsideTasks	97.56% <98.65%> (+0.04%)	⬆️

Files	Coverage Δ
helpers/token_refresh_django.py	100.00% <100.00%> (ø)
services/bots_django/__init__.py	100.00% <100.00%> (ø)
services/bots_django/owner_bots.py	100.00% <100.00%> (ø)
services/bots_django/tests/test_bots.py	100.00% <100.00%> (ø)
services/bots_django/types.py	100.00% <100.00%> (ø)
services/bots_django/public_bots.py	91.30% <91.30%> (ø)
services/repository_django.py	98.33% <98.33%> (ø)
services/tests/test_repository_django.py	99.58% <99.58%> (ø)
services/bots_django/repo_bots.py	93.33% <93.33%> (ø)
services/bots_django/github_apps.py	94.59% <94.59%> (ø)

... and 1 file with indirect coverage changes

[codecov-public-qa]

Codecov Report

Attention: Patch coverage is 98.65841% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 97.30%. Comparing base (5285a97) to head (120cf3c).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found ☺️

@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
+ Coverage   97.26%   97.30%   +0.03%     
==========================================
  Files         411      421      +10     
  Lines       34146    35121     +975     
==========================================
+ Hits        33211    34173     +962     
- Misses        935      948      +13     

Flag	Coverage Δ
integration	97.30% <98.65%> (+0.03%)	⬆️
latest-uploader-overall	97.30% <98.65%> (+0.03%)	⬆️
unit	97.30% <98.65%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.43% <96.81%> (+0.06%)	⬆️
OutsideTasks	97.56% <98.65%> (+0.04%)	⬆️

Files	Coverage Δ
helpers/token_refresh_django.py	100.00% <100.00%> (ø)
services/bots_django/__init__.py	100.00% <100.00%> (ø)
services/bots_django/owner_bots.py	100.00% <100.00%> (ø)
services/bots_django/tests/test_bots.py	100.00% <100.00%> (ø)
services/bots_django/types.py	100.00% <100.00%> (ø)
services/bots_django/public_bots.py	91.30% <91.30%> (ø)
services/repository_django.py	98.33% <98.33%> (ø)
services/tests/test_repository_django.py	99.58% <99.58%> (ø)
services/bots_django/repo_bots.py	93.33% <93.33%> (ø)
services/bots_django/github_apps.py	94.59% <94.59%> (ø)

... and 1 file with indirect coverage changes

[codecov]

Codecov Report

Attention: Patch coverage is 98.65841% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 97.33%. Comparing base (5285a97) to head (120cf3c).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
+ Coverage   97.29%   97.33%   +0.03%     
==========================================
  Files         442      452      +10     
  Lines       34875    35866     +991     
==========================================
+ Hits        33931    34909     +978     
- Misses        944      957      +13     

Flag	Coverage Δ
integration	97.30% <98.65%> (+0.03%)	⬆️
latest-uploader-overall	97.30% <98.65%> (+0.03%)	⬆️
unit	97.30% <98.65%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.49% <96.81%> (+0.06%)	⬆️
OutsideTasks	97.56% <98.65%> (+0.04%)	⬆️

Files	Coverage Δ
helpers/token_refresh_django.py	100.00% <100.00%> (ø)
services/bots_django/__init__.py	100.00% <100.00%> (ø)
services/bots_django/owner_bots.py	100.00% <100.00%> (ø)
services/bots_django/tests/test_bots.py	100.00% <100.00%> (ø)
services/bots_django/types.py	100.00% <100.00%> (ø)
services/bots_django/public_bots.py	91.30% <91.30%> (ø)
services/repository_django.py	98.33% <98.33%> (ø)
services/tests/test_repository_django.py	99.58% <99.58%> (ø)
services/bots_django/repo_bots.py	93.33% <93.33%> (ø)
services/bots_django/github_apps.py	94.59% <94.59%> (ø)

... and 1 file with indirect coverage changes

This change has been scanned for critical changes. Learn more

[giovanni-guidini]

Can we instead change the bots services to be database agnostic?

It doesn't seem great to have 2 different copies of such important services - that would have to be maintained both, and is being actively worked on - in the same project. Especially without a clear deprecation date for the SQLAlchemy models.

I believe there's a way to make this service database agnostic so you can pass either model and have it still work (with helper functions for the models methods and fields, mostly) that would be better for us long-term.

[giovanni-guidini]

Can we instead change the bots services to be database agnostic?

It doesn't seem great to have 2 different copies of such important services - that would have to be maintained both, and is being actively worked on - in the same project. Especially without a clear deprecation date for the SQLAlchemy models.

I believe there's a way to make this service database agnostic so you can pass either model and have it still work (with helper functions for the models methods and fields, mostly) that would be better for us long-term.