Comment 3 for bug 814518

Revision history for this message
dan wendlandt (danwent) wrote :

I took a closer look at this. It seems that the "create_network_badrequest" methods generates a 400 error and passes the unit test because the request lacks a required field ("net-name"), while the "create_port_badrequest" fails because a port has no required fields.

This led me to the conclusion that the existing API parsing logic does not seem to prevent there from being "extra" fields in the request that the API ignores, and adding such logic would be needed to get this test to pass. However, I'm not sure that is actually the right thing to do. I think this is part of a broader discussion about whether plugins should be able to use "data extension" or not. I will send mail to the list about that.

In the mean time, I would advocate for removing this test and deciding to re-insert it based on the decision we make regarding data extension.