Skip to content

TST: Reduce timeout in test_timeout_classic#3267

Merged
bsipocz merged 1 commit into
astropy:mainfrom
pllim:vo-cone
Mar 18, 2025
Merged

TST: Reduce timeout in test_timeout_classic#3267
bsipocz merged 1 commit into
astropy:mainfrom
pllim:vo-cone

Conversation

@pllim

@pllim pllim commented Mar 18, 2025

Copy link
Copy Markdown
Member

Reduce timeout in test_timeout_classic to simulate real world timed out query. This is the remaining failure I found after server admin fixed the SSL cert for vo_conesearch queries. 🤞

to simulate real world timed out query
@pllim

pllim commented Mar 18, 2025

Copy link
Copy Markdown
Member Author

Not sure why RTD failed to start. Hopefully transient. Please attempt rerun. Thanks!

@codecov

codecov Bot commented Mar 18, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.08%. Comparing base (44b034a) to head (9cbb2b1).
Report is 213 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3267   +/-   ##
=======================================
  Coverage   69.08%   69.08%           
=======================================
  Files         232      232           
  Lines       19639    19639           
=======================================
  Hits        13568    13568           
  Misses       6071     6071           

☔ View full report in Codecov by Sentry.
📢 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.

@bsipocz

bsipocz commented Mar 18, 2025

Copy link
Copy Markdown
Member

RTD was working earlier so I'm not concerned about it (I might have just overloaded it with all the commits we were doing today...)

@bsipocz bsipocz added this to the v0.4.10 milestone Mar 18, 2025

@bsipocz bsipocz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@bsipocz bsipocz merged commit 2b259cb into astropy:main Mar 18, 2025
@pllim pllim deleted the vo-cone branch March 18, 2025 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants