fixed unit tests

Signed-off-by: Harish P <harish_p4@dell.com>
opiproject · Dec 16, 2022 · 159c6f5 · 159c6f5
1 parent e9dbeb1
commit 159c6f5
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/goopicsi_test.go b/goopicsi_test.go
@@ -74,6 +74,7 @@ func (suite *GoopcsiTestSuite) TearDownTestSuite() {
 	suite.T().Log("Cleaning up resources..")
 }
 
+// TODO These test cases should be reverted with mock server implementation
 // func TestNVMeControllerConnect(t *testing.T) {
 // 	err := NVMeControllerConnect("12", "", "", 44565, "")
 // 	if err != nil {
@@ -113,6 +114,15 @@ func (suite *GoopcsiTestSuite) TearDownTestSuite() {
 // 	log.Println(resp)
 // }
 
+// func TestExposeRemoteNVMe(t *testing.T) {
+// 	subsystemID, controllerID, err := ExposeRemoteNVMe("nqn.2022-09.io.spdk:test", 10)
+// 	if err != nil {
+// 		log.Println(err)
+// 	}
+// 	log.Printf("Subsystem ID: %s", subsystemID)
+// 	log.Printf("Controller Id: %s", controllerID)
+// }
+
 func (suite *GoopcsiTestSuite) TestDeleteNVMeNamespace() {
 	// positive scenario
 	err := DeleteNVMeNamespace("1")
@@ -123,15 +133,6 @@ func (suite *GoopcsiTestSuite) TestDeleteNVMeNamespace() {
 	assert.Error(suite.T(), err, "DeleteNVMeNamespace failed")
 }
 
-// func TestExposeRemoteNVMe(t *testing.T) {
-// 	subsystemID, controllerID, err := ExposeRemoteNVMe("nqn.2022-09.io.spdk:test", 10)
-// 	if err != nil {
-// 		log.Println(err)
-// 	}
-// 	log.Printf("Subsystem ID: %s", subsystemID)
-// 	log.Printf("Controller Id: %s", controllerID)
-// }
-
 func TestGenerateHostNQN(t *testing.T) {
 	hostNQN := GenerateHostNQN()
 	log.Println(hostNQN)