<div dir="ltr">I'm currently looking at how to fix PyQgsAppStartup but at the moment I'm not sure why it is failing given that it works on the command line normally. <br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 2 Jun 2015 at 11:33 Tim Sutton <<a href="mailto:tim@kartoza.com">tim@kartoza.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi<div></div></div><div style="word-wrap:break-word"><div><br><blockquote type="cite">On 02 Jun 2015, at 02:24, Matthias Kuhn <<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>> wrote:<br><br>Hi,<br><br>For more than a week already we have failing tests on master<br><a href="https://travis-ci.org/qgis/QGIS/builds" target="_blank">https://travis-ci.org/qgis/QGIS/builds</a><br><br>This basically renders the whole test suite a lot less usable.<br><br>Some examples/thoughts:<br><br>* Every pull request fails tests [1] I often look at the test results<br>to check if it's even worth looking at a pull request. Handling pull<br>requests is an issue on its own as there is no direct benefit for the<br>reviewer most of the time. Don't make the reviewer's time even harder!<br><br>* We don't see when a commit trashes our compiler. Today we had a<br>commit which broke the compiler. The author was not informed by mail.<br><br>* We don't see which commit broke a certain test. E.g. At the moment<br>the PyQgsAppStartup test is failing. And to find out which commit broke<br>it one has to dig deep because failing of this test is masked by the<br>already failing tests.<br><br>* It discourages people to write tests if they can be broken easily.<br><br>The tests started to fail with the merge of some big work, just previous<br>to feature freeze. I can understand that people want to get commits into<br>master before feature freeze. But quality should not suffer from this.<br>And if there's something broken, please fix it fast.<br><br>Finally Nyall was starting to fix things - crucial GIS functionality<br>like splitting features - although I don't think he received anything in<br>return, thank you very much, Nyall. The one thing left to do is to add<br>doxygen API documentation - and to fix the newly broken PyQgsAppStartup<br>test. I really hope we will finally see the green light again.<br><br>I am a hippy. I don't like rules and policies. I am not in favor of a<br>very strong policy concerning the tests. Common sense should be valued<br>more than rules. Policies like "every new feature has to come with a<br>test" are no good solution as they encourage people to write silly<br>tests. And I am much too lazy to write a QEP for a policy to revert<br>commits which break the tests.<br><br>So... How do we fix it?<br>If tomorrow everybody on this list writes some API documentation we<br>should have fixed that test by noon.<br>And then I'll promise I'll fix the failing PyQgsAppStartup test to get<br>the green light going again.<br><br>Thank you very much<br>Matthias<br><br>[1] <a href="https://github.com/qgis/QGIS/pulls" target="_blank">https://github.com/qgis/QGIS/pulls</a> ;<br><a href="https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aclosed" target="_blank">https://github.com/qgis/QGIS/pulls?q=is%3Apr+is%3Aclosed</a><br>[2]<br><a href="https://github.com/qgis/QGIS/commit/ded11b32562c100aeaae95bf20e21bcfc38d0777" target="_blank">https://github.com/qgis/QGIS/commit/ded11b32562c100aeaae95bf20e21bcfc38d0777</a><br></blockquote><div><br></div></div></div><div style="word-wrap:break-word"><div><div>I can only wholeheartedly second Matthias’ message above - we should never push to master if the tests don’t pass, and if for some reason they break we should not push anything more to master before the break is fixed.</div><div><br></div><div>Regards</div><div><br></div><div>Tim</div></div></div><div style="word-wrap:break-word"><div><div><br></div><br><blockquote type="cite"><br>_______________________________________________<br>Qgis-developer mailing list<br><a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br><a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br></blockquote><br></div></div><div style="word-wrap:break-word"><div><div><span>—</span><br><span><br></span><span style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><br><span><img height="66" width="160" src="cid:62C890D4-3964-4609-BDE6-7536D5FBDD70"></span><div style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-align:center"><br><br></div><div style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-align:center">Tim Sutton</div><div style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-align:center"><br></div><div style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div style="text-align:center">Visit <a href="http://kartoza.com" target="_blank">http://kartoza.com</a> to find out about open source:</div><div style="text-align:center"><br></div><div><div style="text-align:center">* Desktop GIS programming services</div><div style="text-align:center">* Geospatial web development</div><div style="text-align:center">* GIS Training</div><div style="text-align:center">* Consulting Services</div><div style="text-align:center"><br></div><div><div style="text-align:center">Skype: timlinux Irc: timlinux on #qgis at <a href="http://freenode.net" target="_blank">freenode.net</a></div><div style="text-align:center">Tim is a member of the QGIS Project Steering Committee</div><div style="text-align:center"><br></div><div style="text-align:center">Kartoza is a merger between Linfiniti and Afrispatial</div></div></div></div>
</span></div><br></div></div>_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>