Merge pull request #117 from harshavardhana/pr_out_fix_build_issue

pkg/api/minioapi/minioapi.go

@@ -127,13 +127,17 @@ func (server *minioApi) putBucketHandler(w http.ResponseWriter, req *http.Reques
 	vars := mux.Vars(req)
 	bucket := vars["bucket"]
 	err := server.storage.StoreBucket(bucket)
-	if err != nil {
+	switch err := err.(type) {
+	case nil:
+		w.Header().Set("Server", "Minio")
+		w.Header().Set("Connection", "close")
+	case mstorage.BucketNameInvalid:
 		w.WriteHeader(http.StatusBadRequest)
 		w.Write([]byte(err.Error()))
-		return
+	case mstorage.BucketExists:
+		w.WriteHeader(http.StatusConflict)
+		w.Write([]byte(err.Error()))
 	}
-	w.Header().Set("Server", "Minio")
-	w.Header().Set("Connection", "close")
 }
 
 func (server *minioApi) getObjectHandler(w http.ResponseWriter, req *http.Request) {

pkg/api/minioapi/minioapi_test.go

@@ -241,13 +241,10 @@ func (s *MySuite) TestPutObject(c *C) {
 	testServer := httptest.NewServer(httpHandler)
 	defer testServer.Close()
 
-	err := storage.StoreBucket("bucket")
-	c.Assert(err, IsNil)
-
 	objects, isTruncated, err := storage.ListObjects("bucket", "", 1000)
 	c.Assert(len(objects), Equals, 0)
 	c.Assert(isTruncated, Equals, false)
-	c.Assert(err, IsNil)
+	c.Assert(err, Not(IsNil))
 
 	date1 := time.Now()