Sure. Good point, this is really somewhat implicit.

On Sun, Feb 3, 2013 at 10:28 AM, Alexandre Gramfort <
notifications@github.com> wrote:

> In mne/minimum_norm/tests/test_inverse.py:
>
> > +
> > +def test_make_inverse_operator_bads():
> > +    """Test MNE inverse computation given a mismatch of bad channels.
> > +    """
> > +    fwd_op = read_forward_solution(fname_fwd, surf_ori=True)
> > +
> > +    # test bads
> > +    bad = evoked.info['bads'].pop()
> > +    inv_ = make_inverse_operator(evoked.info, fwd_op, noise_cov, loose=None)
> > +    union_good = set(noise_cov['names']) & set(evoked.ch_names)
> > +    union_bads = set(noise_cov['bads']) & set(evoked.info['bads'])
> > +    evoked.info['bads'].append(bad)
> > +
> > +    assert_true(not set(inv_['info']['ch_names']) - union_good)
> > +
> > +    assert_true(not set(inv_['info']['bads']) - union_bads)
>
> could you make this test more explicit using the len of the difference.
> btw assert_true(not ... looks like an assert_false to me :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/426/files#r2870527>.
>
>