Skip to content

TST: Turning warnings into exceptions#269

Open
pllim wants to merge 4 commits into
spacetelescope:mainfrom
pllim:warn2err
Open

TST: Turning warnings into exceptions#269
pllim wants to merge 4 commits into
spacetelescope:mainfrom
pllim:warn2err

Conversation

@pllim

@pllim pllim commented May 21, 2026

Copy link
Copy Markdown
Contributor

To debug something Robert told me about but also just good to do in general. Also, we don't use Jenkins anymore.

RTs:

Fix #38

@codecov

codecov Bot commented May 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@80b32f9). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #269   +/-   ##
=======================================
  Coverage        ?   79.09%           
=======================================
  Files           ?       34           
  Lines           ?     1612           
  Branches        ?        0           
=======================================
  Hits            ?     1275           
  Misses          ?      337           
  Partials        ?        0           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pllim

This comment was marked as resolved.

@pllim pllim marked this pull request as ready for review May 21, 2026 18:09
@pllim pllim requested a review from a team as a code owner May 21, 2026 18:09
@pllim

pllim commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

OK @stscirij the log is here: https://github.com/spacetelescope/RegressionTests/actions/runs/26292184640/job/77395063193

From what I can understand there, the violation is in the "truth" files for those 2 tests. So to debug, you can go into Artifactory, grab those 2 offending truth files, open it with astropy dev or 8.0.0b1, and you should see the same warning. Then hopefully the warning message would give you enough hints to fix the problem. Once fixed, you replace the truth files, and the problem should go away. Then you can ping me here and I can rerun the RT again with this branch. Hope this helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TST: Turn deprecation warnings into exceptions

1 participant