X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=test%2Ffolders_test.go;h=8f6023a6b5f005a02047bc482d5099f071a19e33;hb=5dc2ff003106f0ced38caadb06033f24c792f9b9;hp=42db7c2821a5dcc3832b40f6c27065572e25f6f0;hpb=d3a41ad067e7752a835c71d9f91ddd7230cd648a;p=src%2Fxds%2Fxds-server.git

diff --git a/test/folders_test.go b/test/folders_test.go
index 42db7c2..8f6023a 100644
--- a/test/folders_test.go
+++ b/test/folders_test.go
@@ -22,14 +22,16 @@ import (
 	"regexp"
 	"testing"
 
-	"gerrit.automotivelinux.org/gerrit/src/xds/xds-server/lib/xsapiv1"
-	"github.com/stretchr/testify/assert"
+	"gerrit.automotivelinux.org/gerrit/src/xds/xds-server.git/lib/xsapiv1"
+	"github.com/stretchr/testify/require"
 )
 
 func TestFolders(t *testing.T) {
+	/*init: check there is no folder*/
+	Debug(t, "check there is no folder")
 	var cfgArray []xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArray))
-	assert.Equal(t, len(cfgArray), 0)
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArray))
+	require.Equal(t, len(cfgArray), 0)
 
 	fPrj := xsapiv1.FolderConfig{
 		Label:      "testproject",
@@ -41,50 +43,57 @@ func TestFolders(t *testing.T) {
 		},
 	}
 	var cfg xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
-	assert.NotNil(t, cfg)
+	Debugf(t, "create folder: \n%v", fPrj)
+	require.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, cfg)
+	Debugf(t, "result folder: \n%v", cfg)
 
 	isCfgPrjMatch := func(fPrj xsapiv1.FolderConfig, cfg xsapiv1.FolderConfig) {
 		re := regexp.MustCompile("^[0-9a-z]+-[0-9a-z]+-[0-9a-z]+-[0-9a-z]+-[0-9a-z]+$")
-		assert.True(t, re.MatchString(cfg.ID)) //ID
-		assert.Equal(t, cfg.Label, fPrj.Label) //Label
-		assert.Equal(t, cfg.ClientPath, fPrj.ClientPath)
-		assert.Equal(t, cfg.Type, fPrj.Type)
-		assert.Equal(t, cfg.Status, "Enable")
-		assert.Equal(t, cfg.IsInSync, true)
-		assert.Equal(t, len(cfg.DefaultSdk), 0)
-		assert.Equal(t, fPrj.ClientData, cfg.ClientData)
-		assert.Equal(t, fPrj.DataPathMap.ServerPath, cfg.DataPathMap.ServerPath)
+		require.True(t, re.MatchString(cfg.ID)) //ID
+		require.Equal(t, cfg.Label, fPrj.Label) //Label
+		require.Equal(t, cfg.ClientPath, fPrj.ClientPath)
+		require.Equal(t, cfg.Type, fPrj.Type)
+		require.Equal(t, cfg.Status, "Enable")
+		require.Equal(t, cfg.IsInSync, true)
+		require.Equal(t, len(cfg.DefaultSdk), 0)
+		require.Equal(t, fPrj.ClientData, cfg.ClientData)
+		require.Equal(t, fPrj.DataPathMap.ServerPath, cfg.DataPathMap.ServerPath)
 	}
 	isCfgPrjMatch(fPrj, cfg)
 	var cfg2 xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Get("/folders/"+cfg.ID, &cfg2))
+	require.Nil(t, HTTPCli.Get("/folders/"+cfg.ID, &cfg2))
 	isCfgPrjMatch(fPrj, cfg2)
 
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArray))
-	assert.Equal(t, len(cfgArray), 1)
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArray))
+	require.Equal(t, len(cfgArray), 1)
 
 	//call with the same uid create error
-	assert.NotNil(t, HTTPCli.Post("/folders", cfg, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", cfg, &cfg))
 
 	/*create/delete folders*/
 	var cfgArrayBis []xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
-	assert.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArray))
-	assert.Equal(t, len(cfgArray), 3)
-	assert.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[1].ID, &cfg))
-	assert.Equal(t, cfg, cfgArray[1])
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
-	assert.Equal(t, len(cfgArrayBis), 2)
-	assert.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[0].ID, &cfg))
-	assert.Equal(t, cfg, cfgArray[0])
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
-	assert.Equal(t, len(cfgArrayBis), 1)
-	assert.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[2].ID, &cfg))
-	assert.Equal(t, cfg, cfgArray[2])
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
-	assert.Equal(t, len(cfgArrayBis), 0)
+	require.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "create folder with id=%v", cfg.ID)
+	require.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "create folder with id=%v", cfg.ID)
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArray))
+	require.Equal(t, len(cfgArray), 3)
+	require.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[1].ID, &cfg))
+	Debugf(t, "delete folder with id=%v", cfg.ID)
+	require.Equal(t, cfg, cfgArray[1])
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
+	require.Equal(t, len(cfgArrayBis), 2)
+	require.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[0].ID, &cfg))
+	Debugf(t, "delete folder with id=%v", cfg.ID)
+	require.Equal(t, cfg, cfgArray[0])
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
+	require.Equal(t, len(cfgArrayBis), 1)
+	require.Nil(t, HTTPCli.Delete("/folders/"+cfgArray[2].ID, &cfg))
+	Debugf(t, "delete folder with id=%v", cfg.ID)
+	require.Equal(t, cfg, cfgArray[2])
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArrayBis))
+	require.Equal(t, len(cfgArrayBis), 0)
 }
 
 func TestFoldersEmptyValues(t *testing.T) {
@@ -99,21 +108,24 @@ func TestFoldersEmptyValues(t *testing.T) {
 	}
 	var cfg xsapiv1.FolderConfig
 	/*ServerPath is empty*/
-	assert.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "error while creating folder with empty serverpath \n%v", fPrj)
 
 	fPrj.DataPathMap.ServerPath = logDir + "sameserverpath"
 	fPrj.ClientPath = ""
 	/*ClientPath is Empty*/
-	assert.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "error while creating folder with empty clientpath \n%v", fPrj)
 
 	fPrj.ClientPath = "logDir"
 	fPrj.Type = ""
 	/*Type is empty*/
-	assert.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "error while creating folder with empty type \n%v", fPrj)
 
 	var cfgArray []xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArray))
-	assert.Equal(t, len(cfgArray), 0)
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArray))
+	require.Equal(t, len(cfgArray), 0)
 }
 
 func TestFoldersPathMapConfig(t *testing.T) {
@@ -129,7 +141,8 @@ func TestFoldersPathMapConfig(t *testing.T) {
 	}
 	var cfg xsapiv1.FolderConfig
 	/*file not present*/
-	assert.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "error while creating folder with no checkfile \n%v", fPrj)
 
 	var checkFileClient = fPrj.ClientPath + "/checkfile"
 	var checkFileServer = fPrj.DataPathMap.ServerPath + "/checkfile"
@@ -140,32 +153,35 @@ func TestFoldersPathMapConfig(t *testing.T) {
 	fPrj.DataPathMap.CheckContent = "CheckContent From Client\n"
 	file, err := os.OpenFile(checkFileClient, os.O_CREATE|os.O_RDWR, 0644)
 	if err != nil {
-		t.Log(err)
+		Debug(t, err)
 	}
 	if err := os.Symlink(checkFileClient, checkFileServer); err != nil {
-		t.Log(err)
+		Debug(t, err)
 	}
 	/*file content differ*/
-	assert.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.NotNil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "error while creating folder with different checkfiles \n%v", fPrj)
 
 	/*write same message*/
 	if _, err := file.WriteString(fPrj.DataPathMap.CheckContent); err != nil {
-		t.Log(err)
+		Debug(t, err)
 	}
-	assert.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	require.Nil(t, HTTPCli.Post("/folders", fPrj, &cfg))
+	Debugf(t, "create folder with same checkfiles \n%v", fPrj)
 
 	/*check server msg: ServerUID needed*/
+	Debugf(t, "check server msg")
 	var APIcfg xsapiv1.APIConfig
-	assert.Nil(t, HTTPCli.Get("/config", &APIcfg))
+	require.Nil(t, HTTPCli.Get("/config", &APIcfg))
 	msg := "Pathmap checked message written by xds-server ID: " + APIcfg.ServerUID + "\n"
 	data, err := ioutil.ReadAll(file)
 	if err != nil {
-		t.Log(err)
+		Debug(t, err)
 	}
-	assert.Equal(t, msg, string(data))
+	require.Equal(t, msg, string(data))
 
-	assert.Nil(t, HTTPCli.Delete("/folders/"+cfg.ID, &cfg))
+	require.Nil(t, HTTPCli.Delete("/folders/"+cfg.ID, &cfg))
 	var cfgArray []xsapiv1.FolderConfig
-	assert.Nil(t, HTTPCli.Get("/folders", &cfgArray))
-	assert.Equal(t, len(cfgArray), 0)
+	require.Nil(t, HTTPCli.Get("/folders", &cfgArray))
+	require.Equal(t, len(cfgArray), 0)
 }