however we should add a second test making sure that only found types are checked (and hence you can pass dicts with one key).

in mag evokeds units=dict(mag='fT') should pass.

also units=dict(mag='fT', misc='AU')

because no misc is in data.

as this case gave rise to recent changes it should be included.
makes sense?

On 18.01.2013, at 08:53, Eric89GXL <notifications@github.com> wrote:

> In mne/tests/test_viz.py:
> 
> > +
> > +def test_plot_topo_image_epochs():
> > +    """Test plotting of epochs image topography
> > +    """
> > +    title = 'ERF images - MNE sample data'
> > +    plot_topo_image_epochs(epochs, layout, sigma=0.5, vmin=-200, vmax=200,
> > +                           colorbar=True, title=title)
> > +
> > +
> > +def test_plot_evoked():
> > +    """Test plotting of evoked
> > +    """
> > +    evoked.plot(proj=True, hline=[1])
> > +    # Assert we get an error with incorrect dict specification
> > +    assert_raises(ValueError, plot_evoked, evoked, hline=[1],
> > +                  units=dict(eeg=10))
> Isn't this a version of that? I picked meg, and passed a dict without mag or grad in it...?
> 
> —
> Reply to this email directly or view it on GitHub.
>