Correct api_version_text to not set a version for 0 and 1; remove test for versions higher than two, allow CDA to say version is not supported.

Author: krowvin

cwms/api.py

@@ -163,7 +163,7 @@ def api_version_text(api_version: int = 0, format: str = "") -> str:
         # XML returns from stdlib as text/xml, but CDA expects application/xml
         mimetype = "application/xml"
     # For present and future versions, append the version number if one is provided. Zero will not set a version.
-    return mimetype + f";version={api_version}" if api_version else mimetype
+    return mimetype if api_version in [0, 1] else mimetype + f";version={api_version}"
 
 
 def get(

tests/api_test.py

@@ -48,10 +48,3 @@ def test_api_headers():
 
     version = api_version_text(api_version=2)
     assert version == "application/json;version=2"
-
-
-def test_api_headers_invalid_version():
-    """An exception should be raised if the API version is not valid."""
-
-    with pytest.raises(InvalidVersion):
-        version = api_version_text(api_version=3)