API call /flow_manager/v2/flows/{dpid} on POST is returning misleading response status

[ArturoQuintana]

@hdiogenes @italovalcy @ajoaoff @rmotitsuki

• If the payload items field is empty, it returns 200 when it should be 400.

reflected on end-to-end test: tests/test_e2e_20_flow_manager.py::TestE2EFlowManager::test_0131_install_flow_should_fail

Code:

def test_0131_install_flow_should_fail(self):

        payload = {}

        api_url = KYTOS_API + '/flow_manager/v2/flows/00:00:00:00:00:00:00:01'
        response = requests.post(api_url, data=json.dumps(payload),
                                 headers={'Content-type': 'application/json'})
        assert response.status_code == 400

The same happens with these cases:

payload = {
            "flows": [
            ]
        }

and

payload = {
            "flows": [
                {

                }
            ]
        }

• A not serializable JSON broke the application, and it should return a 400 error.

Case:

payload = {
            "flows": [
                {
                    "priority"
                }
            ]
        }

The issue mentioned above should be verified for any not serializable JSON