Change the response from bad-request to method not allowed

API.md

@@ -397,7 +397,9 @@ For each Pod, we have fileds need to handle.
     - vlanTag: the vlan tag for `ifName` interface.
     - ipADdress: the IPv4 address of the `ifName` interface.
     - netmask: the IPv4 netmask of the `ifName` interface.
-
+7. restartPolicy: the attribute how the pod restart is container, it should be a string and only valid for those following strings.
+    - Always,OnFailure,Never
+
 Example:
 
 Request Data:

src/net/http/response.go

@@ -103,6 +103,11 @@ func Conflict(req *http.Request, resp http.ResponseWriter, errs ...error) (int,
 	return WriteStatusAndError(req, resp, http.StatusConflict, errs...)
 }
 
+// MethodNotAllow will set the status code http.StatusMethodNotAllowed, to the HTTP response message
+func MethodNotAllow(req *http.Request, resp http.ResponseWriter, errs ...error) (int, error) {
+	return WriteStatusAndError(req, resp, http.StatusMethodNotAllowed, errs...)
+}
+
 // UnprocessableEntity will set the status code http.StatusUnprocessableEntity to the HTTP response message
 func UnprocessableEntity(req *http.Request, resp http.ResponseWriter, errs ...error) (int, error) {
 	return WriteStatusAndError(req, resp, http.StatusUnprocessableEntity, errs...)

src/net/http/response_test.go

@@ -86,6 +86,7 @@ func TestSetStatus(t *testing.T) {
 		{"InternalServerError", InternalServerError, http.StatusInternalServerError},
 		{"Conflict", Conflict, http.StatusConflict},
 		{"UnprocessableEntity", UnprocessableEntity, http.StatusUnprocessableEntity},
+		{"MethodNotAllow", MethodNotAllow, http.StatusMethodNotAllowed},
 	}
 
 	for _, tc := range testCases {

src/server/handler_network.go

@@ -201,7 +201,7 @@ func deleteNetworkHandler(ctx *web.Context) {
 		response.InternalServerError(req.Request, resp.ResponseWriter, err)
 		return
 	} else if len(ret) != 0 {
-		response.BadRequest(req.Request, resp.ResponseWriter, fmt.Errorf("The Network %s still used by some Pods, please close those Pod first", network.Name))
+		response.MethodNotAllow(req.Request, resp.ResponseWriter, fmt.Errorf("The Network %s still used by some Pods, please close those Pod first", network.Name))
 		return
 	}
 

src/server/handler_network_test.go

@@ -199,7 +199,7 @@ func (suite *NetworkTestSuite) TestDeleteNetworkFail() {
 		network   entity.Network
 		errorCode int
 	}{
-		{"NetworkNotExis", entity.Network{
+		{"NetworkDeleteFail", entity.Network{
 			ID:         bson.NewObjectId(),
 			Type:       entity.FakeNetworkType,
 			Name:       namesgenerator.GetRandomName(0),
@@ -241,7 +241,7 @@ func (suite *NetworkTestSuite) TestDeleteNetworkFail() {
 				},
 			},
 		},
-			http.StatusBadRequest},
+			http.StatusMethodNotAllowed},
 	}
 
 	//Create the Pod using the network.