Noted.  Out of curiosity, why are plain assertions not a good idea?  (I'll
push updates in a sec)

On Wed, Mar 27, 2013 at 6:06 PM, Warren Weckesser
<notifications@github.com>wrote:

> In scipy/io/matlab/tests/test_mio.py:
>
> > @@ -977,6 +977,11 @@ def test_loadmat_varnames():
> >      assert_equal(set(vars.keys()), set(['a'] + sys_v_names))
> >      vars = loadmat(eg_file, variable_names=['theta'])
> >      assert_equal(set(vars.keys()), set(['theta'] + sys_v_names))
> > +    vars = loadmat(eg_file, variable_names=('theta',))
> > +    assert_equal(set(vars.keys()), set(['theta'] + sys_v_names))
> > +    vnames = ['theta']
> > +    vars = loadmat(eg_file, variable_names=vnames)
> > +    assert vnames != [], 'List modified'
>
> Don't use the plain assert statement in tests. I think a better test is assert_equal(vnames,
> ['theta']).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scipy/scipy/pull/486/files#r3555182>
> .
>



-- 
Louis Thibault
+33 6 40 24 06 05