test(nodes): remove wall-clock thresholds from template engine tests

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 2 people

nodes/test/test_prompt_template_engine.py

@@ -337,16 +337,14 @@ def test_html_entity_in_template_is_literal(self):
 # ===========================================================================
 class TestSafety:
     def test_large_literal_input_runs_quickly(self):
-        # ~50KB of plain text with no placeholders — tokenizer regex must not
-        # blow up (ReDoS guard). Timing check is intentionally loose.
-        import time
-
+        # ~100KB with one placeholder: verifies the tokenizer regex handles
+        # large literal runs correctly and does not catastrophically backtrack
+        # (ReDoS guard). A regex blow-up would manifest as a multi-minute hang
+        # that pytest would terminate — no wall-clock threshold is needed here,
+        # and asserting one only introduces CI flakiness on slow/busy runners.
         template = 'x' * 50_000 + '{{v}}' + 'y' * 50_000
-        start = time.perf_counter()
         out = render(template, {'v': 'MID'})
-        elapsed = time.perf_counter() - start
-        assert 'MID' in out
-        assert elapsed < 2.0  # generous upper bound
+        assert out == ('x' * 50_000 + 'MID' + 'y' * 50_000)
 
     def test_many_placeholders(self):
         tpl = ''.join(f'{{{{v{i}}}}}' for i in range(500))