<div dir="ltr"><div dir="ltr" class="gmail_msg">I am against moving the test files into the sub-modules. </div><div dir="ltr" class="gmail_msg"><br></div><div class="gmail_msg">It seems like a lot of thrashing (on the down-stream packagers who package the main library and tests separately and on peoples brains) for no obvious gain.</div><div class="gmail_msg"><br></div><div class="gmail_msg">Tom</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Jan 25, 2017 at 1:15 PM Nelle Varoquaux <<a href="mailto:nelle.varoquaux@gmail.com" class="gmail_msg" target="_blank">nelle.varoquaux@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">Hello everyone,<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Now that the pytest migration is almost over (thanks Elliott!), I was wondering if this could be a good opportunity to reorganize the test files to follow a "normal" python package structure.</div><div class="gmail_msg">The recommended python package structure is to place a tests folder in each submodule, and the relevant tests files in this folder.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">To take a concrete example, it would for example require moving the tests corresponding of <a href="http://matplotlib.sphinx_ext.mathmpl.py" class="gmail_msg" target="_blank">matplotlib.sphinx_ext.mathmpl.py</a>  to matplotlib/sphinxext/tests/test_mathmpl.py</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">This would be a more or less incompatible change in the sense that matplotlib.tests seems to be an importable module (it probably shouldn't), with about a dozen functions. We could easily make this backward compatible in case anyone does use this module anywhere.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><div class="gmail_msg">['__builtins__',</div><div class="gmail_msg"> '__cached__',</div><div class="gmail_msg"> '__doc__',</div><div class="gmail_msg"> '__file__',</div><div class="gmail_msg"> '__loader__',</div><div class="gmail_msg"> '__name__',</div><div class="gmail_msg"> '__package__',</div><div class="gmail_msg"> '__path__',</div><div class="gmail_msg"> '__spec__',</div><div class="gmail_msg"> '_multiprocess_can_split_',</div><div class="gmail_msg"> 'absolute_import',</div><div class="gmail_msg"> 'assert_str_equal',</div><div class="gmail_msg"> 'difflib',</div><div class="gmail_msg"> 'division',</div><div class="gmail_msg"> 'os',</div><div class="gmail_msg"> 'print_function',</div><div class="gmail_msg"> 'setup',</div><div class="gmail_msg"> 'six',</div><div class="gmail_msg"> 'unicode_literals']</div></div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">If we go down that route, we need to completely finish the pytest migration first.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Cheers,</div><div class="gmail_msg">N</div><div class="gmail_msg"><br class="gmail_msg"></div></div>
_______________________________________________<br class="gmail_msg">
Matplotlib-devel mailing list<br class="gmail_msg">
<a href="mailto:Matplotlib-devel@python.org" class="gmail_msg" target="_blank">Matplotlib-devel@python.org</a><br class="gmail_msg">
<a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" class="gmail_msg" target="_blank">https://mail.python.org/mailman/listinfo/matplotlib-devel</a><br class="gmail_msg">
</blockquote></div></div>