Merge pull request #107 from linkernetworks/johnlin/invalidIPforStorage

add test for invalid NFS IP
linkernetworks · Jul 17, 2018 · e27451e · e27451e
2 parents 48acd3c + fbdea81
commit e27451e
Showing 1 changed file with 26 additions and 3 deletions.
diff --git a/src/server/handler_storage_test.go b/src/server/handler_storage_test.go
@@ -122,18 +122,41 @@ func (suite *StorageTestSuite) TestCreateStorageFail() {
 			Fake: &entity.FakeStorage{
 				FakeParameter: "Yo",
 				IWantFail:     true,
-			}},
+			},
+		},
 			http.StatusInternalServerError},
+		{"inValidNFSIP", entity.Storage{
+			Type:             entity.FakeStorageType,
+			Name:             namesgenerator.GetRandomName(1),
+			StorageClassName: namesgenerator.GetRandomName(0),
+			IP:               "256.256.256.256",
+			PATH:             "/myspace",
+			Fake: &entity.FakeStorage{
+				FakeParameter: "Yo",
+			},
+		},
+			http.StatusBadRequest},
+		{"lackStorageName", entity.Storage{
+			Type:             entity.FakeStorageType,
+			StorageClassName: namesgenerator.GetRandomName(0),
+			IP:               "256.256.256.256",
+			PATH:             "/myspace",
+			Fake: &entity.FakeStorage{
+				FakeParameter: "Yo",
+			},
+		},
+			http.StatusBadRequest},
 		{"StorageTypeError", entity.Storage{
 			Name:             namesgenerator.GetRandomName(0),
 			StorageClassName: namesgenerator.GetRandomName(1),
-			Type:             "non-exist",
+			Type:             "none-exist",
 			IP:               "192.168.5.100",
 			PATH:             "/myspace",
 			Fake: &entity.FakeStorage{
 				FakeParameter: "Yo",
 				IWantFail:     true,
-			}},
+			},
+		},
 			http.StatusBadRequest},
 	}