Skip to content

Commit b9d85b9

Browse files
authored
Merge pull request #43 from stat-kwon/master
Change validation of options variable
2 parents 4d621ae + 657e871 commit b9d85b9

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/cloudforet/cost_analysis/connector/google_storage_collector.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def get_cost_data(self, bucket_name: str):
5959

6060
@staticmethod
6161
def _check_options(options: dict) -> None:
62-
if "base_url" not in options:
62+
if "base_url" not in options or "bucket_name" not in options:
6363
raise ERROR_REQUIRED_PARAMETER(key="options.base_url")
6464

6565
@staticmethod

src/cloudforet/cost_analysis/connector/http_file_connector.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ def get_cost_data(self, base_url):
5050

5151
@staticmethod
5252
def _check_options(options: dict) -> None:
53-
if "base_url" not in options:
53+
if "base_url" not in options or "bucket_name" not in options:
5454
raise ERROR_REQUIRED_PARAMETER(key="options.base_url")
5555

5656
def _get_csv(self, base_url: str) -> List[dict]:

0 commit comments

Comments
 (0)