Code review comment for lp:~jtv/maas/api-list-networks

Revision history for this message
Raphaël Badin (rvb) wrote :

> > [0]
> >
> > 50 === added file 'src/maasserver/tests/test_api_networks.py'
> > [...]
> > 83 + def test_list_returns_network(self):
> >
> > The name of this test is strange. One would expect
> > test_list_returns_network*s*
>
> Except in this case it's only one.

Well, I'd say it's a list of networks… with only one network. For the casual reader, test_list_returns_network looks weird.

« Back to merge proposal