[ZCM] [ZC] 1104/ 4 Comment "testSetupServers fails on Windows"

[Collector: Zope Bugs, Features, and Patches ...]

Issue #1104 Update (Comment) "testSetupServers fails on Windows"
 Status Accepted, Zope/bug medium
To followup, visit:
  http://zope.org/Collectors/Zope/1104

==============================================================
= Comment - Entry #4 by tim_one on Oct 31, 2003 11:38 am

The bit about ConfigurationError was a red herring:  assertRaises() failures are darned hard to interpret correctly.

What actually happened when I removed the "# conflict" comment is starter.setupServers() didn't raise any exception, so assertRaises() was complaining that it *wanted* to see a ConfigurationError.

BTW, I don't think you've checked your changes in on Zope-2_7-branch, which is where I'm running the test (see "Version info:" above).

________________________________________
= Comment - Entry #3 by fdrake on Oct 31, 2003 11:31 am

The syntax of the configuration data in that test is bogus;
the intent was that "# conflict" be a comment, but comments
must be on lines by themselves.

I've fixed that and made ZConfig stricter about what it
considers a valid hostname; the later fix would have
caused the syntax error to be identified earlier regardless
of platform.

Still looking into the rest.
________________________________________
= Assign - Entry #2 by tim_one on Oct 31, 2003 10:50 am

 Status: Pending => Accepted

 Supporters added: fdrake

Noting that if I change the line

               address 18092 # conflict

to

               address 18092

in the test (removing the "# conflict" comment), the test fails in a very different way:

 FAIL: testSetupServers (Zope.Startup.tests.testStarter.ZopeStarterTestCase)
 ---------------------------------------------------------------------
 Traceback (most recent call last):
   File "C:\Code\Zope-2_7-branch\lib/python\Zope\Startup\tests\testStarter.py", line 194, in testSetupServers
   self.assertRaisesZConfig.ConfigurationError, starter.setupServers)
   File "C:\PYTHON23\lib\unittest.py", line 295, in failUnlessRaises
    raise self.failureException, excName
AssertionError: ConfigurationError

So it looks (a) the comment isn't getting stripped out on Windows; and, (b) ConfigurationError isn't a subclass of ZConfig.ConfigurationError on Windows.

Assigning Fred cuz I bet he's less lost than me.
________________________________________
= Request - Entry #1 by tim_one on Oct 31, 2003 10:42 am

testSetupServers has been failing for a long time on Windows, whether I run it, or whether from ChrisW's automated test runner reports.  It doesn't fail on Linux.

 ERROR: testSetupServers (Zope.Startup.tests.testStarter.ZopeStarterTestCase)
 ----------------------------------------------------------------------
 Traceback (most recent call last):
   File "C:\Code\Zope-2_7-branch\lib/python\Zope\Startup\tests\testStarter.py", line 194, in testSetupServers
     self.assertRaises(ZConfig.ConfigurationError, starter.setupServers)
   File "C:\PYTHON23\lib\unittest.py", line 289, in failUnlessRaises
     callableObj(*args, **kwargs)
   File "C:\Code\Zope-2_7-branch\lib/python\Zope\Startup\__init__.py", line 187, in setupServers
     servers.append(server.create())
   File "C:\Code\Zope-2_7-branch\lib/python\ZServer\datatypes.py", line 93, in create
     logger_object=access_logger)
   File "C:\Code\Zope-2_7-branch\lib/python\ZServer\FTPServer.py", line 600, in __init__
     ftp_server.__init__(self, None, *args, **kw)
   File "C:\Code\Zope-2_7-branch\lib/python\ZServer\medusa\ftp_server.py", line 727, in __init__
     self.bind ((self.ip, self.port))
   File "C:\PYTHON23\lib\asyncore.py", line 300, in bind
     return self.socket.bind(addr)
   File "<string>", line 1, in bind
 TypeError: an integer is required

The second half of testSetupServers() appears to be setting up an error case, but it's not dying in the way the test expects.  Instead

   self.ip == '18092 # conflict'

and

   self.port == None

at the time of the failure, and socket.bind has no idea what to make of that.  socket.bind raises TypeError, which is sensible given those nonsense arguments.

The test is expecting ZConfig.ConfigurationError instead.
==============================================================