[Zope3-checkins] CVS: Zope3/src/zope/app/i18nfile/tests -
test_i18nfile.py:1.4 test_i18nimage.py:1.4
Stephan Richter
srichter at cosmos.phy.tufts.edu
Thu Mar 18 22:17:43 EST 2004
Update of /cvs-repository/Zope3/src/zope/app/i18nfile/tests
In directory cvs.zope.org:/tmp/cvs-serv2488/src/zope/app/i18nfile/tests
Modified Files:
test_i18nfile.py test_i18nimage.py
Log Message:
Modernized the File and Image API to rely more heavily on schemas instead of
accessor and mutator methods. Removed a lot of unnecessary cruft from the
object classes and their views.
=== Zope3/src/zope/app/i18nfile/tests/test_i18nfile.py 1.3 => 1.4 ===
--- Zope3/src/zope/app/i18nfile/tests/test_i18nfile.py:1.3 Fri Mar 5 17:09:08 2004
+++ Zope3/src/zope/app/i18nfile/tests/test_i18nfile.py Thu Mar 18 22:17:42 2004
@@ -11,10 +11,10 @@
# FOR A PARTICULAR PURPOSE.
#
##############################################################################
-"""
+"""I18n File Tests
+
$Id$
"""
-
import unittest
from zope.interface.verify import verifyClass
from zope.i18n.tests.testii18naware import TestII18nAware
@@ -39,13 +39,13 @@
def testEmpty(self):
file = self._makeFile()
- self.assertEqual(file.getContentType(), '')
+ self.assertEqual(file.contentType, '')
self.assertEqual(file.getData(), '')
self.assertEqual(file.getDefaultLanguage(), 'en')
def testConstructor(self):
file = self._makeFile('Foobar')
- self.assertEqual(file.getContentType(), '')
+ self.assertEqual(file.contentType, '')
self.assertEqual(file.getData(), 'Foobar')
self.assertEqual(file.getData('en'), 'Foobar')
self.assertEqual(file.getData('nonexistent'), 'Foobar')
@@ -54,7 +54,7 @@
file = self._makeFile('Foobar', 'text/plain')
- self.assertEqual(file.getContentType(), 'text/plain')
+ self.assertEqual(file.contentType, 'text/plain')
self.assertEqual(file.getData(), 'Foobar')
self.assertEqual(file.getData('en'), 'Foobar')
self.assertEqual(file.getData('nonexistent'), 'Foobar')
@@ -63,7 +63,7 @@
file = self._makeFile(data='Foobar', contentType='text/plain')
- self.assertEqual(file.getContentType(), 'text/plain')
+ self.assertEqual(file.contentType, 'text/plain')
self.assertEqual(file.getData(), 'Foobar')
self.assertEqual(file.getData('en'), 'Foobar')
self.assertEqual(file.getData('nonexistent'), 'Foobar')
@@ -72,7 +72,7 @@
file = self._makeFile(data='Foobar', contentType='text/plain',
defaultLanguage='fr')
- self.assertEqual(file.getContentType(), 'text/plain')
+ self.assertEqual(file.contentType, 'text/plain')
self.assertEqual(file.getData(), 'Foobar')
self.assertEqual(file.getData('en'), 'Foobar')
self.assertEqual(file.getData('nonexistent'), 'Foobar')
@@ -82,8 +82,8 @@
def testMutators(self):
file = self._makeFile()
- file.setContentType('text/plain')
- self.assertEqual(file.getContentType(), 'text/plain')
+ file.contentType = 'text/plain'
+ self.assertEqual(file.contentType, 'text/plain')
self.assertEqual(sorted(file.getAvailableLanguages()), ['en'])
file.setData('Foobar')
@@ -95,24 +95,27 @@
self.assertEqual(file.getData('fr'), 'Barbaz')
self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'fr'])
- file.edit('Blah', 'text/html')
- self.assertEqual(file.getContentType(), 'text/html')
+ file.data = 'Blah'
+ file.contentType = 'text/html'
+ self.assertEqual(file.contentType, 'text/html')
self.assertEqual(file.getData(), 'Blah')
self.assertEqual(file.getData('fr'), 'Barbaz')
self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'fr'])
- file.edit('Quux', 'text/html', 'lt')
- self.assertEqual(file.getContentType(), 'text/html')
+ file.setData('Quux', 'lt')
+ file.contentType = 'text/html'
+ self.assertEqual(file.contentType, 'text/html')
self.assertEqual(file.getData(), 'Blah')
self.assertEqual(file.getData('fr'), 'Barbaz')
self.assertEqual(file.getData('lt'), 'Quux')
self.assertEqual(file.getSize(), len('Blah'))
self.assertEqual(file.getSize('fr'), len('Barbaz'))
self.assertEqual(file.getSize('lt'), len('Quux'))
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'fr', 'lt'])
+ self.assertEqual(sorted(file.getAvailableLanguages()),
+ ['en', 'fr', 'lt'])
file.removeLanguage('lt')
- self.assertEqual(file.getContentType(), 'text/html')
+ self.assertEqual(file.contentType, 'text/html')
self.assertEqual(file.getData(), 'Blah')
self.assertEqual(file.getData('fr'), 'Barbaz')
self.assertEqual(file.getSize(), len('Blah'))
@@ -122,7 +125,7 @@
self.assertEqual(file.getSize('lt'), len('Blah'))
file.removeLanguage('nonexistent')
- self.assertEqual(file.getContentType(), 'text/html')
+ self.assertEqual(file.contentType, 'text/html')
self.assertEqual(file.getData(), 'Blah')
self.assertEqual(file.getData('fr'), 'Barbaz')
self.assertEqual(file.getSize(), len('Blah'))
@@ -131,7 +134,8 @@
self.assertEqual(file.getData('lt'), 'Blah')
self.assertEqual(file.getSize('lt'), len('Blah'))
- self.assertRaises(ValueError, file.removeLanguage, file.getDefaultLanguage())
+ self.assertRaises(ValueError, file.removeLanguage,
+ file.getDefaultLanguage())
self.assertRaises(ValueError, file.setDefaultLanguage, 'nonexistent')
def testLargeDataInput(self):
=== Zope3/src/zope/app/i18nfile/tests/test_i18nimage.py 1.3 => 1.4 ===
--- Zope3/src/zope/app/i18nfile/tests/test_i18nimage.py:1.3 Fri Mar 5 17:09:08 2004
+++ Zope3/src/zope/app/i18nfile/tests/test_i18nimage.py Thu Mar 18 22:17:42 2004
@@ -11,7 +11,7 @@
# FOR A PARTICULAR PURPOSE.
#
##############################################################################
-"""
+"""I18n Image Tests
$Id$
"""
@@ -40,14 +40,14 @@
def testEmpty(self):
file = self._makeImage()
- self.assertEqual(file.getContentType(), '')
+ self.assertEqual(file.contentType, '')
self.assertEqual(file.getData(), '')
self.assertEqual(file.getDefaultLanguage(), 'en')
def testConstructor(self):
file = self._makeImage('Data')
- self.assertEqual(file.getContentType(), '')
+ self.assertEqual(file.contentType, '')
self.assertEqual(file.getData(), 'Data')
self.assertEqual(file.getData('en'), 'Data')
self.assertEqual(file.getData('nonexistent'), 'Data')
@@ -55,7 +55,7 @@
self.assertEqual(sorted(file.getAvailableLanguages()), ['en'])
file = self._makeImage('Data', defaultLanguage='fr')
- self.assertEqual(file.getContentType(), '')
+ self.assertEqual(file.contentType, '')
self.assertEqual(file.getData(), 'Data')
self.assertEqual(file.getData('en'), 'Data')
self.assertEqual(file.getData('nonexistent'), 'Data')
@@ -63,76 +63,22 @@
self.assertEqual(sorted(file.getAvailableLanguages()), ['fr'])
def testMutators(self):
- # XXX What's the point of this test? Does it test that data
- # contents override content-type? Or not? If the former, then
- # real image data should be used.
- file = self._makeImage()
-
- file.setContentType('text/plain')
- self.assertEqual(file.getContentType(), 'text/plain')
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en'])
-
- file.setData('Foobar')
- self.assertEqual(file.getData(), 'Foobar')
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en'])
-
- file.edit('Blah', 'text/html')
- self.assertEqual(file.getContentType(), 'text/html')
- self.assertEqual(file.getData(), 'Blah')
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en'])
-
- file.setData('Foobar in lt', 'lt')
- self.assertEqual(file.getData(), 'Blah')
- self.assertEqual(file.getData('lt'), 'Foobar in lt')
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'lt'])
-
- file.edit('Blah in fr', 'text/html', 'fr')
- self.assertEqual(file.getContentType(), 'text/html')
- self.assertEqual(file.getData(), 'Blah')
- self.assertEqual(file.getData('lt'), 'Foobar in lt')
- self.assertEqual(file.getData('fr'), 'Blah in fr')
- self.assertEqual(sorted(file.getAvailableLanguages()),
- ['en', 'fr', 'lt'])
-
- file.removeLanguage('lt')
- self.assertEqual(file.getContentType(), 'text/html')
- self.assertEqual(file.getData(), 'Blah')
- self.assertEqual(file.getData('fr'), 'Blah in fr')
- self.assertEqual(file.getSize(), len('Blah'))
- self.assertEqual(file.getSize('fr'), len('Blah in fr'))
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'fr'])
- self.assertEqual(file.getData('lt'), 'Blah')
- self.assertEqual(file.getSize('lt'), len('Blah'))
-
- file.removeLanguage('nonexistent')
- self.assertEqual(file.getContentType(), 'text/html')
- self.assertEqual(file.getData(), 'Blah')
- self.assertEqual(file.getData('fr'), 'Blah in fr')
- self.assertEqual(file.getSize(), len('Blah'))
- self.assertEqual(file.getSize('fr'), len('Blah in fr'))
- self.assertEqual(sorted(file.getAvailableLanguages()), ['en', 'fr'])
- self.assertEqual(file.getData('lt'), 'Blah')
- self.assertEqual(file.getSize('lt'), len('Blah'))
-
- self.assertRaises(ValueError, file.removeLanguage,
- file.getDefaultLanguage())
-
- self.assertRaises(ValueError, file.setDefaultLanguage, 'nonexistent')
+ image = self._makeImage()
# Check that setData updates content type only when updating the
# default language. Need some real images or at least headers
# for that.
gifHdr = 'GIF87a\x20\x00\x10\x00'
- file.setData(gifHdr)
- self.assertEqual(file.getContentType(), 'image/gif')
+ image.setData(gifHdr)
+ self.assertEqual(image.contentType, 'image/gif')
pngHdr = '\211PNG\r\n\032\n\0\0\0\x20\0\0\0\x10'
- file.setData(pngHdr, 'fr')
- self.assertEqual(file.getContentType(), 'image/gif')
+ image.setData(pngHdr, 'fr')
+ self.assertEqual(image.contentType, 'image/gif')
- file.setData(pngHdr, 'en')
- self.assertEqual(file.getContentType(), 'image/png')
+ image.setData(pngHdr, 'en')
+ self.assertEqual(image.contentType, 'image/png')
def testInterface(self):
from zope.app.file.interfaces import IImage
@@ -157,6 +103,7 @@
file.setData('', language='lt')
file.setDefaultLanguage('lt')
self.assertEqual(file.getDefaultLanguage(), 'lt')
+
def test_suite():
loader = unittest.TestLoader()
More information about the Zope3-Checkins
mailing list