Fix types

Author: snigdhas

src/sentry/models/groupopenperiod.py

@@ -243,7 +243,7 @@ def get_open_periods_for_group(
     return open_periods
 
 
-def create_open_period(group: Group, start_time: datetime) -> GroupOpenPeriod:
+def create_open_period(group: Group, start_time: datetime) -> None:
     if not features.has("organizations:issue-open-periods", group.project.organization):
         return
 
@@ -254,7 +254,7 @@ def create_open_period(group: Group, start_time: datetime) -> GroupOpenPeriod:
 
     # There are some historical cases where we log multiple regressions for the same group,
     # but we only want to create a new open period for the first regression
-    return GroupOpenPeriod.objects.create(
+    GroupOpenPeriod.objects.create(
         group=group,
         project=group.project,
         date_started=start_time,

tests/sentry/workflow_engine/models/test_incident_groupopenperiod.py

@@ -159,7 +159,12 @@ def test_create_second_relationship(self) -> None:
 
         open_period.group.update(status=GroupStatus.RESOLVED)
         open_period.update(date_ended=timezone.now())
-        open_period_2 = create_open_period(open_period.group, timezone.now())
+        create_open_period(open_period.group, timezone.now())
+        open_period_2 = (
+            GroupOpenPeriod.objects.filter(group=open_period.group)
+            .order_by("-date_started")
+            .first()
+        )
 
         # Create new incident and new relationship
         incident2 = self.create_incident(