https://issues.qgis.org/https://issues.qgis.org/favicon.ico2013-07-14T23:12:20ZQGIS Issue TrackingQGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=431332013-07-14T23:12:20ZVincent Moravincent.mora@oslandia.com
<ul><li><strong>Assignee</strong> set to <i>Vincent Mora</i></li></ul> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=431542013-07-15T04:08:11ZVincent Moravincent.mora@oslandia.com
<ul></ul>What I understand from the demand is:
<ul>
<li>for alternative plugin paths
<ul>
<li>checking, if the environment variable QGIS_PLUGINPATH exists, that the directory $QGIS_PLUGINPATH exists and is readable</li>
</ul></li>
</ul>
<ul>
<li>for config path
<ul>
<li>running qgis --configpath $TEST_DIR</li>
<li>checking that $TEST_DIR/qgis.db has been created</li>
</ul></li>
</ul>
<ul>
<li>for settings
<ul>
<li>running --optionspath $TEST_DIR</li>
<li>checking that $TEST_DIR/QGIS/QGIS2.ini has been created</li>
</ul></li>
</ul>
<p>Regis, can you please tell me if this is what you had in mind ?</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=432252013-07-16T01:45:07ZHugo Mercierhugo.mercier@oslandia.com
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Feedback</i></li><li><strong>Assignee</strong> changed from <i>Vincent Mora</i> to <i>Regis Haubourg</i></li></ul> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=433372013-07-18T04:12:26ZHugo Mercierhugo.mercier@oslandia.com
<ul><li><strong>Assignee</strong> changed from <i>Regis Haubourg</i> to <i>Hugo Mercier</i></li><li><strong>Status</strong> changed from <i>Feedback</i> to <i>In Progress</i></li><li><strong>Pull Request or Patch supplied</strong> changed from <i>No</i> to <i>Yes</i></li></ul><p>Pull request submitted :<br /><a class="external" href="https://github.com/qgis/Quantum-GIS/pull/749">https://github.com/qgis/Quantum-GIS/pull/749</a></p>
<p>A new unit test is added that tests QGIS_PLUGINPATH, --configpath and --optionspath</p>
Two character encoding issues have been found :
<ul>
<li>in main.cpp</li>
<li>in the way python plugin paths are parsed</li>
</ul> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=436592013-07-29T04:01:14ZRegis Haubourgregis.haubourg@oslandia.com
<ul></ul><p>Hi Hugo, <br />Did you file bugs and fixe the character encoding issues mentionned or should we do it?</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=436602013-07-29T04:22:08ZVincent Moravincent.mora@oslandia.com
<ul></ul><p>Hi Regis,</p>
<p>The bug fixes are included in the PR <a class="issue tracker-2 status-5 priority-3 priority- closed" href="https://issues.qgis.org/issues/749" title="Graticule plugin enhancements (Closed)">#749</a>.</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=441172013-08-16T04:00:45ZHugo Mercierhugo.mercier@oslandia.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Fixed. Pull requested : <a class="external" href="https://github.com/qgis/Quantum-GIS/pull/749">https://github.com/qgis/Quantum-GIS/pull/749</a></p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=447162013-09-15T01:51:47ZHugo Mercierhugo.mercier@oslandia.com
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Feedback</i></li><li><strong>Assignee</strong> changed from <i>Hugo Mercier</i> to <i>Larry Shaffer</i></li></ul><p>It has been merged by Tim (<a class="changeset" href="https://issues.qgis.org/projects/qgis/repository/revisions/64c947b2f05da81d8a3a119132c8ba95669076f6" title="Merge pull request #749 from Oslandia/issue8045b Unit tests for command line arguments">64c947b2f05da81d8a3a119132c8ba95669076f6</a></p>
<p>The only missing part is whether this new test (PyQgsAppStartup) works correctly on MacOSX.<br />Larry, do you think you could have a look and tell me if it fails ?</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=447322013-09-15T13:28:35ZLarry Shaffer
<ul><li><strong>Target version</strong> changed from <i>Version 2.0.0</i> to <i>Future Release - High Priority</i></li></ul><p>Hi Hugo,</p>
<p>I have made some changes to the test. See <a href="https://github.com/qgis/Quantum-GIS/pull/898" class="external">pull request</a> (<a href="https://github.com/qgis/Quantum-GIS/pull/898.patch" class="external">as patch</a>). Please review.</p>
<p>These are the issues I had on Mac (see commit message as well):</p>
<ul>
<li>Test directories needed to be created first</li>
<li>Subdirectory for .ini file is "qgis.org" on Mac</li>
</ul>
<p>I have not tested on Win or Linux (will next week).</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=447482013-09-16T23:43:45ZHugo Mercierhugo.mercier@oslandia.com
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li></ul><p>Thanks Larry for the fix and the improvements ! It works fine on Linux here.<br />Still Windows to test</p> QGIS Application - Feature request #8045: add unit test for startup options (pluginpath, configpath)https://issues.qgis.org/issues/8045?journal_id=485832014-02-13T11:31:34ZJürgen Fischerjef@norbit.de
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>