[CalendarServer-changes] [8790] CalendarServer/branches/users/glyph/case-insensitive-uid

source_changes at macosforge.org source_changes at macosforge.org
Wed Feb 29 10:15:58 PST 2012


Revision: 8790
          http://trac.macosforge.org/projects/calendarserver/changeset/8790
Author:   glyph at apple.com
Date:     2012-02-29 10:15:58 -0800 (Wed, 29 Feb 2012)
Log Message:
-----------
Fix a minor SQL generation bug to allow right-hand-side expressions.

Modified Paths:
--------------
    CalendarServer/branches/users/glyph/case-insensitive-uid/twext/enterprise/dal/test/test_sqlsyntax.py

Property Changed:
----------------
    CalendarServer/branches/users/glyph/case-insensitive-uid/

Modified: CalendarServer/branches/users/glyph/case-insensitive-uid/twext/enterprise/dal/test/test_sqlsyntax.py
===================================================================
--- CalendarServer/branches/users/glyph/case-insensitive-uid/twext/enterprise/dal/test/test_sqlsyntax.py	2012-02-29 18:15:55 UTC (rev 8789)
+++ CalendarServer/branches/users/glyph/case-insensitive-uid/twext/enterprise/dal/test/test_sqlsyntax.py	2012-02-29 18:15:58 UTC (rev 8790)
@@ -483,11 +483,23 @@
                          [173, 7]))
 
 
+    def test_rightHandSideExpression(self):
+        """
+        Arbitrary expressions may be used as the right-hand side of a
+        comparison operation.
+        """
+        self.assertEquals(
+            Select(From=self.schema.FOO,
+                   Where=self.schema.FOO.BAR >
+                   (self.schema.FOO.BAZ + 3)).toSQL(),
+            SQLFragment("select * from FOO where BAR > (BAZ + ?)", [3])
+        )
+
+
     def test_setSelects(self):
         """
         L{SetExpression} produces set operation on selects.
         """
-        
         # Simple UNION
         self.assertEquals(
             Select(
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20120229/01e4bfba/attachment.html>


More information about the calendarserver-changes mailing list