Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: fzipi

tests/conftest.py

@@ -278,7 +278,7 @@ def test_template():
               User-Agent: OWASP CRS
             method: {{ method }}
             port: 80
-            uri: "{{ uri }}
+            uri: "{{ uri }}"
           output:
             log_contains: id "{{ id }}"
 """

tests/test_actions.py

@@ -1105,17 +1105,17 @@ def test_action_ctl_force_request_body_variable() -> None:
 def test_multiple_actions_combined() -> None:
     """Test rule with multiple different actions"""
     rule_text = """
-    SecRule ARGS "@rx attack" \
-        "id:999001,\
-        phase:2,\
-        deny,\
-        status:403,\
-        log,\
-        msg:'SQL Injection Attempt',\
-        severity:CRITICAL,\
-        tag:'attack-sqli',\
-        setvar:'tx.anomaly_score=+5',\
-        t:lowercase,\
+    SecRule ARGS "@rx attack" \\
+        "id:999001,\\
+        phase:2,\\
+        deny,\\
+        status:403,\\
+        log,\\
+        msg:'SQL Injection Attempt',\\
+        severity:CRITICAL,\\
+        tag:'attack-sqli',\\
+        setvar:'tx.anomaly_score=+5',\\
+        t:lowercase,\\
         t:urlDecode"
     """
     parsed_rule = parser.process_from_str(rule_text)

tests/test_conftest_examples.py

@@ -5,18 +5,16 @@
 These examples show how the shared fixtures and helper functions from conftest.py
 can simplify test code and reduce duplication.
 
-Note: The helper functions from conftest.py are automatically available in the test
-namespace when using pytest. You can also import them explicitly using:
+Note: Fixtures defined in conftest.py are automatically available in the test
+namespace when using pytest. Plain helper functions must be imported explicitly,
+for example:
     from tests.conftest import parse_rule, assert_parse_success, etc.
 """
 import pytest
 
 # Import helper functions from conftest.py
-# These are automatically available via pytest's conftest mechanism
-import sys
-import os
-sys.path.insert(0, os.path.dirname(__file__))
-from conftest import (
+# Fixtures are auto-discovered by pytest; helper functions are imported explicitly
+from tests.conftest import (
     parse_rule,
     assert_parse_success,
     get_rules_by_type,

tests/test_operators.py

@@ -83,7 +83,7 @@ def test_operator_contains_word() -> None:
     for rule in parsed_rule.rules:
         assert rule.__class__.__name__ == "SecRule"
         # containsWord is parsed but STRING may be empty - just verify it exists
-        if hasattr(rule.operator, 'containsWord'):
+        if getattr(rule.operator, "containsWord", None) is not None:
             matches += 1
     assert matches == 1