From fdd16f6c75aa4f88baf9183410b68a85949c01de Mon Sep 17 00:00:00 2001 From: idavidov Date: Sat, 19 Aug 2023 15:40:40 +0300 Subject: [PATCH] raize exception when no diffs in MR --- pr_agent/git_providers/gitlab_provider.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pr_agent/git_providers/gitlab_provider.py b/pr_agent/git_providers/gitlab_provider.py index ef7210ee..6a9f9fe2 100644 --- a/pr_agent/git_providers/gitlab_provider.py +++ b/pr_agent/git_providers/gitlab_provider.py @@ -174,8 +174,13 @@ class GitLabProvider(GitProvider): def get_relevant_diff(self, relevant_file, relevant_line_in_file): changes = self.mr.changes() # Retrieve the changes for the merge request once + if not changes: + logging.error('No changes found for the merge request.') + return None all_diffs = self.mr.diffs.list(get_all=True) - + if not all_diffs: + logging.error('No diffs found for the merge request.') + raise ValueError(f"Could not get diff for merge request {self.id_mr}") for d in all_diffs: for change in changes['changes']: if change['new_path'] == relevant_file and relevant_line_in_file in change['diff']: