[add] dict tests require flush

Author: filipecosta90

Makefile

@@ -4,7 +4,7 @@ GOCMD=GO111MODULE=on go
 GOBUILD=$(GOCMD) build
 GOINSTALL=$(GOCMD) install
 GOCLEAN=$(GOCMD) clean
-GOTEST=$(GOCMD) test
+GOTEST=$(GOCMD) test -count=1
 GOGET=$(GOCMD) get
 GOMOD=$(GOCMD) mod
 

redisearch/client_test.go

@@ -9,6 +9,12 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
+func flush(c *Client) (err error) {
+	conn := c.pool.Get()
+	defer conn.Close()
+	return conn.Send("FLUSHALL")
+}
+
 func TestClient_Get(t *testing.T) {
 
 	c := createClient("test-get")
@@ -134,7 +140,8 @@ func TestClient_MultiGet(t *testing.T) {
 
 func TestClient_DictAdd(t *testing.T) {
 	c := createClient("TestClient_DictAdd_Index")
-	c.Drop()
+	// dict tests require flushall
+	flush(c)
 
 	type fields struct {
 		pool ConnPool
@@ -153,8 +160,8 @@ func TestClient_DictAdd(t *testing.T) {
 	}{
 		{"empty-error", fields{pool: c.pool, name: c.name}, args{"dict1", []string{}}, 0, true},
 		{"1-term", fields{pool: c.pool, name: c.name}, args{"dict1", []string{"term1"}}, 1, false},
-		{"2nd-time-term", fields{pool: c.pool, name: c.name}, args{"dict1", []string{"term1"}}, 0, false},
-		{"multi-term", fields{pool: c.pool, name: c.name}, args{"dict1", []string{"t1", "t2", "t3", "t4", "t5"}}, 5, false},
+		{"2nd-time-term", fields{pool: c.pool, name: c.name}, args{"dict1", []string{"term1","term1"}}, 1, false},
+		{"multi-term", fields{pool: c.pool, name: c.name}, args{"dict-multi-term", []string{"t1", "t2", "t3", "t4", "t5"}}, 5, false},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -170,6 +177,7 @@ func TestClient_DictAdd(t *testing.T) {
 			if gotNewTerms != tt.wantNewTerms {
 				t.Errorf("DictAdd() gotNewTerms = %v, want %v", gotNewTerms, tt.wantNewTerms)
 			}
+			i.DictDel(tt.args.dictionaryName, tt.args.terms)
 		})
 	}
 }