Author: PaulM
Date: 2010-07-06 18:27:34 -0500 (Tue, 06 Jul 2010)
New Revision: 13427

Modified:
   django/branches/soc2010/test-refactor/django/test/utils.py
Log:
[soc2010/test-refactor] Fixed @skipIfDBEngine to be more explicit


Modified: django/branches/soc2010/test-refactor/django/test/utils.py
===================================================================
--- django/branches/soc2010/test-refactor/django/test/utils.py  2010-07-05 
17:22:29 UTC (rev 13426)
+++ django/branches/soc2010/test-refactor/django/test/utils.py  2010-07-06 
23:27:34 UTC (rev 13427)
@@ -88,5 +88,7 @@
     """
     if not reason:
         reason = "not supported on this database"
-    return skipIf(settings.DATABASES[DEFAULT_DB_ALIAS]['ENGINE'] in engine,
-                  reason)
+    settings_engine = settings.DATABASES[DEFAULT_DB_ALIAS]['ENGINE']
+    if isinstance(engine, basestring):
+        return skipIf(settings_engine == engine, reason)
+    return skipIf(settings_engine in engine, reason)

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-upda...@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to