Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fix PyQgsAppStartup on windows
  • Loading branch information
jef-n committed Jan 12, 2016
1 parent 917ea5c commit 0ba089e
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tests/src/python/test_qgsappstartup.py
Expand Up @@ -139,20 +139,20 @@ def testPyQgisStartupEnvVar(self):
# verify PYQGIS_STARTUP env variable file is run by embedded interpreter
# create a temp python module that writes out test file
testfile = 'pyqgis_startup.txt'
testfilepath = os.path.join(self.TMP_DIR, testfile)
testfilepath = os.path.join(self.TMP_DIR, testfile).replace('\\', '/')
testcode = [
"f = open('{0}', 'w')\n".format(testfilepath.replace('\\', '\\\\')),
"f = open('{0}', 'w')\n".format(testfilepath),
"f.write('This is a test')\n",
"f.close()\n"
]
testmod = os.path.join(self.TMP_DIR, 'pyqgis_startup.py')
testmod = os.path.join(self.TMP_DIR, 'pyqgis_startup.py').replace('\\', '/')
f = open(testmod, 'w')
f.writelines(testcode)
f.close()
msg = 'Failed to create test file from executing PYQGIS_STARTUP file'
msg = 'Creation of test file by executing PYQGIS_STARTUP file failed'
assert self.doTestStartup(
testFile=testfilepath,
timeOut=15,
timeOut=120,
env={'PYQGIS_STARTUP': testmod}), msg


Expand Down

3 comments on commit 0ba089e

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jef-n Can I revert this? It hasn't fixed the test on the nightly Windows builds (see http://dash.orfeo-toolbox.org/testDetails.php?test=36669232&build=215399), and seems to have broken the test under the debian nightlies (http://dash.orfeo-toolbox.org/testDetails.php?test=36656832&build=215373)

@jef-n
Copy link
Member Author

@jef-n jef-n commented on 0ba089e Jan 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nyalldawson not so sure that it's this commit, it essentially just raises the timeout. I suspect that it's the cloudflare transition that causes this. Let's see if 764c649 helps.

@jef-n
Copy link
Member Author

@jef-n jef-n commented on 0ba089e Jan 24, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.