-
Notifications
You must be signed in to change notification settings - Fork 191
CLOUD-929 - Initial pytest test migration #2058
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
Open
jvpasinatto
wants to merge
33
commits into
main
Choose a base branch
from
pytest-complete
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
f3e089d
Initial pytest test migration
jvpasinatto 7bdf097
Merge branch 'main' into pytest-complete
jvpasinatto fb96a0c
Refactor to adress comments
jvpasinatto 1fc136f
Merge branch 'main' into pytest-complete
jvpasinatto 37aa18d
Make wait more robust
jvpasinatto c784bf0
Add liveness test
jvpasinatto 4b4bf43
Merge branch 'main' into pytest-complete
gkech cafa5f4
Add python rules in makefile
jvpasinatto ab14a77
update dependencies
jvpasinatto 8fabdbe
Add more type hints
jvpasinatto 9195e8a
Print env vars in test initialization and more
jvpasinatto b572b0d
Add resources collection on failure
jvpasinatto 44858f8
move python scripts to folder
jvpasinatto ad7521d
fix liveness test and add more type hints
jvpasinatto 9adffbc
fix init deploy test
jvpasinatto 70d4f06
add py-fmt rule
jvpasinatto 974c0d1
Update test readme and small fixes
jvpasinatto 11eed52
use rich to handle logging
jvpasinatto 8a100f3
Add bash wrapper
jvpasinatto ad60319
update lock file
jvpasinatto ba739c0
fix openshift detection
jvpasinatto a8cd1d4
divide tools into separated files plus improvements
jvpasinatto 079499c
Fix report generation
jvpasinatto 9a7560b
Merge branch 'main' into pytest-complete
jvpasinatto 76c1436
Merge branch 'main' into pytest-complete
jvpasinatto b5c5903
remove test retry and add permission in gh action py-check
jvpasinatto c6c23d3
Merge branch 'main' into pytest-complete
egegunes fb8cc71
Merge branch 'main' into pytest-complete
gkech ee5f59a
fixes
jvpasinatto cfde3d8
Merge branch 'main' into pytest-complete
jvpasinatto d051208
tests updates and fixes
jvpasinatto 2a8e134
more fixes
jvpasinatto 78121c5
fixes
jvpasinatto 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
Some comments aren't visible on the classic Files Changed page.
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 |
|---|---|---|
| @@ -0,0 +1,36 @@ | ||
| name: e2e-tests Python Quality Check | ||
|
|
||
| on: | ||
| pull_request: | ||
| paths: | ||
| - 'e2e-tests/**/*.py' | ||
| - 'pyproject.toml' | ||
| - 'uv.lock' | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| quality-check: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v5 | ||
|
|
||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version-file: "pyproject.toml" | ||
|
|
||
| - name: Install dependencies | ||
| run: uv sync --locked | ||
|
|
||
| - name: Run ruff check | ||
| run: uv run ruff check e2e-tests/ | ||
|
|
||
| - name: Run mypy | ||
| run: uv run mypy e2e-tests/ | ||
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
Large diffs are not rendered by default.
Oops, something went wrong.
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.
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.