assert isn't a function, using parens will cause the two arguments to be treated as a 2-tuple, hence always true.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/zope/tal/dummyengine.py b/src/zope/tal/dummyengine.py
index 26109ad..2cb2310 100644
--- a/src/zope/tal/dummyengine.py
+++ b/src/zope/tal/dummyengine.py
@@ -85,8 +85,8 @@ class DummyEngine(object):
         return value
 
     def evaluate(self, expression):
-        assert (expression.startswith("$") and expression.endswith("$"),
-            expression)
+        assert expression.startswith("$") and expression.endswith("$"), \
+            expression
         expression = expression[1:-1]
         m = name_match(expression)
         if m:
@@ -152,8 +152,8 @@ class DummyEngine(object):
         return self.evaluate(expr)
 
     def evaluateMacro(self, macroName):
-        assert (macroName.startswith("$") and macroName.endswith("$"),
-            macroName)
+        assert macroName.startswith("$") and macroName.endswith("$"), \
+            macroName
         macroName = macroName[1:-1]
         file, localName = self.findMacroFile(macroName)
         if not file: