[SciPy-Dev] line endings in test failures?

[Skipper Seabold]

On Thu, May 20, 2010 at 1:09 PM, Skipper Seabold <jsseabold at gmail.com> wrote:
> On Thu, May 20, 2010 at 12:52 PM, Pauli Virtanen <pav at iki.fi> wrote:
>> Thu, 20 May 2010 10:44:34 -0400, Skipper Seabold wrote:
>> [clip]
>>> Ah, so it is.  Just out of curiosity, why did you turn this always on?
>>>  I don't see what assert introspection adds in a case like this.  Is
>>> it possible to make it optional but a default keyword to
>>> NoseTester.test's extra_argv?
>>
>> There were some tests that only did
>>
>>        assert (something)
>>
>> and it was useful to see what the something was on the buildbot, when
>> they were failing. I don't think this can be turned on on a per-test
>> basis.
>>
>> Anyway, if it seems too disturbing, I guess it can be turned back off.
>>
>
> It really doesn't matter to me, if it's helpful to you, I just had to
> sift through a lot of extra noise refactoring some tests.  We use a
> monkey-patched version of NoseTester for statsmodels and if you apply
> the following diff (numpy/tesing/nosetester.py), it would allow me to
> get around the --detailed-errors.  I don't think it changes the new
> default behavior for numpy, but makes it optional.
>
>
> Index: nosetester.py
> ===================================================================
> --- nosetester.py       (revision 8417)
> +++ nosetester.py       (working copy)
> @@ -230,9 +230,6 @@
>             argv+=['--cover-package=%s' % self.package_name, '--with-coverage',
>                    '--cover-tests', '--cover-inclusive', '--cover-erase']
>
> -        # enable assert introspection
> -        argv += ['--detailed-errors']
> -
>         # bypass these samples under distutils
>         argv += ['--exclude','f2py_ext']
>         argv += ['--exclude','f2py_f90_ext']
> @@ -249,8 +246,8 @@
>         plugins += [p() for p in nose.plugins.builtin.plugins]
>         return argv, plugins
>
> -    def test(self, label='fast', verbose=1, extra_argv=None, doctests=False,
> -             coverage=False):
> +    def test(self, label='fast', verbose=1, extra_argv=['--detailed-errors'],
> +            doctests=False, coverage=False):
>         """
>         Run tests for module using nose.
>

I filed a ticket with a patch, so this doesn't get forgotten.

http://projects.scipy.org/numpy/ticket/1498

Skipper
>