Merge branch 'rvinnakota/indirect-misses' into staging

Author: rohitvinnakota-codecov

codecov/settings_base.py

@@ -412,15 +412,13 @@
             HttpxIntegration(),
         ],
         environment=SENTRY_ENV,
-        set_default_pii=True,
         traces_sample_rate=SENTRY_SAMPLE_RATE,
         profiles_sample_rate=float(
             os.environ.get("SERVICES__SENTRY__PROFILE_SAMPLE_RATE", 0.01)
         ),
     )
     if os.getenv("CLUSTER_ENV"):
         sentry_sdk.set_tag("cluster", os.getenv("CLUSTER_ENV"))
-    sentry_sdk.set_user({"ip_address": "{{auto}}"})
 elif IS_DEV:
     sentry_sdk.init(
         spotlight=IS_DEV,

compare/commands/compare/interactors/tests/test_fetch_impacted_files.py

@@ -511,7 +511,7 @@ def test_impacted_files_filtered_by_misses_count_ascending(self, read_file):
         }
         comparison = None
         impacted_files = self.execute(None, self.comparison_report, comparison, filters)
-        assert [file.head_name for file in impacted_files] == ["fileA", "fileB"]
+        assert [file.head_name for file in impacted_files] == ["fileB", "fileA"]
 
     @patch("shared.api_archive.archive.ArchiveService.read_file")
     def test_impacted_files_filtered_by_misses_count_descending(self, read_file):
@@ -524,7 +524,7 @@ def test_impacted_files_filtered_by_misses_count_descending(self, read_file):
         }
         comparison = None
         impacted_files = self.execute(None, self.comparison_report, comparison, filters)
-        assert [file.head_name for file in impacted_files] == ["fileB", "fileA"]
+        assert [file.head_name for file in impacted_files] == ["fileA", "fileB"]
 
     @patch("shared.api_archive.archive.ArchiveService.read_file")
     def test_impacted_files_without_filters(self, read_file):

graphql_api/tests/test_impacted_file.py

@@ -404,6 +404,7 @@ def test_fetch_impacted_files(self, read_file):
             "commit": self.commit.commitid,
         }
         data = self.gql_request(query_impacted_files, variables=variables)
+        print(data)
         assert data == {
             "owner": {
                 "repository": {
@@ -449,7 +450,7 @@ def test_fetch_impacted_files(self, read_file):
                                             "percentCovered": 85.71428571428571
                                         },
                                         "changeCoverage": 44.047619047619044,
-                                        "missesCount": 2,
+                                        "missesCount": 1,
                                     },
                                 ]
                             },

services/comparison.py

@@ -884,7 +884,6 @@ def has_changes(self) -> bool:
     def misses_count(self) -> int:
         total_misses = 0
         total_misses += self._direct_misses_count
-        total_misses += self._unintended_misses_count
         return total_misses
 
     @cached_property