Skip to content

[DOP-39512] Remove fastapi dependeciy_override and Stub - #409

Merged
dolfinus merged 2 commits into
developfrom
feature/DOP-39512
Aug 6, 2026
Merged

[DOP-39512] Remove fastapi dependeciy_override and Stub#409
dolfinus merged 2 commits into
developfrom
feature/DOP-39512

Conversation

@TiGrib

@TiGrib TiGrib commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Change Summary

Remove fastapi dependeciy_override and Stub to speed up application

Related issue number

[DOP-39512]

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@TiGrib
TiGrib requested a review from dolfinus August 6, 2026 09:35
@TiGrib TiGrib self-assigned this Aug 6, 2026
Comment thread syncmaster/server/services/auth.py Outdated
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
syncmaster/db
   factory.py1650069%20, 36–38, 40
syncmaster/server
   __init__.py3530091%67–69
syncmaster/server/api/v1
   auth.py3020093%32, 41
   runs.py64612289%100->101, 101, 132–134, 138, 159->160, 160
syncmaster/server/dependencies
   __init__.py22000%4, 6
syncmaster/server/providers/auth
   dummy_provider.py61256457%54–56, 58–63, 65–68, 70–72, 79–80, 84, 89, 99–101, 111, 115
   keycloak_provider.py861710379%61, 78–80, 111–113, 115->116, 116–117, 123->124, 124–125, 153->154, 154–155, 159–162
   oauth2_gateway_provider.py55146274%46->47, 47–49, 53–56, 66->67, 67–68, 78–79, 98, 111, 117
syncmaster/server/services
   get_user.py32312289%47->49, 49, 59->60, 60–61
TOTAL535364082617187% 

@dolfinus
dolfinus merged commit 47c0d84 into develop Aug 6, 2026
24 of 25 checks passed
@dolfinus
dolfinus deleted the feature/DOP-39512 branch August 6, 2026 10:02
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