[Fix] Fix group based composites

diff --git a/src/libserver/composites.c b/src/libserver/composites.c
index 7dff938022cc88056418a074a8ee24724e5be005..c1ee471f8b9dfafd53228a8b6282805a27207e83 100644 (file)
--- a/src/libserver/composites.c
+++ b/src/libserver/composites.c
@@ -394,7 +394,7 @@ rspamd_composite_expr_process (void *ud,
        struct rspamd_task *task = cd->task;
        GHashTableIter it;
        gpointer k, v;
-       gdouble rc = epsilon, max = epsilon;
+       gdouble rc = 0, max = 0;
 
        if (isset (cd->checked, cd->composite->id * 2)) {
                /* We have already checked this composite, so just return its value */

diff --git a/test/functional/cases/109_composites.robot b/test/functional/cases/109_composites.robot
index 47fc0c28efb506b1757e2d5197abca28c46ccc78..8da18fe977bb88244d1534e6c44ddd48f7cbe483 100644 (file)
--- a/test/functional/cases/109_composites.robot
+++ b/test/functional/cases/109_composites.robot
@@ -12,59 +12,35 @@ ${MESSAGE}      ${TESTDIR}/messages/spam_message.eml
 ${RSPAMD_SCOPE}  Suite
 
 *** Test Cases ***
-Composites - Score
-  Scan File  ${MESSAGE}
-  Expect Score  50
-  Expect Required Score To Be Null
-
-Composites - Expressions
+Composites - All in One
   Scan File  ${MESSAGE}
   Expect Symbol With Score  EXPRESSIONS  5
   Expect Symbol With Score  EXPRESSIONS_B  0
-
-Composites - Policy: remove_weight
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  POLICY_REMOVE_WEIGHT  5
   Expect Symbol With Score  POLICY_REMOVE_WEIGHT_B  0
   Do Not Expect Symbol  POLICY_REMOVE_WEIGHT_A
-
-Composites - Policy: force removing
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  POLICY_FORCE_REMOVE  5.00
   Expect Symbol With Score  POLICY_FORCE_REMOVE_A  1.00
   Do Not Expect Symbol  POLICY_FORCE_REMOVE_B
-
-Composites - Policy: leave
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  POLICY_LEAVE  5.00
   Do Not Expect Symbol  POLICY_LEAVE_A
   Expect Symbol With Score  POLICY_LEAVE_B  1.00
-
-Composites - Default policy: remove_weight
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  DEFAULT_POLICY_REMOVE_WEIGHT  5.00
   Expect Symbol With Score  DEFAULT_POLICY_REMOVE_WEIGHT_A  0.00
   Expect Symbol With Score  DEFAULT_POLICY_REMOVE_WEIGHT_B  0.00
-
-Composites - Default policy: remove_symbol
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  DEFAULT_POLICY_REMOVE_SYMBOL  5.00
   Do Not Expect Symbol  DEFAULT_POLICY_REMOVE_SYMBOL_A
   Do Not Expect Symbol  DEFAULT_POLICY_REMOVE_SYMBOL_B
-
-Composites - Default policy: leave
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  DEFAULT_POLICY_LEAVE  5.00
   Expect Symbol With Score  DEFAULT_POLICY_LEAVE_A  1.00
   Expect Symbol With Score  DEFAULT_POLICY_LEAVE_B  1.00
-
-Composites - Symbol groups
-  Scan File  ${MESSAGE}
   Expect Symbol With Score  SYMBOL_GROUPS  5.00
   Expect Symbol With Score  POSITIVE_A  -1.00
   Expect Symbol With Score  ANY_A  -1.00
   Expect Symbol With Score  NEGATIVE_B  1.00
   Do Not Expect Symbol  NEGATIVE_A
+  Expect Score  50
+  Expect Required Score To Be Null
 
 Composites - Opts Plain
   Scan File  ${MESSAGE}  opts=sym1