Change get_constructor_params to instead by get_deployment_params and…

… use contract container as first entry so that it isn't repeated in the deployment script.

scripts/deploy_lynx.py

@@ -37,14 +37,12 @@ def main():
     config = DeploymentConfig.from_file(DEPLOYMENT_CONFIG_FILEPATH)
 
     LynxRootApplication = deployer.deploy(
-        project.LynxRootApplication,  # TODO should the container be returned by call below?
-        *config.get_constructor_params(project.LynxRootApplication, locals()),
+        *config.get_deployment_params(project.LynxRootApplication, locals()),
         publish=PUBLISH,
     )
 
     LynxTACoChildApplication = deployer.deploy(
-        project.LynxTACoChildApplication,
-        *config.get_constructor_params(project.LynxTACoChildApplication, locals()),
+        *config.get_deployment_params(project.LynxTACoChildApplication, locals()),
         publish=PUBLISH,
     )
 
@@ -55,15 +53,13 @@ def main():
     )
 
     LynxRitualToken = deployer.deploy(
-        project.LynxRitualToken,
-        *config.get_constructor_params(project.LynxRitualToken, locals()),
+        *config.get_deployment_params(project.LynxRitualToken, locals()),
         publish=PUBLISH,
     )
 
     # Lynx Coordinator
     Coordinator = deployer.deploy(
-        project.Coordinator,
-        *config.get_constructor_params(project.Coordinator, locals()),
+        *config.get_deployment_params(project.Coordinator, locals()),
         publish=PUBLISH,
     )
 

scripts/deployment.py

@@ -84,7 +84,7 @@ def from_file(cls, config_filepath: Path) -> "DeploymentConfig":
             config = OrderedDict(json.load(config_file))
         return cls(config)
 
-    def get_constructor_params(
+    def get_deployment_params(
         self, container: ContractContainer, context: typing.Dict[str, Any], interactive: bool = True
     ) -> List[Any]:
         contract_name = container.contract_type.name
@@ -95,4 +95,5 @@ def get_constructor_params(
         )
         if interactive:
             _confirm_resolution(resolved_params, contract_name)
-        return list(resolved_params.values())
+
+        return [container, *resolved_params.values()]