Fix #1525

[ischoegl]

Changes proposed in this pull request

Avoid test failures if optional pint package is not found

If applicable, fill in the issue number this pull request is fixing

Closes #1525

Checklist

• The pull request includes a clear description of this code change
• Commit messages have short titles and reference relevant issues
• Build passes (scons build & scons test) and unit tests address code coverage
• Style & formatting of contributed code follows contributing guidelines
• The pull request is ready for review

[codecov]

Codecov Report

Merging #1569 (5b9766c) into main (6ba497c) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1569      +/-   ##
==========================================
+ Coverage   70.45%   70.47%   +0.01%     
==========================================
  Files         379      379              
  Lines       59093    59093              
  Branches    21230    21230              
==========================================
+ Hits        41636    41643       +7     
+ Misses      14376    14369       -7     
  Partials     3081     3081              

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[speth]

Thanks, @ischoegl. That's simpler than I thought it would be!

[bryanwweber]

Is pint tested anywhere on CI still after these removals?

[speth]

Yes, if you look at main.yaml, you will see that it is still installed for almost all of the runners. The cases it is removed from represent ~7 of the ~40 CI runs (and we weren't even running the tests that use pint for the various Sundials versions anyway).

[bryanwweber]

Thank you! I didn't check, sorry

[ischoegl]

Thanks, @ischoegl. That's simpler than I thought it would be!

@bryanwweber’s comment on #1525 was key ..