Fix in the cadc module in anticipation of coming changes to the servers#2326
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2326 +/- ##
==========================================
- Coverage 62.98% 62.93% -0.06%
==========================================
Files 131 131
Lines 17059 17084 +25
==========================================
+ Hits 10745 10752 +7
- Misses 6314 6332 +18
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
| for service_def in datalink: | ||
| if service_def.semantics == 'http://www.openadc.org/caom2#pkg': | ||
| # pkg is an alternative for downloading multiple | ||
| if service_def.semantics in \ |
There was a problem hiding this comment.
no need for line break, long lines up to ~110-120 is fine when they provide better readability
|
|
||
| @pytest.mark.skipif(one_test, reason='One test mode') | ||
| @pytest.mark.skipif(not pyvo_OK, reason='not pyvo_OK') | ||
| @pytest.mark.skip('https://github.com/astropy/astroquery/issues/2325') |
There was a problem hiding this comment.
I feel leaving these as failing tests are better, it ensures that they are not forgotten. Or maybe change them to xfail, so then we would notice easily if something got fixed upstream.
| xfail_strict = true | ||
| remote_data_strict = true | ||
| addopts = --doctest-rst | ||
| #addopts = --doctest-rst |
There was a problem hiding this comment.
this has to stay, we switched on testing the docs :)
bsipocz
left a comment
There was a problem hiding this comment.
Thanks! I'll fix my comment before merging.
|
Locally I run into timeouts (I suspect my IP might got blacklisted), so merging it without ensuring that the remote tests are all OK. |
|
Thanks @andamian! |
Also had to comment out a few int tests with unrelated errors that require further investigation.
The patch is simple, has been tested and it's ready to go.