Validate svo_fps query parameter names locally

[eerovaher]

SvoFpsClass.data_from_svo() now checks the names of the query parameters and only connects with the server if all names are valid. This allows replacing a failing remote test that checked for invalid queries with a local test.

Contributes towards #2203

[codecov]

Codecov Report

Merging #2446 (7eab91e) into main (f5ff578) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2446      +/-   ##
==========================================
+ Coverage   62.90%   62.92%   +0.01%     
==========================================
  Files         133      133              
  Lines       17291    17300       +9     
==========================================
+ Hits        10877    10886       +9     
  Misses       6414     6414              

Impacted Files	Coverage Δ
astroquery/svo_fps/core.py	95.23% <100.00%> (+1.29%)	⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[bsipocz]

This will need a short changelog mention after all the API has changed to correctly raise an exception rather than return an empty result.

Otherwise all looks good.

[bsipocz]

Is there any way to parse this in rather than hard wire the params? I suppose they may not change it too often, so the hard-wiring should also work in practice.

[eerovaher]

If the parameter names are not hard-coded here then they have to be monkeypatched for non-remote tests, which requires them to be hard-coded there. I have instead added a new remote test that should raise alarms if QUERY_PARAMETER becomes outdated.

[bsipocz]

That's a solid middle ground, thanks.