On Tue, Sep 13, 2016 at 11:27 PM, Mads Kiilerich <m...@kiilerich.com> wrote:
> On 09/13/2016 09:54 PM, Thomas De Schampheleire wrote:
>> Having the entire set under
>> kallithea/tests/ seems more future-proof.
>> It seems the trailing dot is accepted and does what I hoped it does.
>> v3 is coming up...
> I agree with the commit message in the posted v3, but the patch content
> doesn't seem to be what is described?
> I guess the chosen solution is to have
> pytest.register_assert_rewrite('kallithea.tests.')
> in tests/__init__.py but move helper functions (and constants?) out and put
> them in something like common.py or base.py ?

The location of the register was indeed wrong. The fact of not moving
the existing code to a separate file was intentional and seen as
future improvement, because I thought it would require changes in each
test, but in fact it is not true, and I'm cooking up a v4 that will do

kallithea-general mailing list

Reply via email to