API unittests coverage is unsufficient

Hi,

I just discovered when trying to add unittests for Host API that the coverage is not sufficient for all the controllers.

For example, this section :
    def test_make_app(self):
        self.app.make_app()
        self.fake_ff.assert_called_once_with(self.fake_app().config,
                                             admin_user='admin',
                                             admin_tenant_name='admin',
                                             auth_port='35357',
                                             auth_protocol='http',
                                             auth_version='v2.0',
                                             admin_password='climate',
                                             auth_host='127.0.0.1')

only checks that the remote method is called, but doesn't check the values, which can lead to a successful test while the method could raise an Exception.

Let me give a quick example on why only checking assert_called is not enough.
Provided this code snippet :
import A
def foo()
    myA = A()
    mydict = myA.to_dict()
    return mydict['id']

and provided this unittest method :
def test_foo(self):
    fake_Adict = self.patch(A,'to_dict')
    self.fake_Adict.assert_called_once()

it will work.
If we change foo() to return mydict['name'] instead of mydict['id'], the test will still say OK while it will break at runtime...

The coverage would be better saying :
fake_Adict.return_value = {'id': 'bar'}
self.assertEqual('bar', test_foo())