correct count on hard sequential triggers needed, to match spec

uwcirg · Mar 28, 2024 · ee5503f · ee5503f
1 parent 0946f74
commit ee5503f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/portal/trigger_states/models.py b/portal/trigger_states/models.py
@@ -162,7 +162,7 @@ def opted_out_domains(self):
     def sequential_threshold_reached(self):
         """Returns True IFF at least one domain above sequential threshold
 
-        Business rule says when >= 2 sequential hard triggers for a given
+        Business rule says when >= 3 sequential hard triggers for a given
         domain are achieved, the user gets the option to opt-out of follow
         up from clinician.  This convenience method looks through all domains
         in triggers to see if at least one has adequate sequential count of
@@ -175,7 +175,7 @@ def sequential_threshold_reached(self):
             return
 
         for domain, link_triggers in self.triggers['domain'].items():
-            if link_triggers.get(sequential_hard_trigger_count_key, 0) > 1:
+            if link_triggers.get(sequential_hard_trigger_count_key, 0) > 2:
                 return True
 
     def reminder_due(self, as_of_date=None):

diff --git a/tests/fixtures/trigger_state.py b/tests/fixtures/trigger_state.py
@@ -14,7 +14,7 @@ def mock_triggers():
             'joint_pain': {
                 'ironman_ss.4': 'hard', 'ironman_ss.6': 'soft'},
             'insomnia': {
-                '_sequential_hard_trigger_count': 2,
+                '_sequential_hard_trigger_count': 3,
                 '_opt_out_this_visit': True},
             'fatigue': {'ironman_ss.9': 'hard'},
             'anxious': {'ironman_ss.12': 'soft'},