Skip to content

Unit test for everest constraints plot#13704

Merged
eilskra merged 2 commits into
equinor:mainfrom
eilskra:constraint-plot-test
Jun 4, 2026
Merged

Unit test for everest constraints plot#13704
eilskra merged 2 commits into
equinor:mainfrom
eilskra:constraint-plot-test

Conversation

@eilskra
Copy link
Copy Markdown
Contributor

@eilskra eilskra commented Jun 3, 2026

Issue
Resolves #13597

Approach
💭🧪📈
🤔

  • 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')

@eilskra eilskra self-assigned this Jun 3, 2026
@eilskra eilskra added the maintenance Not a bug now but could be one day, repaying technical debt label Jun 3, 2026
@eilskra eilskra changed the title Constraint plot test Unit test for everest constraints plot Jun 3, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.09%. Comparing base (e7fc16b) to head (6c5bf40).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13704      +/-   ##
==========================================
+ Coverage   89.94%   90.09%   +0.15%     
==========================================
  Files         468      468              
  Lines       33402    33411       +9     
==========================================
+ Hits        30043    30103      +60     
+ Misses       3359     3308      -51     
Flag Coverage Δ
cli-tests 35.79% <ø> (-0.01%) ⬇️
fuzz 43.38% <ø> (-0.01%) ⬇️
gui-tests 59.27% <ø> (-0.01%) ⬇️
performance-and-unit-tests 77.82% <ø> (-0.03%) ⬇️
test 46.08% <ø> (+0.17%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 7 files with indirect coverage changes

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 3, 2026

Merging this PR will not alter performance

✅ 36 untouched benchmarks


Comparing eilskra:constraint-plot-test (6c5bf40) with main (cbe09cc)

Open in CodSpeed

Comment thread tests/everest/unit_tests/plots/test_everest_constraints_plot.py
Comment thread tests/everest/unit_tests/plots/test_everest_constraints_plot.py Outdated
@eilskra eilskra force-pushed the constraint-plot-test branch from 1de9cfd to c16d9f5 Compare June 4, 2026 08:32
@eilskra eilskra force-pushed the constraint-plot-test branch from c16d9f5 to 6c5bf40 Compare June 4, 2026 10:23
Copy link
Copy Markdown
Contributor

@eqbech eqbech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

@eilskra eilskra merged commit b64500c into equinor:main Jun 4, 2026
40 checks passed
@eilskra eilskra deleted the constraint-plot-test branch June 4, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Not a bug now but could be one day, repaying technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Payback test coverage for everest_constraints_plot.py

4 participants