Skip to content

Commit

Permalink
Remove mark_analyses_deleted
Browse files Browse the repository at this point in the history
  • Loading branch information
seallard committed Jan 10, 2024
1 parent 18f37a1 commit c1b33ad
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 28 deletions.
14 changes: 0 additions & 14 deletions cg/apps/tb/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,20 +91,6 @@ def is_latest_analysis_completed(self, case_id: str) -> bool:
def is_latest_analysis_qc(self, case_id: str) -> bool:
return self.get_latest_analysis_status(case_id=case_id) == AnalysisStatus.QC

def mark_analyses_deleted(self, case_id: str) -> list | None:
"""Mark all analyses for case deleted without removing analysis files"""
request_body = {
"case_id": case_id,
}
response = self.query_trailblazer(
command="mark-analyses-deleted", request_body=request_body
)
if response:
if isinstance(response, list):
return [TrailblazerAnalysis.model_validate(analysis) for analysis in response]
if isinstance(response, dict):
return [TrailblazerAnalysis.model_validate(response)]

def add_pending_analysis(
self,
case_id: str,
Expand Down
1 change: 0 additions & 1 deletion cg/meta/workflow/analysis.py
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,6 @@ def get_analysis_finish_path(self, case_id: str) -> Path:

def add_pending_trailblazer_analysis(self, case_id: str) -> None:
self.check_analysis_ongoing(case_id)
self.trailblazer_api.mark_analyses_deleted(case_id)
self.trailblazer_api.add_pending_analysis(
case_id=case_id,
email=environ_email(),
Expand Down
10 changes: 0 additions & 10 deletions tests/cli/workflow/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@ class MockTB:

def __init__(self):
self._link_was_called = False
self._mark_analyses_deleted_called = False
self._add_pending_was_called = False
self._add_pending_analysis_was_called = False
self._family = None
Expand Down Expand Up @@ -187,11 +186,6 @@ def first_was_called(self):

return Row()

def mark_analyses_deleted(self, case_id: str):
"""Mock this function"""
self._case_id = case_id
self._mark_analyses_deleted_called = True

def add_pending(self, case_id: str, email: str):
"""Mock this function"""
self._case_id = case_id
Expand All @@ -204,10 +198,6 @@ def add_pending_analysis(self, case_id: str, email: str):
self._email = email
self._add_pending_analysis_was_called = True

def mark_analyses_deleted_called(self):
"""check if mark_analyses_deleted was called"""
return self._mark_analyses_deleted_called

def add_pending_was_called(self):
"""check if add_pending was called"""
return self._add_pending_was_called
Expand Down
3 changes: 0 additions & 3 deletions tests/mocks/tb_mock.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@ def is_latest_analysis_ongoing(self, *args, **kwargs) -> bool:
def add_pending_analysis(self, *args, **kwargs) -> None:
return None

def mark_analyses_deleted(self, *args, **kwargs) -> None:
return None

def add_commit(self, *args, **kwargs) -> None:
return None

Expand Down

0 comments on commit c1b33ad

Please sign in to comment.