Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

Commit 8ed604e

Browse files
update
1 parent 6f15258 commit 8ed604e

File tree

2 files changed

+14
-7
lines changed

2 files changed

+14
-7
lines changed

graphql_api/types/impacted_file/impacted_file.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,12 @@ def resolve_segments(
9898
# segments with no diff changes and at least 1 unintended change
9999
segments = [segment for segment in segments if segment.has_unintended_changes]
100100
elif filters.get("has_unintended_changes") is False:
101-
file_comparison.reset_lines_without_unintended_changes()
102-
segments = file_comparison.segments
101+
filtered_segments = []
102+
for segment in segments:
103+
if segment.has_diff_changes:
104+
segment.reset_lines_without_unintended_changes()
105+
filtered_segments.append(segment)
106+
segments = filtered_segments
103107

104108
return SegmentComparisons(results=segments)
105109

services/comparison.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -481,14 +481,17 @@ def has_unintended_changes(self):
481481
return True
482482
return False
483483

484+
484485
def reset_lines_without_unintended_changes(self):
485486
if self.lines is None:
486487
return
487-
filtered = [l for l in self.lines if not l.get('unintended_change', False)]
488-
if hasattr(self, '_calculated_changes_and_lines'):
489-
summary, _ = self._calculated_changes_and_lines
490-
self._calculated_changes_and_lines = (summary, filtered)
491-
self.lines = filtered
488+
new_lines = []
489+
for line in self.lines:
490+
head_coverage = line.coverage["base"]
491+
base_coverage = line.coverage["head"]
492+
if not (line.added or line.removed) and (base_coverage != head_coverage):
493+
new_lines.append(line)
494+
self.lines = new_lines
492495

493496

494497
class FileComparison:

0 commit comments

Comments
 (0)