jyapayne/PySide
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed QtNetwork test to use '127.0.0.1' instead of 'localhost'

Browse source 
This is necessary to make all test to be able run on any buildbot machine.
Fix Http server shutdown sequence to avoid deadlocks.

Fixes bug #587

Reviewer: Hugo Parente Lima <hugo.lima@openbossa.org>
          Marcelo Lira <marcelo.lira@openbossa.org>
This commit is contained in:
Renato Araujo Oliveira Filho 2011-01-05 14:42:39 -03:00 • committed by Hugo Parente Lima
commit e4eaf410af
3 changed files with 29 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

12
tests/QtNetwork/accessManager_test.py
View file

@ -19,20 +19,26 @@ class AccessManagerCase(UsesQCoreApplication):
def tearDown(self):
super(AccessManagerCase, self).tearDown()
if self.httpd:
self.httpd.shutdown()
self.httpd = None
def goAway(self):
self.httpd.shutdown()
self.app.quit()
self.httpd = None
def slot_replyFinished(self, reply):
self.assertEqual(type(reply), QNetworkReply)
self.called = True
self.app.quit()
self.goAway()
def testNetworkRequest(self):
manager = QNetworkAccessManager()
manager.finished.connect(self.slot_replyFinished)
manager.get(QNetworkRequest(QUrl("http://localhost:%s" % self.httpd.port())))
manager.get(QNetworkRequest(QUrl("http://127.0.0.1:%s" % self.httpd.port())))
self.app.exec_()
self.assert_(self.called)
self.httpd.shutdown()
if __name__ == '__main__':
unittest.main()