Skip to content

Commit 267c439

Browse files
committed
added downloader factory helper
1 parent f237d85 commit 267c439

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

package/cloudshell/iac/terraform/downloaders/downloader.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import logging
12
from typing import Type
23

34
from cloudshell.iac.terraform.constants import ATTRIBUTE_NAMES
@@ -23,7 +24,7 @@ def download_terraform_module(self) -> str:
2324

2425
# get downloader mapped to git provider
2526
provider = self._shell_helper.attr_handler.get_attribute(ATTRIBUTE_NAMES.GIT_PROVIDER)
26-
downloader = self._get_downloader_class(provider)(logger=self._shell_helper.logger)
27+
downloader = self._downloader_factory(provider, logger=self._shell_helper.logger)
2728

2829
# download repo and return working dir
2930
self._shell_helper.sandbox_messages.write_message("downloading Terraform module from repository...")
@@ -53,3 +54,7 @@ def _get_downloader_class(self, git_provider: str) -> Type[GitScriptDownloaderBa
5354
if git_provider.lower() not in git_downloader_map:
5455
raise NotImplementedError(f"Git Provider '{git_provider}' not supported")
5556
return git_downloader_map[git_provider.lower()]
57+
58+
def _downloader_factory(self, git_provider: str, logger: logging.Logger) -> GitScriptDownloaderBase:
59+
downloader_class = self._get_downloader_class(git_provider)
60+
return downloader_class(logger)

0 commit comments

Comments
 (0)