From 3d56d2266307c9df17649f86d64b8227e9e7ad82 Mon Sep 17 00:00:00 2001
From: ImMin5 <mino@megazone.com>
Date: Fri, 26 Jan 2024 16:11:46 +0900
Subject: [PATCH] fix: check workspace error at identity manager

---
 .../cost_analysis/manager/identity_manager.py        |  3 +--
 .../cost_analysis/model/cost_report_data/request.py  |  3 +--
 .../cost_analysis/service/cost_report_serivce.py     | 12 ++++++------
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/spaceone/cost_analysis/manager/identity_manager.py b/src/spaceone/cost_analysis/manager/identity_manager.py
index ef568d9a..f46b1618 100644
--- a/src/spaceone/cost_analysis/manager/identity_manager.py
+++ b/src/spaceone/cost_analysis/manager/identity_manager.py
@@ -29,12 +29,11 @@ def get_domain_name(self, domain_id: str) -> str:
     def check_workspace(self, workspace_id: str, domain_id: str) -> None:
         system_token = config.get_global("TOKEN")
 
-        domain_info = self.identity_conn.dispatch(
+        self.identity_conn.dispatch(
             "Workspace.check",
             {"workspace_id": workspace_id, "domain_id": domain_id},
             token=system_token,
         )
-        return domain_info["name"]
 
     @cache.cacheable(
         key="cost-analysis:workspace-name:{domain_id}:{workspace_id}:name", expire=300
diff --git a/src/spaceone/cost_analysis/model/cost_report_data/request.py b/src/spaceone/cost_analysis/model/cost_report_data/request.py
index ce5dffae..1d6df0f8 100644
--- a/src/spaceone/cost_analysis/model/cost_report_data/request.py
+++ b/src/spaceone/cost_analysis/model/cost_report_data/request.py
@@ -26,8 +26,7 @@ class CostReportDataSearchQueryRequest(BaseModel):
 class CostReportDataAnalyzeQueryRequest(BaseModel):
     query: Union[dict, None] = None
     cost_report_data_id: Union[str, None] = None
-    report_year: Union[str, None] = None
-    report_month: Union[str, None] = None
+    is_confirmed: Union[bool, None] = None
     product: Union[str, None] = None
     provider: Union[str, None] = None
     data_source_id: Union[str, None] = None
diff --git a/src/spaceone/cost_analysis/service/cost_report_serivce.py b/src/spaceone/cost_analysis/service/cost_report_serivce.py
index fe923680..ae8dc5da 100644
--- a/src/spaceone/cost_analysis/service/cost_report_serivce.py
+++ b/src/spaceone/cost_analysis/service/cost_report_serivce.py
@@ -116,7 +116,7 @@ def get(self, params: CostReportGetRequest) -> Union[CostReportResponse, dict]:
     )
     @append_query_filter(
         [
-            "state",
+            "status",
             "cost_report_id",
             "workspace_id",
             "domain_id",
@@ -126,7 +126,8 @@ def get(self, params: CostReportGetRequest) -> Union[CostReportResponse, dict]:
         [
             "report_number",
             "workspace_name",
-            "report_year" "report_month",
+            "report_year",
+            "report_month",
         ]
     )
     @convert_model
@@ -137,9 +138,8 @@ def list(
 
         query = params.query or {}
 
-        # todo: temporary code
-        # if params.status is None:
-        #     query["filter"].append({"k": "status", "v": "SUCCESS", "o": "eq"})
+        if params.status is None:
+            query["filter"].append({"k": "status", "v": "SUCCESS", "o": "eq"})
 
         cost_report_vos, total_count = self.cost_report_mgr.list_cost_reports(query)
 
@@ -170,7 +170,7 @@ def stat(self, params: CostReportDataStatQueryRequest) -> dict:
         """Stat cost reports"""
 
         query = params.query or {}
-        return self.cost_report_mgr.stat_cost_reports(params.query)
+        return self.cost_report_mgr.stat_cost_reports(query)
 
     def create_cost_report(self, cost_report_config_vo: CostReportConfig):
         domain_id = cost_report_config_vo.domain_id