tests: internal: conditionals: NULL record should fail condition

Signed-off-by: Eduardo Silva <[email protected]>

Author: edsiper

tests/internal/conditionals.c

@@ -364,7 +364,7 @@ void test_condition_equals()
     cond = flb_condition_create(FLB_COND_OP_AND);
     TEST_CHECK(cond != NULL);
 
-    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_EQ, 
+    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_EQ,
                                     "error", 0, RECORD_CONTEXT_BODY) == FLB_TRUE);
 
     result = flb_condition_evaluate(cond, &record_data->chunk);
@@ -445,7 +445,7 @@ void test_condition_not_equals()
     cond = flb_condition_create(FLB_COND_OP_AND);
     TEST_CHECK(cond != NULL);
 
-    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_NEQ, 
+    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_NEQ,
                                     "error", 0, RECORD_CONTEXT_BODY) == FLB_TRUE);
 
     result = flb_condition_evaluate(cond, &record_data->chunk);
@@ -734,15 +734,38 @@ void test_condition_invalid_expressions()
     result = flb_condition_evaluate(NULL, &record_data->chunk);
     TEST_CHECK(result == FLB_TRUE);
 
-    /* Test NULL record */
+    /* Test NULL record with AND condition */
     cond = flb_condition_create(FLB_COND_OP_AND);
     TEST_CHECK(cond != NULL);
 
     TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_EQ,
                                     "error", 0, RECORD_CONTEXT_BODY) == FLB_TRUE);
 
     result = flb_condition_evaluate(cond, NULL);
-    TEST_CHECK(result == FLB_TRUE);
+    TEST_CHECK(result == FLB_FALSE);  /* NULL record should fail condition */
+
+    flb_condition_destroy(cond);
+
+    /* Test NULL record with OR condition */
+    cond = flb_condition_create(FLB_COND_OP_OR);
+    TEST_CHECK(cond != NULL);
+
+    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_EQ,
+                                    "error", 0, RECORD_CONTEXT_BODY) == FLB_TRUE);
+    TEST_CHECK(flb_condition_add_rule(cond, "$level", FLB_RULE_OP_EQ,
+                                    "warn", 0, RECORD_CONTEXT_BODY) == FLB_TRUE);
+
+    result = flb_condition_evaluate(cond, NULL);
+    TEST_CHECK(result == FLB_FALSE);  /* NULL record should fail condition */
+
+    flb_condition_destroy(cond);
+
+    /* Test NULL record with empty condition */
+    cond = flb_condition_create(FLB_COND_OP_AND);
+    TEST_CHECK(cond != NULL);
+
+    result = flb_condition_evaluate(cond, NULL);
+    TEST_CHECK(result == FLB_FALSE);  /* NULL record should fail even with empty condition */
 
     flb_condition_destroy(cond);