Changeset 4360 for waeup/branches

Timestamp:

25 Jun 2009, 13:23:30 (15 years ago)

Author:

uli

Message:

Fix tests to get better coverage.

File:

• waeup/branches/ulif-rewrite/src/waeup/csvfile/README.txt (modified) (3 diffs)

waeup/branches/ulif-rewrite/src/waeup/csvfile/README.txt

@@ -134 +134 @@
      >>> toBool('TRUE')
      True
+
+     >>> toBool('no')
+     False
 
    If we pass in a boolean then this will be returned unchanged:
@@ -335 +338 @@
     ... """)
 
+We create a wrapper that requires 'col1' and 'col2' but does not check
+in constructor, whether this requirement is met:
+
+    >>> from waeup.csvfile.interfaces import ICSVFile
+    >>> from waeup.csvfile import CSVFile
+    >>> class ICSVFile13(ICSVFile):
+    ...   """A CSV file that contains a 'special_col' column.
+    ...   """
+
+    >>> class CSVFile13(CSVFile):
+    ...   required_fields = ['col1', 'col2']
+    ...   grok.context(basestring)
+    ...   grok.implements(ICSVFile13)
+    ...   grok.provides(ICSVFile13)
+    ...   def __init__(self, context):
+    ...     self.path = context
+
+    >>> grok.testing.grok_component('CSVFile13',  CSVFile13)
+    True
+
+.. warn:: This is bad design as :class:`ICSVFile` instances should
+          always raise an exception in their constructor if a file
+          does not meet the basic requirements.
+
+          The base constructor will check for the correct values. So,
+          if you do not overwrite the base constructor, instances will
+          check on creation time, whether they can handle the desired
+          file.
+
+
 We create a wrapper that requires 'col1' and 'col2':
 
@@ -352 +385 @@
     True
 
+Now the rules applied to all wrappers are:
 
 * If no instance of a certain wrapper can be created from the given