[Checkins] SVN: Products.CMFCore/trunk/Products/CMFCore/tests/test_DirectoryView.py Assert is a statement, not a function. Removed parens in the extended form that caused test to always pass.

Matthew Wilkes matthew at matthewwilkes.co.uk
Tue Oct 28 18:38:50 EDT 2008


Log message for revision 92662:
  Assert is a statement, not a function.  Removed parens in the extended form that caused test to always pass.

Changed:
  U   Products.CMFCore/trunk/Products/CMFCore/tests/test_DirectoryView.py

-=-
Modified: Products.CMFCore/trunk/Products/CMFCore/tests/test_DirectoryView.py
===================================================================
--- Products.CMFCore/trunk/Products/CMFCore/tests/test_DirectoryView.py	2008-10-28 21:33:22 UTC (rev 92661)
+++ Products.CMFCore/trunk/Products/CMFCore/tests/test_DirectoryView.py	2008-10-28 22:38:49 UTC (rev 92662)
@@ -196,8 +196,8 @@
     def test_ignored(self):
         # Test that "artifact" files and dirs are ignored
         for name in '#test1', 'CVS', '.test1', 'test1~':
-            assert(name not in self.ob.fake_skin.objectIds(),
-                   '%s not ignored' % name)
+            assert name not in self.ob.fake_skin.objectIds(), \
+                   '%s not ignored' % name
 
     def test_surrogate_writethrough(self):
         # CMF Collector 316: It is possible to cause ZODB writes because



More information about the Checkins mailing list