[Checkins] SVN: Sandbox/ctheune/zodbupgrade/src/zodbupgrade/analyze.py Document that I just can't get a test case for this issue.

Christian Theune ct at gocept.com
Wed Jun 3 02:32:30 EDT 2009


Log message for revision 100605:
  Document that I just can't get a test case for this issue.
  

Changed:
  U   Sandbox/ctheune/zodbupgrade/src/zodbupgrade/analyze.py

-=-
Modified: Sandbox/ctheune/zodbupgrade/src/zodbupgrade/analyze.py
===================================================================
--- Sandbox/ctheune/zodbupgrade/src/zodbupgrade/analyze.py	2009-06-02 21:24:46 UTC (rev 100604)
+++ Sandbox/ctheune/zodbupgrade/src/zodbupgrade/analyze.py	2009-06-03 06:32:30 UTC (rev 100605)
@@ -58,6 +58,9 @@
                         "factory %r does not have __name__, can't check canonical location" % factory)
                     continue
                 if not hasattr(factory, '__module__'):
+                    # TODO: This case isn't covered with a test. I just
+                    # couldn't provoke a factory to not have a __module__ but
+                    # users reported this issue to me.
                     logging.warn(
                         "factory %r does not have __module__, can't check canonical location" % factory)
                     continue



More information about the Checkins mailing list