-
Notifications
You must be signed in to change notification settings - Fork 15
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
Test with aiida-core=2.6.1 and python 3.11 #760
Merged
Merged
Changes from 11 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
224caee
Tests with aiida-core=2.6.1
danielhollas 0b91268
Update links in README
danielhollas 6c6fa05
Collect coverage only for aiida-core=2.3 runs
danielhollas 54f1108
Report pytest timings
danielhollas 3a67907
More durations
danielhollas 3e799cf
Filter warnings as in AWB
danielhollas 8d24942
Don't report durations below 1s
danielhollas fa88aee
Bump pytest
danielhollas e658a2d
Fix integration tests on PRs from forks
danielhollas d3d465f
less warnings filtered
danielhollas 9b0b741
Merge branch 'main' into test-aiida-2.6.1
danielhollas 6b8b422
bump uv version
danielhollas 1de2940
Bump pytest-cov
danielhollas 68d3dba
Revert full-stack version
danielhollas 1d808fa
Merge branch 'main' into test-aiida-2.6.1
danielhollas ed4b525
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 18dfd9e
Bump uv version
danielhollas a0c5364
Update ruff
danielhollas 4b68ca3
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 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 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 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 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 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 |
---|---|---|
|
@@ -4,3 +4,14 @@ requires = [ | |
"wheel" | ||
] | ||
build-backend = "setuptools.build_meta" | ||
|
||
[tool.pytest.ini_options] | ||
addopts = '--strict-config --strict-markers --durations=30 --durations-min=1 -ra' | ||
filterwarnings = [ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I took these from AWB. There are definitely more warnings that either have to be fixed or ignored. |
||
'ignore:Object of type .* not in session, .* operation along .* will not proceed:', | ||
'ignore:Creating AiiDA configuration:UserWarning:', | ||
'ignore:metadata.*traitlets.traitlets.Unicode object:DeprecationWarning:', | ||
# For some reason we get this error, see | ||
# https://github.com/aiidalab/aiidalab-widgets-base/issues/551 | ||
'ignore:Exception ignored in:pytest.PytestUnraisableExceptionWarning:_pytest', | ||
] |
This file contains 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason, running tests with aiida-core >=2.4 takes significantly more time (by 2 minutes), see #766. We can speed up these tests by not collecting the code coverage, i.e. we only collect code coverage with aiida-core==2.3.
With this change the test times are roughly equal.
(note we use a similar trick in aiida-core)