[Checkins] SVN: Products.CMFTopic/trunk/Products/CMFTopic/ Correct test assertion about the value of DateCriteria. It's edit method mandates the value to be of type int, so we should respect that in tests.

Hanno Schlichting hannosch at hannosch.eu
Fri May 6 14:59:12 EDT 2011


Log message for revision 121571:
  Correct test assertion about the value of DateCriteria. It's edit method mandates the value to be of type int, so we should respect that in tests.
  

Changed:
  U   Products.CMFTopic/trunk/Products/CMFTopic/CHANGES.txt
  U   Products.CMFTopic/trunk/Products/CMFTopic/tests/test_exportimport.py

-=-
Modified: Products.CMFTopic/trunk/Products/CMFTopic/CHANGES.txt
===================================================================
--- Products.CMFTopic/trunk/Products/CMFTopic/CHANGES.txt	2011-05-06 18:44:41 UTC (rev 121570)
+++ Products.CMFTopic/trunk/Products/CMFTopic/CHANGES.txt	2011-05-06 18:59:12 UTC (rev 121571)
@@ -4,6 +4,9 @@
 2.3.0-alpha (unreleased)
 ------------------------
 
+- Correct test assertion about the value of DateCriteria. It's edit method
+  mandates the value to be of type int, so we should respect that in tests.
+
 - Require at least Zope 2.13.4.
 
 - Deal with deprecation warnings for Zope 2.13.

Modified: Products.CMFTopic/trunk/Products/CMFTopic/tests/test_exportimport.py
===================================================================
--- Products.CMFTopic/trunk/Products/CMFTopic/tests/test_exportimport.py	2011-05-06 18:44:41 UTC (rev 121570)
+++ Products.CMFTopic/trunk/Products/CMFTopic/tests/test_exportimport.py	2011-05-06 18:59:12 UTC (rev 121571)
@@ -30,7 +30,7 @@
 _CRITERIA_DATA = (
     ('a', 'String Criterion', {'value': 'A'}),
     ('b', 'Integer Criterion', {'value': 3, 'direction': 'min'}),
-    ('c', 'Friendly Date Criterion', {'value': DateTime(_DATE_STR),
+    ('c', 'Friendly Date Criterion', {'value': DateTime(_DATE_STR).timeTime(),
                                       'operation': 'min',
                                       'daterange': 'old',
                                      }),



More information about the checkins mailing list