fix(pacmak): use builtins.str instead of str for type annotations

DanielMSchmidt · Dec 1, 2022 · fd06e86 · fd06e86
1 parent 0b7dcc8
commit fd06e86
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/jsii-pacmak/lib/targets/python/type-name.ts b/packages/jsii-pacmak/lib/targets/python/type-name.ts
@@ -311,7 +311,7 @@ class UserType implements TypeName {
     const wrapType =
       typeAnnotation && parameterType && isStruct
         ? (pyType: string) =>
-            `typing.Union[${pyType}, typing.Dict[str, typing.Any]]`
+            `typing.Union[${pyType}, typing.Dict[builtins.str, typing.Any]]`
         : (pyType: string) => pyType;
 
     // Emit aliased imports for dependencies (this avoids name collisions)

diff --git a/packages/jsii-pacmak/test/targets/python/type-name.test.ts b/packages/jsii-pacmak/test/targets/python/type-name.test.ts
@@ -246,8 +246,8 @@ describe(toTypeName, () => {
     {
       name: 'Struct parameter type annotation',
       input: { fqn: `${assembly.name}.Struct` },
-      forwardPythonType: `typing.Union["Struct", typing.Dict[str, typing.Any]]`,
-      pythonType: `typing.Union[Struct, typing.Dict[str, typing.Any]]`,
+      forwardPythonType: `typing.Union["Struct", typing.Dict[builtins.str, typing.Any]]`,
+      pythonType: `typing.Union[Struct, typing.Dict[builtins.str, typing.Any]]`,
       context: {
         typeAnnotation: true,
         parameterType: true,