<div dir="ltr"><p dir="ltr">Hi David, (and cc'ing back in qgis-dev)</p>
<p dir="ltr">When you open a pull request against the main QGIS master repo, the automated test suite (<a href="https://travis-ci.org/qgis/QGIS">https://travis-ci.org/qgis/QGIS</a>) will kick in and run a test build and run all the existing unit tests to avoid regressions. It will also run some "meta" tests, checking code quality and adherence to the code standards.</p><p>So that's the first step - open your PR against QGIS master (not against your fork), and see how this goes. Then you basically keep revising your branch and pushing to github and for each new push Travis will retest until you get a green light.</p><p>The next step is adding new tests to cover your work. Without tests it *will* break in some future QGIS release, so that's why testing is mandatory for something like this. The provider test also runs a bunch of complex cases against each provider to ensure that they correctly handle a bunch of corner cases, so it will help ensure that your provider is well behaved and stable.</p><p>Here's some other provider tests which are good models to copy from:</p><p><a href="https://github.com/qgis/QGIS/blob/master/tests/src/python/test_provider_shapefile.py">https://github.com/qgis/QGIS/blob/master/tests/src/python/test_provider_shapefile.py</a><br></p><p><a href="https://github.com/qgis/QGIS/blob/master/tests/src/python/test_provider_spatialite.py">https://github.com/qgis/QGIS/blob/master/tests/src/python/test_provider_spatialite.py</a><br></p><p>(Note that both also implement additional tests specific to that provider, eg testRepack for shapefiles). The ms-sql provider test is a good example of a minimal test, which only implements the things required for the generic provider test (although you could also omit the enableCompiler/disableComplier methods if you haven't implemented an expression compiler for db2 yet). Then you'll also need to include relevant test data - see <a href="https://github.com/qgis/QGIS/tree/master/tests/testdata/provider">https://github.com/qgis/QGIS/tree/master/tests/testdata/provider</a> for what's expected in the test files, eg <a href="https://github.com/qgis/QGIS/blob/master/tests/testdata/provider/testdata_mssql.sql">https://github.com/qgis/QGIS/blob/master/tests/testdata/provider/testdata_mssql.sql</a>.</p><p>Hope that helps!</p><p>Nyall</p><p><br></p><p dir="ltr"><br></p>
<p dir="ltr">On 26 Jan 2016 10:05, "David Adler" <<a href="mailto:dadler@adtechgeospatial.com" target="_blank">dadler@adtechgeospatial.com</a>> wrote:<br>
><br>
> I'm not clear on what the "automated CI" is or how one runs is.<br>
><br>
> We did a fork and pull to our fork to do the development.<br>
><br>
> I'm still a bit hazy on setting up and running the tests.  Does a database need to be operational in order to run the tests? test_provider_mssql.py doesn't look like it does much.<br>
><br>
><br>
> On 1/25/2016 4:24 PM, Nyall Dawson wrote:<br>
>><br>
>><br>
>> On 26 Jan 2016 07:57, "David Adler" <<a href="mailto:dadler@adtechgeospatial.com" target="_blank">dadler@adtechgeospatial.com</a>> wrote:<br>
>> ><br>
>> > We would like to add a DB2 provider to QGIS so that QGIS users can access spatial tables on DB2 for LUW and DB2 for z/OS.<br>
>> ><br>
>> > We have implemented and tested much of this based on an existing database provider.<br>
>> ><br>
>> > What is the mechanism to have this reviewed and approved?<br>
>><br>
>> Open a pull request on github, and let the automated CI run its checks over the code.<br>
>><br>
>> Tips to get things reviewed/merged faster:<br>
>> - Make sure all your API additions are correctly documented<br>
>> - Include unit tests. At a minimum a new vector provider needs a test which inherits from ProviderTestCase. See tests/src/python/test_provider_mssql.py for an example. This ensures that the provider behaves consistently with what QGIS expects from a vector data provider.<br>
>><br>
>> Nyall<br>
>><br>
>> ><br>
>> ><br>
>> ><br>
>> > ---<br>
>> > This email has been checked for viruses by Avast antivirus software.<br>
>> > <a href="https://www.avast.com/antivirus" target="_blank">https://www.avast.com/antivirus</a><br>
>> ><br>
>> > _______________________________________________<br>
>> > Qgis-developer mailing list<br>
>> > <a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
>> > List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
>> > Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
><br>
><br>
> This email has been sent from a virus-free computer protected by Avast. <br>
> <a href="http://www.avast.com" target="_blank">www.avast.com</a></p>
</div>