fix: add warning about python sdk login_user breaking change in 21.4.0 (

#579)

packages/sdk-codegen/src/python.gen.spec.ts

@@ -75,6 +75,7 @@ DelimSequence = model.DelimSequence
 # NOTE: Do not edit this file generated by Looker SDK Codegen
 import datetime
 from typing import Any, MutableMapping, Optional, Sequence, Union
+import warnings
 
 from . import models
 from looker_sdk.rtl import api_methods
@@ -409,6 +410,20 @@ return response`
             transport_options=transport_options
 )
 assert isinstance(response, list)
+return response`
+      const actual = gen.httpCall(indent, method)
+      expect(actual).toEqual(expected)
+    })
+    it('assert login_user has a warning', () => {
+      const method = apiTestModel.methods.login_user
+      const expected = `warnings.warn("login_user behavior changed significantly in 21.4.0. See https://git.io/JOtH1")
+response = self.post(
+            f"/login/{user_id}",
+            models.AccessToken,
+            query_params={"associative": associative},
+            transport_options=transport_options
+)
+assert isinstance(response, models.AccessToken)
 return response`
       const actual = gen.httpCall(indent, method)
       expect(actual).toEqual(expected)

packages/sdk-codegen/src/python.gen.ts

@@ -108,6 +108,7 @@ export class PythonGen extends CodeGen {
 # ${this.warnEditing()}
 import datetime
 from typing import Any, MutableMapping, Optional, Sequence, Union
+import warnings
 
 from . import models
 from ${this.packagePath}.rtl import api_methods
@@ -401,15 +402,20 @@ ${this.hooks.join('\n')}
     }
     result = this.argFill(result, returnType)
     result = this.argFill(result, `f"${method.endpoint}"`)
-    return result
+    return this.bumper(indent) + result
   }
 
   httpCall(indent: string, method: IMethod) {
     const bump = indent + this.indentStr
     const args = this.httpArgs(bump, method)
-    const methodCall = `${indent}response = ${this.it(
+    let methodCall = `${indent}response = ${this.it(
       method.httpMethod.toLowerCase()
     )}`
+    if (method.name === 'login_user') {
+      methodCall =
+        `${indent}warnings.warn("login_user behavior changed significantly ` +
+        `in 21.4.0. See https://git.io/JOtH1")\n${methodCall}`
+    }
     let assertTypeName = this.methodReturnType(method)
     switch (method.type.className) {
       case 'ArrayType':
@@ -432,7 +438,7 @@ ${this.hooks.join('\n')}
     const returnStmt = `${indent}return response`
     return (
       `${methodCall}(\n` +
-      `${bump.repeat(3)}${args}\n` +
+      `${this.bumper(indent)}${args}\n` +
       `${indent})\n` +
       `${assertion}\n` +
       `${returnStmt}`

python/looker_sdk/sdk/api31/methods.py

@@ -26,6 +26,7 @@
 # NOTE: Do not edit this file generated by Looker SDK Codegen for Looker 21.4 API 3.1
 import datetime
 from typing import Any, MutableMapping, Optional, Sequence, Union
+import warnings
 
 from . import models
 from looker_sdk.rtl import api_methods
@@ -111,6 +112,9 @@ def login_user(
         transport_options: Optional[transport.TransportOptions] = None,
     ) -> models.AccessToken:
         """Login user"""
+        warnings.warn(
+            "login_user behavior changed significantly in 21.4.0. See https://git.io/JOtH1"
+        )
         response = self.post(
             f"/login/{user_id}",
             models.AccessToken,

python/looker_sdk/sdk/api40/methods.py

@@ -26,6 +26,7 @@
 # NOTE: Do not edit this file generated by Looker SDK Codegen for Looker 21.4 API 4.0
 import datetime
 from typing import Any, MutableMapping, Optional, Sequence, Union
+import warnings
 
 from . import models
 from looker_sdk.rtl import api_methods
@@ -111,6 +112,9 @@ def login_user(
         transport_options: Optional[transport.TransportOptions] = None,
     ) -> models.AccessToken:
         """Login user"""
+        warnings.warn(
+            "login_user behavior changed significantly in 21.4.0. See https://git.io/JOtH1"
+        )
         response = self.post(
             f"/login/{user_id}",
             models.AccessToken,

release-please-config.json

@@ -40,6 +40,7 @@
     },
     "python": {
       "release-type": "python",
+      "release-as": "21.4.1",
       "draft": true,
       "package-name": "looker_sdk"
     }