Account for edge case where apiVersion param has no default (#678)

Author: lgarber-akamai

linodecli/baked/operation.py

@@ -447,7 +447,13 @@ def _resolve_api_version(
             None,
         )
         if version_param is not None:
-            return version_param.schema.default
+            schema = version_param.schema
+
+            if schema.default:
+                return schema.default
+
+            if schema.enum and len(schema.enum) > 0:
+                return schema.enum[0]
 
         return None
 

tests/fixtures/api_url_components_test.yaml

@@ -46,3 +46,21 @@ paths:
           description: foobar
           content:
             application/json: {}
+
+  /{apiVersion}/bar:
+    parameters:
+    - name: apiVersion
+      in: path
+      required: true
+      schema:
+        type: string
+        enum:
+          - v1000
+          - v1000beta
+    get:
+      operationId: barGet
+      responses:
+        '200':
+          description: foobar
+          content:
+            application/json: {}

tests/unit/test_operation.py

@@ -296,6 +296,25 @@ def test_resolve_api_components(self, get_openapi_for_api_components_tests):
             operation=root.paths["/{apiVersion}/bar/foo"].post, params=[]
         ) == ("http://localhost", "/{apiVersion}/bar/foo", "v100beta")
 
+    def test_resolve_api_version(self, get_openapi_for_api_components_tests):
+        root = get_openapi_for_api_components_tests
+
+        assert (
+            OpenAPIOperation._resolve_api_version(
+                params=root.paths["/{apiVersion}/bar/foo"].parameters,
+                server_url="http://localhost",
+            )
+            == "v9canary"
+        )
+
+        assert (
+            OpenAPIOperation._resolve_api_version(
+                params=root.paths["/{apiVersion}/bar"].parameters,
+                server_url="http://localhost",
+            )
+            == "v1000"
+        )
+
     def test_resolve_docs_url_legacy(self, get_openapi_for_docs_url_tests):
         root = get_openapi_for_docs_url_tests