A few minor comments, but also more generally it would be helpful to have a README.rst with the new test to document its purpose. Without knowing the overall aim of the test app it is harder to review and know whether it is fit for purpose.
The merge is also going to conflict at the moment, so could it be updated to merge cleanly.
A few minor comments, but also more generally it would be helpful to have a README.rst with the new test to document its purpose. Without knowing the overall aim of the test app it is harder to review and know whether it is fit for purpose.
The merge is also going to conflict at the moment, so could it be updated to merge cleanly.