re-add the RuleModel code to see if that will help these other failing tests for now, start updating tests that are failing because of the logic change

Author: saponifi3d

src/sentry/workflow_engine/receivers/project_workflows.py

@@ -39,15 +39,16 @@
 def create_default_rules(
     project: Project,
     default_rules: bool = False,
+    RuleModel: type[Rule] = Rule,
     **kwargs: Any,
 ) -> None:
     rule_data = DEFAULT_RULE_DATA
 
     if not default_rules:
         return
 
-    with transaction.atomic(router.db_for_write(Rule)):
-        rule = Rule.objects.create(
+    with transaction.atomic(router.db_for_write(RuleModel)):
+        rule = RuleModel.objects.create(
             project=project,
             label=DEFAULT_RULE_LABEL,
             data=rule_data,

tests/sentry/receivers/test_onboarding.py

@@ -175,7 +175,11 @@ def test_project_created__default_workflow(self) -> None:
 
         assert Detector.objects.filter(project=project, type=ErrorGroupType.slug).count() == 1
         assert Detector.objects.filter(project=project, type=IssueStreamGroupType.slug).count() == 1
-        assert DetectorWorkflow.objects.filter(workflow=workflow).count() == 2
+
+        # Ensuer we have 1 connection to the issue stream, this triggers for both monitors above.
+        result_connection = DetectorWorkflow.objects.filter(workflow=workflow)
+        assert result_connection.count() == 1
+        assert result_connection[0].type == IssueStreamGroupType.slug
 
     @patch("sentry.analytics.record", wraps=record)
     def test_project_created_with_origin(self, record_analytics: MagicMock) -> None: