Change arg order to preserve backward compatibility and skip test whe…

…n xmllint not available

astropy/io/votable/tests/test_schema_versions.py

@@ -58,23 +58,19 @@
 ]
 
 
+@pytest.mark.skipif(sys.platform.startswith("win"), reason="Cannot test on Windows")
 @pytest.mark.parametrize(
     "votable_file,schema_version,expected_return_code,expected_msg_re", test_cases
 )
 def test_schema_versions(
     votable_file, schema_version, expected_return_code, expected_msg_re
 ):
     """Test that xmllint gives expected results for the given file and schema version."""
-
-    # We need xmllint so won't try with Windows.
-    if sys.platform.startswith("win"):
-        return
-
     try:
         rc, stdout, stderr = validate_schema(votable_file, schema_version)
     except OSError:
-        # If xmllint is not installed, we want the test to pass anyway
-        return
+        # If xmllint is not installed, we want to skip the test
+        pytest.skip("xmllint is not available so will not do schema validation")
 
     assert rc == expected_return_code
     if rc == 3:

astropy/io/votable/tree.py

@@ -1843,10 +1843,10 @@ def __init__(
         equinox=None,
         epoch=None,
         system=None,
-        refposition=None,
         id=None,
         config=None,
         pos=None,
+        refposition=None,
         **extra,
     ):
         if config is None: