test(event_manager): Test case

src/sentry/event_manager.py

@@ -2109,8 +2109,8 @@ def _handle_regression(group: Group, event: BaseEvent, release: Release | None)
 
     # we only mark it as a regression if the event's release is newer than
     # the release which we originally marked this as resolved
-    elif GroupResolution.has_resolution(group, release):
-        return None
+    # elif GroupResolution.has_resolution(group, release):
+    #     return None
 
     elif has_pending_commit_resolution(group):
         return None

tests/sentry/event_manager/test_event_manager.py

@@ -564,6 +564,84 @@ def test_resolved_in_release_regression_activity_follows_semver(
         assert regressed_activity.data["follows_semver"] is True
         assert regressed_activity.data["resolved_in_version"] == "foo@1.0.0"
 
+    def resolve_in_release(self, group: Group, release: Release) -> Activity:
+        group.update(status=GroupStatus.RESOLVED, substatus=None)
+        resolution = GroupResolution.objects.create(
+            release=release,
+            group=group,
+            type=GroupResolution.Type.in_release,
+        )
+        return Activity.objects.create(
+            group=group,
+            project=group.project,
+            type=ActivityType.SET_RESOLVED_IN_RELEASE.value,
+            ident=resolution.id,
+            data={"version": release.version},
+        )
+
+    @mock.patch("sentry.event_manager.plugin_is_regression")
+    def test_does_not_resolve_for_semver(self, plugin_is_regression: mock.MagicMock) -> None:
+        plugin_is_regression.return_value = True
+
+        def _date(minutes: int) -> datetime:
+            return timezone.now() - timedelta(minutes=minutes)
+
+        first_release_version = "com.foo.FooTest@387.0+387.0.18"
+        first_release = self.create_release(version=first_release_version, date_added=_date(90))
+
+        manager = EventManager(
+            make_event(
+                event_id="a" * 32,
+                checksum="a" * 32,
+                # 10 minutes after release creation
+                timestamp=time() - 80 * 60,
+                release=first_release.version,
+            )
+        )
+        event = manager.save(self.project.id)
+        assert event.group is not None
+        group = event.group
+
+        patch_version6 = "com.foo.FooTest@388.0+388.0.6"
+        release_06 = self.create_release(version=patch_version6, date_added=_date(70))
+
+        patch_version8 = "com.foo.Foo@388.0+388.0.8"
+        release_08 = self.create_release(version=patch_version8, date_added=_date(65))
+
+        self.create_release(version="com.foo.Foo@388.0+388.0.13", date_added=_date(60))
+
+        # Marking as resolved in the second latest release
+        self.resolve_in_release(group, release_08)
+
+        # Creates event for an older release
+        manager = EventManager(
+            make_event(
+                event_id="c" * 32,
+                checksum="a" * 32,
+                timestamp=time(),
+                # Notice that the event is for an older release
+                release=release_06.version,
+            )
+        )
+        event = manager.save(self.project.id)
+        assert event.group is not None
+        group = event.group
+
+        # # XXX: The GH issue shows that it should have unresolved
+        # assert group.status == GroupStatus.RESOLVED
+        # # XXX: The GH issue shows that it should have a regression
+        # with pytest.raises(Activity.DoesNotExist):
+        #     Activity.objects.get(group=group, type=ActivityType.SET_REGRESSION.value)
+
+        # When GroupResolution.has_resolution(group, release) does not execute we reproduce the issue
+        assert group.status == GroupStatus.UNRESOLVED
+        regressed_activity = Activity.objects.get(
+            group=group, type=ActivityType.SET_REGRESSION.value
+        )
+        assert regressed_activity.data["version"] == release_06.version
+        assert regressed_activity.data["follows_semver"] is True
+        assert regressed_activity.data["resolved_in_version"] == release_08.version
+
     def test_has_pending_commit_resolution(self) -> None:
         project_id = self.project.id
         event = self.make_release_event("1.0", project_id)