-
Notifications
You must be signed in to change notification settings - Fork 0
refactor: added settings-driven DI and CSRF middleware with SSE subscribed event #73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
728743c
fixes
HardMax71 e1fcdd9
fixes
HardMax71 9c2b6e8
fixes
HardMax71 884fb04
fixes
HardMax71 118bd6d
fixes
HardMax71 277cabf
fixes
HardMax71 6027ac4
fixes
HardMax71 7cc31fe
fixes (removed duplicate logins)
HardMax71 4abcb71
flaky test fix
HardMax71 aa3c8ac
xdist-group
HardMax71 5489e39
pyproject fix
HardMax71 93a79e6
durations=0 for tests: checking which ones take most time
HardMax71 bc944e1
optimizations of sse routes tests
HardMax71 48aa71f
optimizations of sse routes tests
HardMax71 824d686
security service added to DI, simplified settings passing
HardMax71 e88e606
more DI for settings obj, also removed random bootstrap_servers being…
HardMax71 72a7733
mypy
HardMax71 2cb4d4d
removed env block from toml file, deps updated accordingly
HardMax71 9a22a4c
Otel transient error fix
HardMax71 b3cdef8
Otel transient error fix + DI for metrics
HardMax71 4d78cc1
other fixes
HardMax71 57fd0b1
other fixes
HardMax71 8120957
mypy fixes part 1
HardMax71 a3907bb
mypy fixes part 2
HardMax71 47d1215
mypy fixes part 3
HardMax71 8cde784
mypy fixes part 5; also added csrf middleware
HardMax71 5796a27
less fixtures, passing directly xx_user fixture as logged in user of …
HardMax71 726e2f9
settings fixes (lifespan now reads from DI)
HardMax71 f1109d5
new sse event (subscribed), updated tests, dlq retry msgs moved to ma…
HardMax71 4452603
fixes
HardMax71 a202287
fixes
HardMax71 c6837fc
fixes
HardMax71 e67ef8a
fixes
HardMax71 1016bf7
single source of truth regarding loading Settings - DI (not get_setti…
HardMax71 d375e5e
fixes + parallel execution
HardMax71 2f1a020
non-flaky tests
HardMax71 225c737
fixes
HardMax71 a5a1f99
fixes
HardMax71 9ff0f12
fixes
HardMax71 2f0ee35
fixes
HardMax71 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -47,6 +47,7 @@ jobs: | |
| run: | | ||
| cd backend | ||
| uv run pytest tests/unit -v -rs \ | ||
| --durations=0 \ | ||
| --cov=app \ | ||
| --cov-report=xml --cov-report=term | ||
|
|
||
|
|
@@ -90,32 +91,12 @@ jobs: | |
| docker compose -f docker-compose.ci.yaml up -d --wait --wait-timeout 120 | ||
| docker compose -f docker-compose.ci.yaml ps | ||
|
|
||
| - name: Create Kafka topics | ||
| timeout-minutes: 2 | ||
| env: | ||
| KAFKA_BOOTSTRAP_SERVERS: localhost:9092 | ||
| KAFKA_TOPIC_PREFIX: "ci.${{ github.run_id }}." | ||
| run: | | ||
| cd backend | ||
| uv run python -m scripts.create_topics | ||
|
|
||
| - name: Run integration tests | ||
| timeout-minutes: 10 | ||
| env: | ||
| MONGO_ROOT_USER: root | ||
| MONGO_ROOT_PASSWORD: rootpassword | ||
| MONGODB_HOST: 127.0.0.1 | ||
| MONGODB_PORT: 27017 | ||
| MONGODB_URL: mongodb://root:[email protected]:27017/?authSource=admin | ||
| KAFKA_BOOTSTRAP_SERVERS: localhost:9092 | ||
| KAFKA_TOPIC_PREFIX: "ci.${{ github.run_id }}." | ||
| SCHEMA_REGISTRY_URL: http://localhost:8081 | ||
| REDIS_HOST: localhost | ||
| REDIS_PORT: 6379 | ||
| SCHEMA_SUBJECT_PREFIX: "ci.${{ github.run_id }}." | ||
| run: | | ||
| cd backend | ||
| uv run pytest tests/integration -v -rs \ | ||
| --durations=0 \ | ||
| --cov=app \ | ||
| --cov-report=xml --cov-report=term | ||
|
|
||
|
|
@@ -184,32 +165,15 @@ jobs: | |
| timeout 90 bash -c 'until sudo k3s kubectl cluster-info; do sleep 5; done' | ||
| kubectl create namespace integr8scode --dry-run=client -o yaml | kubectl apply -f - | ||
|
|
||
| - name: Create Kafka topics | ||
| timeout-minutes: 2 | ||
| env: | ||
| KAFKA_BOOTSTRAP_SERVERS: localhost:9092 | ||
| KAFKA_TOPIC_PREFIX: "ci.${{ github.run_id }}." | ||
| run: | | ||
| cd backend | ||
| uv run python -m scripts.create_topics | ||
|
|
||
| - name: Run E2E tests | ||
| timeout-minutes: 10 | ||
| env: | ||
| MONGO_ROOT_USER: root | ||
| MONGO_ROOT_PASSWORD: rootpassword | ||
| MONGODB_URL: mongodb://root:[email protected]:27017/?authSource=admin | ||
| KAFKA_BOOTSTRAP_SERVERS: localhost:9092 | ||
| KAFKA_TOPIC_PREFIX: "ci.${{ github.run_id }}." | ||
| SCHEMA_REGISTRY_URL: http://localhost:8081 | ||
| REDIS_HOST: localhost | ||
| REDIS_PORT: 6379 | ||
| SCHEMA_SUBJECT_PREFIX: "ci.${{ github.run_id }}." | ||
| KUBECONFIG: /home/runner/.kube/config | ||
| K8S_NAMESPACE: integr8scode | ||
| run: | | ||
| cd backend | ||
| uv run pytest tests/e2e -v -rs \ | ||
| --durations=0 \ | ||
| --cov=app \ | ||
| --cov-report=xml --cov-report=term | ||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.