wbcompliance 1.54.19__tar.gz → 1.59.2__tar.gz

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wbcompliance
-Version: 1.54.19
+Version: 1.59.2
 Summary: A workbench module for managing compliance.
 Author-email: Christopher Wittlinger <c.wittlinger@stainly.com>
 Requires-Dist: wbcore

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/filters/risk_management/checks.py

@@ -12,6 +12,14 @@ class RiskCheckFilterSet(wb_filters.FilterSet):
         distinct=True,
         hidden=True,
     )
+    activators = wb_filters.MultipleChoiceContentTypeFilter(
+        label="Activators",
+        field_name="activators",
+        object_id_label="activator_id",
+        content_type_label="activator_content_type",
+        distinct=True,
+        hidden=True,
+    )
     passive_check = wb_filters.BooleanFilter(initial=True, required=True, label="Passive")
 
     class Meta:

wbcompliance-1.59.2/wbcompliance/migrations/0022_riskincident_precheck.py

@@ -0,0 +1,39 @@
+# Generated by Django 5.0.12 on 2025-11-28 09:18
+
+from django.db import migrations, models
+import django.db.models.deletion
+from django.db.models import Exists, OuterRef
+
+
+def handle_data(apps, schema_editor):
+    CheckedObjectIncidentRelationship = apps.get_model('wbcompliance', 'CheckedObjectIncidentRelationship')
+    RiskIncident = apps.get_model('wbcompliance', 'RiskIncident')
+    CheckedObjectIncidentRelationship.objects.filter(incident__isnull=True).delete()
+    RiskIncident.objects.filter(checked_object_relationships__isnull=True).delete()
+    RiskIncident.objects.annotate(
+        is_precheck=Exists(CheckedObjectIncidentRelationship.objects.filter(incident=OuterRef("id"), rule_check__activator_id__isnull=False))
+    ).filter(is_precheck=True).update(precheck=True)
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('wbcompliance', '0021_riskcheck_passive_check'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='riskincident',
+            name='precheck',
+            field=models.BooleanField(default=False, help_text='If true, this incident was created during a precheck evaluation', verbose_name='Precheck Incident'),
+        ),
+        migrations.RunSQL(sql="SET CONSTRAINTS ALL IMMEDIATE;"),
+        migrations.RunPython(handle_data),
+        migrations.RunSQL(sql="SET CONSTRAINTS ALL DEFERRED;"),
+        migrations.AlterField(
+            model_name='checkedobjectincidentrelationship',
+            name='incident',
+            field=models.ForeignKey(default=None, on_delete=django.db.models.deletion.CASCADE,
+                                    related_name='checked_object_relationships', to='wbcompliance.riskincident'),
+            preserve_default=False,
+        ),
+    ]

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/compliance_task.py

@@ -75,7 +75,7 @@ class ComplianceTask(WBModel):
         @classmethod
         def get_color_map(cls) -> list:
             colors = [WBColor.GREEN_LIGHT.value, WBColor.YELLOW_LIGHT.value, WBColor.RED_LIGHT.value]
-            return [choice for choice in zip(cls, colors)]
+            return [choice for choice in zip(cls, colors, strict=False)]
 
     title = models.CharField(max_length=255, verbose_name=_("Title"))
     description = models.TextField(default="", blank=True, verbose_name=_("Description"))
@@ -171,7 +171,7 @@ class ComplianceTaskInstance(models.Model):
                 WBColor.BLUE_LIGHT.value,
                 WBColor.RED_LIGHT.value,
             ]
-            return [choice for choice in zip(cls, colors)]
+            return [choice for choice in zip(cls, colors, strict=False)]
 
     task = models.ForeignKey(
         on_delete=models.CASCADE,
@@ -235,7 +235,7 @@ class ComplianceAction(models.Model):
         @classmethod
         def get_color_map(cls) -> list:
             colors = [WBColor.GREY.value, WBColor.YELLOW_LIGHT.value, WBColor.GREEN_LIGHT.value]
-            return [choice for choice in zip(cls, colors)]
+            return [choice for choice in zip(cls, colors, strict=False)]
 
     title = models.CharField(max_length=255, verbose_name=_("Title"))
     description = models.TextField(
@@ -310,7 +310,7 @@ class ComplianceEvent(models.Model):
         @classmethod
         def get_color_map(cls) -> list:
             colors = [WBColor.GREY.value, WBColor.BLUE_LIGHT.value]
-            return [choice for choice in zip(cls, colors)]
+            return [choice for choice in zip(cls, colors, strict=False)]
 
     type_event = models.CharField(
         max_length=32,
@@ -414,7 +414,7 @@ class ReviewComplianceTask(ComplianceDocumentMixin, ComplexToStringMixin, WBMode
                 WBColor.YELLOW_LIGHT.value,
                 WBColor.GREEN_LIGHT.value,
             ]
-            return [choice for choice in zip(cls, colors)]
+            return [choice for choice in zip(cls, colors, strict=False)]
 
     class Meta:
         verbose_name = "Review Compliance Task"

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/enums.py

@@ -10,4 +10,4 @@ class IncidentSeverity(models.TextChoices):
     @classmethod
     def get_color_map(cls) -> list:
         colors = [WBColor.GREEN_LIGHT.value, WBColor.YELLOW_LIGHT.value, WBColor.RED_LIGHT.value]
-        return [choice for choice in zip(cls, colors)]
+        return [choice for choice in zip(cls, colors, strict=False)]

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/risk_management/checks.py

@@ -1,6 +1,7 @@
 from contextlib import suppress
+from datetime import date, timedelta
 from types import DynamicClassAttribute
-from typing import Self
+from typing import Any, Self
 
 from celery import shared_task
 from django.contrib.contenttypes.fields import GenericForeignKey
@@ -8,12 +9,14 @@ from django.contrib.contenttypes.models import ContentType
 from django.db import models
 from django.template import Context, Template
 from django.utils.translation import gettext_lazy as _
+from pandas._libs.tslibs.offsets import BDay
+from psycopg.types.range import DateRange
 from wbcore.contrib.color.enums import WBColor
 from wbcore.contrib.icons import WBIcon
 from wbcore.models import WBModel
 from wbcore.utils.models import ComplexToStringMixin
 
-from .incidents import CheckedObjectIncidentRelationship
+from .incidents import CheckedObjectIncidentRelationship, RiskIncident, RiskIncidentType
 
 
 class RiskCheckManager(models.Manager):
@@ -99,9 +102,16 @@ class RiskCheck(ComplexToStringMixin, WBModel):
     @property
     def needs_incident(self) -> bool:
         # if the there is a similar checked object relationship, we needs to create a general incident if the check evaluates to a breach
-        return self.rule.checked_object_relationships.filter(
-            checked_object_content_type=self.checked_object_content_type, checked_object_id=self.checked_object_id
-        ).exists()
+        return (
+            self.rule.checked_object_relationships.filter(
+                checked_object_content_type=self.checked_object_content_type, checked_object_id=self.checked_object_id
+            ).exists()
+            or self.precheck
+        )
+
+    @property
+    def precheck(self) -> bool:
+        return self.activator_id is not None
 
     @property
     def previous_check(self) -> Self | None:
@@ -127,6 +137,20 @@ class RiskCheck(ComplexToStringMixin, WBModel):
         self.checked_object_repr = str(self.checked_object)
         super().save(*args, **kwargs)
 
+    def delete(self, *args, **kwargs):
+        incidents = RiskIncident.all_objects.filter(
+            id__in=list(
+                CheckedObjectIncidentRelationship.objects.filter(rule_check_id=self.id).values_list(
+                    "incident", flat=True
+                )
+            )
+        )
+        super().delete(*args, **kwargs)
+        # ensure that incident without subincidents are deleted automatically
+        for incident in incidents:
+            if not incident.checked_object_relationships.exists():
+                incident.delete()
+
     def compute_str(self) -> str:
         return _("{} - {}").format(
             self.checked_object,
@@ -164,7 +188,7 @@ class RiskCheck(ComplexToStringMixin, WBModel):
                 # If the check is passive, we aggregated incident per breached object and return it for further processing
             if self.needs_incident:
                 report = report_template.render(Context({"report_details": incident_result.report_details}))
-                incident, created = self.rule.get_or_create_incident(
+                incident, created = self.get_or_create_incident(
                     self.evaluation_date,
                     incident_result.severity,
                     incident_result.breached_object,
@@ -192,6 +216,68 @@ class RiskCheck(ComplexToStringMixin, WBModel):
         self.save()
         return incidents
 
+    def get_or_create_incident(
+        self,
+        evaluation_date: date,
+        incident_severity: "RiskIncidentType",
+        breached_object: Any,
+        breached_object_repr: str,
+    ) -> tuple[RiskIncident, bool]:
+        """
+        Utility function to get or create incident base on the given breached object
+
+        Args:
+            evaluation_date: The incident date
+            incident_severity: The incident severity
+            breached_object: The breached object (i.e. the object that triggers the incident)
+            breached_object_repr: Its string representation
+
+        Returns:
+            A tuple (incident, is_created)
+        """
+
+        # Consider that if an incident happens one business day in the future or past, it is continue
+        date_range = DateRange(  # type: ignore
+            (evaluation_date - BDay(1)).date(), (evaluation_date + BDay(1)).date(), "[]"
+        )
+        incidents = RiskIncident.objects.filter(rule=self.rule, date_range__overlap=date_range, precheck=self.precheck)
+        if breached_object:
+            # If a breached_object is provided, we lookup over it
+            incidents = incidents.filter(
+                breached_content_type=ContentType.objects.get_for_model(breached_object),
+                breached_object_id=breached_object.id,
+            )
+        else:
+            # Otherwise, we lookup over its string representation. Might need to change as this is not very robust
+            incidents = incidents.filter(
+                breached_object_repr=breached_object_repr,
+            )
+        created = False
+        if incident := incidents.first():
+            incident.date_range = DateRange(  # type: ignore
+                lower=incident.date_range.lower, upper=(evaluation_date + timedelta(days=1))
+            )
+            incident.severity = incident_severity
+        else:
+            incident = RiskIncident(
+                rule=self.rule,
+                date_range=DateRange(lower=evaluation_date, upper=(evaluation_date + timedelta(days=1))),  # type: ignore
+                breached_content_type=ContentType.objects.get_for_model(breached_object) if breached_object else None,
+                breached_object_id=breached_object.id if breached_object else None,
+                breached_object_repr=breached_object_repr,
+                severity=incident_severity,
+                status=RiskIncident.Status.OPEN,
+            )
+            created = True
+
+        if self.rule.automatically_close_incident:
+            incident.status = RiskIncident.Status.CLOSED
+        elif self.precheck:
+            incident.status = RiskIncident.Status.IGNORED
+        incident.precheck = self.precheck
+        incident.save()
+        return incident, created
+
     class Meta:
         verbose_name = "Risk Check"
         verbose_name_plural = "Risk Checks"

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/risk_management/incidents.py

@@ -29,7 +29,6 @@ from wbcore.metadata.configs.buttons import ActionButton, ButtonDefaultColor
 from wbcore.metadata.configs.display.instance_display.shortcuts import (
     create_simple_display,
 )
-from wbcore.models import WBModel
 from wbcore.utils.models import ComplexToStringMixin
 
 User: BaseUser = get_user_model()
@@ -109,13 +108,11 @@ class RiskIncidentMixin(models.Model):
         abstract = True
 
 
-class CheckedObjectIncidentRelationship(ComplexToStringMixin, RiskIncidentMixin, WBModel):
+class CheckedObjectIncidentRelationship(ComplexToStringMixin, RiskIncidentMixin):
     incident = models.ForeignKey(
         to="wbcompliance.RiskIncident",
         related_name="checked_object_relationships",
-        null=True,
-        blank=True,
-        on_delete=models.SET_NULL,
+        on_delete=models.CASCADE,
     )
 
     rule_check = models.ForeignKey(
@@ -229,6 +226,9 @@ class CheckedObjectIncidentRelationship(ComplexToStringMixin, RiskIncidentMixin,
             models.Index(fields=["incident", "rule_check"]),
         ]
 
+    def __str__(self) -> str:
+        return super().__str__()
+
     def compute_str(self) -> str:
         return _("{} {} Sub Incident for checked_object {}").format(self.status, self.severity, self.rule_check)
 
@@ -246,8 +246,12 @@ class CheckedObjectIncidentRelationship(ComplexToStringMixin, RiskIncidentMixin,
 
 
 class RiskIncidentDefaultManager(models.Manager):
+    def __init__(self, only_passive_checks: bool = True):
+        self.only_passive_checks = only_passive_checks
+        super().__init__()
+
     def get_queryset(self):
-        return (
+        qs = (
             super()
             .get_queryset()
             .annotate(
@@ -257,9 +261,12 @@ class RiskIncidentDefaultManager(models.Manager):
                 ignore_until=TruncDate("ignore_until_time"),
             )
         )
+        if self.only_passive_checks:
+            qs = qs.filter(precheck=False)
+        return qs
 
 
-class RiskIncident(ComplexToStringMixin, RiskIncidentMixin, WBModel):
+class RiskIncident(ComplexToStringMixin, RiskIncidentMixin):
     """
     Instance defining the incident that has happened during a check initiated by a certain rule
     """
@@ -295,6 +302,11 @@ class RiskIncident(ComplexToStringMixin, RiskIncidentMixin, WBModel):
     ignore_duration = models.DurationField(
         blank=True, null=True, help_text=_("If set, will ignore the forthcoming incidents for the specified duration")
     )
+    precheck = models.BooleanField(
+        default=False,
+        verbose_name=_("Precheck Incident"),
+        help_text="If true, this incident was created during a precheck evaluation",
+    )
 
     def get_ignore_until_date(self) -> date | None:
         if self.last_ignored_date and self.ignore_duration:
@@ -365,6 +377,7 @@ class RiskIncident(ComplexToStringMixin, RiskIncidentMixin, WBModel):
         return dict()
 
     objects = RiskIncidentDefaultManager()
+    all_objects = RiskIncidentDefaultManager(only_passive_checks=False)
 
     class Meta:
         verbose_name = "Risk Incident"
@@ -382,6 +395,9 @@ class RiskIncident(ComplexToStringMixin, RiskIncidentMixin, WBModel):
             )
         ]
 
+    def __str__(self) -> str:
+        return super().__str__()
+
     def save(self, *args, **kwargs):
         if not self.breached_object_repr:
             self.breached_object_repr = str(self.breached_content_object)

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/risk_management/mixins.py

@@ -97,8 +97,8 @@ class RiskCheckMixin(models.Model):
             )[0]
             if asynchronously:
                 transaction.on_commit(
-                    lambda: evaluate_as_task.delay(
-                        check.id, *dto, override_incident=True, ignore_informational_threshold=True
+                    lambda check_id=check.id: evaluate_as_task.delay(
+                        check_id, *dto, override_incident=True, ignore_informational_threshold=True
                     )
                 )
             else:

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/models/risk_management/rules.py

@@ -22,7 +22,6 @@ from django.template.loader import get_template
 from django.utils.functional import cached_property
 from django.utils.translation import gettext_lazy as _
 from pandas.tseries.offsets import BDay
-from psycopg.types.range import DateRange
 from rest_framework.reverse import reverse
 from wbcore.contrib.directory.models import Person
 from wbcore.contrib.guardian.models.mixins import PermissionObjectModelMixin
@@ -33,7 +32,7 @@ from wbcore.utils.rrules import convert_rrulestr_to_dict
 
 from .backend import AbstractRuleBackend
 from .checks import RiskCheck
-from .incidents import CheckedObjectIncidentRelationship, RiskIncident, RiskIncidentType
+from .incidents import CheckedObjectIncidentRelationship, RiskIncident
 
 User: BaseUser = get_user_model()
 
@@ -47,6 +46,9 @@ class RuleGroup(models.Model):
             self.name = self.key.title()
         super().save(*args, **kwargs)
 
+    def __str__(self) -> str:
+        return self.name
+
     @classmethod
     def get_representation_endpoint(cls) -> str:
         return "wbcompliance:rulegrouprepresentation-list"
@@ -60,7 +62,7 @@ class RuleGroup(models.Model):
         return "{{name}}"
 
 
-class RuleCheckedObjectRelationship(ComplexToStringMixin, models.Model):
+class RuleCheckedObjectRelationship(ComplexToStringMixin):
     rule = models.ForeignKey(
         to="wbcompliance.RiskRule", related_name="checked_object_relationships", on_delete=models.CASCADE
     )
@@ -261,7 +263,7 @@ class RuleBackend(models.Model):
         return "{{name}}"
 
 
-class RuleThreshold(ComplexToStringMixin, models.Model):
+class RuleThreshold(ComplexToStringMixin):
     """
     Represent the list of threshold and its associated severity link to a certain rule
     """
@@ -447,7 +449,7 @@ class RiskRule(PermissionObjectModelMixin, WBModel):
     def checked_object_representation(self) -> str:
         try:
             backend_class = self.rule_backend.backend_class
-            checked_object_repr = getattr(backend_class, "OBJECT_FIELD_NAME").title()
+            checked_object_repr = backend_class.OBJECT_FIELD_NAME.title()
         except AttributeError:
             checked_object_repr = "Object"
         return checked_object_repr
@@ -508,69 +510,6 @@ class RiskRule(PermissionObjectModelMixin, WBModel):
 
         return permissions
 
-    def get_or_create_incident(
-        self,
-        evaluation_date: date,
-        incident_severity: "RiskIncidentType",
-        breached_object: Any,
-        breached_object_repr: str,
-    ) -> tuple[RiskIncident, bool]:
-        """
-        Utility function to get or create incident base on the given breached object
-
-        Args:
-            rule:  The rule this incident happens
-            evaluation_date: The incident date
-            incident_severity: The incident severity
-            breached_object: The breached object (i.e. the object that triggers the incident)
-            breached_object_repr: Its string representation
-
-        Returns:
-            A tuple (incident, is_created)
-        """
-
-        # Consider that if an incident happens one business day in the future or past, it is continue
-        date_range = DateRange(  # type: ignore
-            (evaluation_date - BDay(1)).date(), (evaluation_date + BDay(1)).date(), "[]"
-        )
-        incidents = RiskIncident.objects.filter(rule=self, date_range__overlap=date_range)
-        if breached_object:
-            # If a breached_object is provided, we lookup over it
-            incidents = incidents.filter(
-                breached_content_type=ContentType.objects.get_for_model(breached_object),
-                breached_object_id=breached_object.id,
-            )
-        else:
-            # Otherwise, we lookup over its string representation. Might need to change as this is not very robust
-            incidents = incidents.filter(
-                breached_object_repr=breached_object_repr,
-            )
-        if incident := incidents.first():
-            incident.date_range = DateRange(
-                lower=incident.date_range.lower, upper=(evaluation_date + timedelta(days=1))
-            )  # type: ignore
-            if self.automatically_close_incident:
-                incident.status = RiskIncident.Status.CLOSED
-            incident.save()
-            return incident, False
-        else:
-            return (
-                RiskIncident.objects.create(
-                    rule=self,
-                    date_range=DateRange(lower=evaluation_date, upper=(evaluation_date + timedelta(days=1))),  # type: ignore
-                    breached_content_type=ContentType.objects.get_for_model(breached_object)
-                    if breached_object
-                    else None,
-                    breached_object_id=breached_object.id if breached_object else None,
-                    breached_object_repr=breached_object_repr,
-                    severity=incident_severity,
-                    status=RiskIncident.Status.CLOSED
-                    if self.automatically_close_incident
-                    else RiskIncident.Status.OPEN,
-                ),
-                True,
-            )
-
     def notify(self, evaluation_date: date):
         """
         Create the notification for this rule relationship and all implied persons
@@ -659,7 +598,7 @@ class RiskRule(PermissionObjectModelMixin, WBModel):
 def pre_save_risk_threshold(sender, instance, **kwargs):
     with suppress(RuleThreshold.DoesNotExist):
         pre_save_instance = RuleThreshold.objects.get(id=instance.id)
-        RiskIncident.objects.filter(rule=instance.rule, severity=pre_save_instance.severity).update(
+        RiskIncident.all_objects.filter(rule=instance.rule, severity=pre_save_instance.severity).update(
             severity=instance.severity
         )
         CheckedObjectIncidentRelationship.objects.filter(
@@ -669,7 +608,7 @@ def pre_save_risk_threshold(sender, instance, **kwargs):
 
 @receiver(pre_delete, sender="wbcompliance.RuleThreshold")
 def pre_delete_risk_threshold(sender, instance, **kwargs):
-    RiskIncident.objects.filter(rule=instance.rule, severity=instance.severity).delete()
+    RiskIncident.all_objects.filter(rule=instance.rule, severity=instance.severity).delete()
     CheckedObjectIncidentRelationship.objects.filter(incident__rule=instance.rule, severity=instance.severity).delete()
 
 

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/serializers/risk_management/checks.py

@@ -17,7 +17,7 @@ class RiskCheckModelSerializer(wb_serializers.ModelSerializer):
     _rule = RiskRuleRepresentationSerializer(source="rule")
     _checked_object_content_type = ContentTypeRepresentationSerializer(source="checked_object_content_type")
 
-    incident_reports = wb_serializers.TextField()
+    incident_reports = wb_serializers.TextField(read_only=True)
     status_icon = IconSelectField(read_only=True)
 
     @wb_serializers.register_resource()

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/tests/disable_signals.py

@@ -1,6 +1,15 @@
 from collections import defaultdict
 
-from django.db.models.signals import *
+from django.db.models.signals import (
+    post_delete,
+    post_init,
+    post_migrate,
+    post_save,
+    pre_delete,
+    pre_init,
+    pre_migrate,
+    pre_save,
+)
 
 
 class DisableSignals(object):
@@ -50,7 +59,7 @@ class DisableSignals(object):
 #                     self.disconnect(signal)
 
 
-class temp_disconnect_signal:
+class TempDisconnectSignal:
     """Temporarily disconnect a model from a signal"""
 
     def __init__(self, signal, receiver, sender, dispatch_uid=None):
@@ -72,11 +81,3 @@ class temp_disconnect_signal:
             sender=self.sender,
             dispatch_uid=self.dispatch_uid,
         )
-
-
-class DisableSignalsNotification(temp_disconnect_signal):
-    def __init__(self, dispatch_uid=None):
-        self.signal = post_save
-        self.receiver = post_create_notification
-        self.sender = Notification
-        self.dispatch_uid = dispatch_uid

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/tests/risk_management/models/test_incidents.py

@@ -85,7 +85,7 @@ class TestRiskIncidentFactory:
             (fake.date_object(), False),
         ],
     )
-    def test_get_or_create_incident(self, risk_rule, risk_incident_type, evaluation_date, is_breached_object):
+    def test_get_or_create_incident(self, risk_check, risk_incident_type, evaluation_date, is_breached_object):
         if is_breached_object:
             breached_object = get_anonymous_user()
             breached_object_repr = str(get_anonymous_user())
@@ -94,11 +94,11 @@ class TestRiskIncidentFactory:
             breached_object_repr = fake.text(max_nb_chars=125)
 
         assert not RiskIncident.objects.exists()
-        risk_rule.get_or_create_incident(evaluation_date, risk_incident_type, breached_object, breached_object_repr)
+        risk_check.get_or_create_incident(evaluation_date, risk_incident_type, breached_object, breached_object_repr)
         if breached_object:
             assert (
                 RiskIncident.objects.filter(
-                    rule=risk_rule,
+                    rule=risk_check.rule,
                     date_range__contains=evaluation_date,
                     breached_content_type=ContentType.objects.get_for_model(breached_object),
                     breached_object_id=breached_object.id,
@@ -108,19 +108,21 @@ class TestRiskIncidentFactory:
         else:
             assert (
                 RiskIncident.objects.filter(
-                    rule=risk_rule, date_range__contains=evaluation_date, breached_object_repr=breached_object_repr
+                    rule=risk_check.rule,
+                    date_range__contains=evaluation_date,
+                    breached_object_repr=breached_object_repr,
                 ).count()
                 == 1
             )
         assert RiskIncident.objects.count() == 1
         # Check if we reevaluate this incident one business day later, the incident is consider the same
-        risk_rule.get_or_create_incident(
+        risk_check.get_or_create_incident(
             (evaluation_date + BDay(1)).date(), risk_incident_type, breached_object, breached_object_repr
         )
         assert RiskIncident.objects.count() == 1
 
         # Check if we reevaluate this incident three business day later, the incident is consider the discontinue, and then a new incident is created
-        risk_rule.get_or_create_incident(
+        risk_check.get_or_create_incident(
             (evaluation_date + BDay(3)).date(), risk_incident_type, breached_object, breached_object_repr
         )
 
@@ -135,7 +137,7 @@ class TestRiskIncidentFactory:
         risk_incident.save()
         assert risk_incident.status == RiskIncident.Status.RESOLVED
         new_evaluation_date = subincident.rule_check.evaluation_date + timedelta(days=1)
-        incident, created = risk_incident.rule.get_or_create_incident(
+        incident, created = subincident.rule_check.get_or_create_incident(
             new_evaluation_date,
             subincident.severity,
             risk_incident.breached_content_object,

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/tests/test_views.py

@@ -342,7 +342,7 @@ class TestSpecificViewsets(UserTestMixin):
         assert response.data.get("results")
 
     @pytest.mark.parametrize("mvs", [ComplianceTaskMatrixPandasViewSet])
-    def test_PandasAPIView(self, mvs, compliance_task_factory, compliance_task_instance_factory):
+    def test_pandasapiview(self, mvs, compliance_task_factory, compliance_task_instance_factory):
         request = APIRequestFactory().get("")
         request.user = get_or_create_superuser()
 

{wbcompliance-1.54.19 → wbcompliance-1.59.2}/wbcompliance/viewsets/buttons/compliance_form.py

@@ -20,9 +20,9 @@ class AbstractComplianceDocumentButtonConfig:
         return bt.ActionButton(
             method=RequestType.PATCH,
             key="regenerate_document",
-            action_label=_("Regenerate Document"),
-            title=_("Regenerate Document"),
-            label=_("Regenerate Document"),
+            action_label=_("PDF Document is being generated"),
+            title=_("Generate PDF Document"),
+            label=_("Generate PDF Document"),
             icon=WBIcon.DOCUMENT.icon,
             serializer=ComplianceDocumentSerializer,
             instance_display=create_simple_display([["send_email"]]),