Skip to content

Conversation

@berland
Copy link
Contributor

@berland berland commented Jan 2, 2026

Issue
Resolves noise in pytest output from expected warnings

Approach
Assert or ignore each warning

  • PR title captures the intent of the changes, and is fitting for release notes.
  • Added appropriate release note label
  • Commit history is consistent and clean, in line with the contribution guidelines.
  • Make sure unit tests pass locally after every commit (git rebase -i main --exec 'just rapid-tests')

When applicable

  • When there are user facing changes: Updated documentation
  • New behavior or changes to existing untested code: Ensured that unit tests are added (See Ground Rules).
  • Large PR: Prepare changes in small commits for more convenient review
  • Bug fix: Add regression test for the bug
  • Bug fix: Add backport label to latest release (format: 'backport release-branch-name')

/tmp/venv312/lib/python3.12/site-packages/_pytest/config/__init__.py:1428: PytestConfigWarning: Unknown config option: test_paths

   self._warn_or_fail_if_strict(f"Unknown config option: {key}\n")
This is not permitted according to the typing of ForwardModelStepPlugin
@berland berland self-assigned this Jan 2, 2026
@berland berland added the release-notes:skip If there should be no mention of this in release notes label Jan 2, 2026
@berland berland added this to SCOUT Jan 2, 2026
@berland berland moved this to Fast Track in SCOUT Jan 2, 2026
@berland berland moved this from Fast Track to Ready for Review in SCOUT Jan 2, 2026
The Ensemble object that is mocked is treated as as sync code, providing an AsyncMock
triggers a warning on function not being awaited
When snake_oil_case is in use, this is sometimes triggered
through hypothesis.
@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.60%. Comparing base (d553866) to head (964b8bd).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12562      +/-   ##
==========================================
- Coverage   90.62%   90.60%   -0.02%     
==========================================
  Files         432      432              
  Lines       29738    29738              
==========================================
- Hits        26949    26945       -4     
- Misses       2789     2793       +4     
Flag Coverage Δ
cli-tests 37.59% <ø> (ø)
gui-tests 68.70% <ø> (+0.01%) ⬆️
performance-and-unit-tests 74.12% <ø> (-0.02%) ⬇️
test 38.32% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 2, 2026

CodSpeed Performance Report

Merging #12562 will not alter performance

Comparing berland:resolve_warnings (964b8bd) with main (d553866)

Summary

✅ 22 untouched

@github-project-automation github-project-automation bot moved this from Ready for Review to Reviewed in SCOUT Jan 6, 2026
@berland berland merged commit 7bdb6d2 into equinor:main Jan 6, 2026
35 checks passed
@github-project-automation github-project-automation bot moved this from Reviewed to Done in SCOUT Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:skip If there should be no mention of this in release notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants