fix undefined _fbthrift__py3_module_name bug

Summary:
This fixes `NameError: name '_fbthrift__py3_module_name' is not defined`.

The current template only defines this symbol when the IDL defines at least one type (`struct`, `union`, `typedef`, or `enum`). However, it's possible for an IDL to define no types but include `service` methods with containers. The symbol is needed if there are any containers, even containers of primitives.

So we move the variable name definition out of the `#program:has_types?` gate.

Reviewed By: stanleyz510

Differential Revision: D85972493

fbshipit-source-id: 2cf81e37159878fe2fbe07b7bd49a9d3f2209365

Author: ahilger

thrift/compiler/generate/templates/py3/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py.mustache

@@ -34,6 +34,8 @@ want to include.
 }}
 {{> common/auto_generated_py}}
 {{#program:auto_migrate?}}
+{{! Note this is needed even if IDL defines no types, in the case where service has arguments that are container<primitive>}}
+_fbthrift__py3_module_name = "{{#program:py3Namespaces}}{{value}}.{{/program:py3Namespaces}}{{program:name}}.types"
 {{#program:has_types?}}
 from {{#program:py3Namespaces}}{{value}}.{{/program:py3Namespaces}}{{program:name}}.thrift_types import (
     {{#program:structs}}
@@ -46,7 +48,6 @@ from {{#program:py3Namespaces}}{{value}}.{{/program:py3Namespaces}}{{program:nam
     {{typedef:name}},
     {{/program:typedefs}}
 )
-_fbthrift__py3_module_name = "{{#program:py3Namespaces}}{{value}}.{{/program:py3Namespaces}}{{program:name}}.types"
 {{/program:has_types?}}
 {{#program:hasContainerTypes}}
 import importlib

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate/gen-py3/includes/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,11 +4,11 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "includes.types"
 from includes.thrift_types import (
     Included,
     ExampleIncluded,
     IncludedConstant,
     IncludedInt64,
     TransitiveFoo,
 )
-_fbthrift__py3_module_name = "includes.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate/gen-py3/module/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,7 +4,7 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "module.types"
 from module.thrift_types import (
     MyStruct,
 )
-_fbthrift__py3_module_name = "module.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate/gen-py3/transitive/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,8 +4,8 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "transitive.types"
 from transitive.thrift_types import (
     Foo,
     ExampleFoo,
 )
-_fbthrift__py3_module_name = "transitive.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_includes/gen-py3/includes/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,11 +4,11 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "includes.types"
 from includes.thrift_types import (
     Included,
     ExampleIncluded,
     IncludedConstant,
     IncludedInt64,
     TransitiveFoo,
 )
-_fbthrift__py3_module_name = "includes.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_includes/gen-py3/transitive/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,8 +4,8 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "transitive.types"
 from transitive.thrift_types import (
     Foo,
     ExampleFoo,
 )
-_fbthrift__py3_module_name = "transitive.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_service/gen-py3/includes/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,11 +4,11 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "includes.types"
 from includes.thrift_types import (
     Included,
     ExampleIncluded,
     IncludedConstant,
     IncludedInt64,
     TransitiveFoo,
 )
-_fbthrift__py3_module_name = "includes.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_service/gen-py3/module/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,7 +4,7 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "module.types"
 from module.thrift_types import (
     MyStruct,
 )
-_fbthrift__py3_module_name = "module.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_service/gen-py3/service/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,8 +4,8 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "service.types"
 from service.thrift_types import (
     IncludesIncluded,
     IncludesTransitiveFoo,
 )
-_fbthrift__py3_module_name = "service.types"

thrift/compiler/test/fixtures/includes/out/py3_auto_migrate_service/gen-py3/transitive/types_auto_FBTHRIFT_ONLY_DO_NOT_USE.py

@@ -4,8 +4,8 @@
 # DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
 #  @generated
 #
+_fbthrift__py3_module_name = "transitive.types"
 from transitive.thrift_types import (
     Foo,
     ExampleFoo,
 )
-_fbthrift__py3_module_name = "transitive.types"