tiny fix

pingcap · Feb 27, 2019 · 042ce32 · 042ce32
1 parent a58e58c
commit 042ce32
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/tests/e2e/create.go b/tests/e2e/create.go
@@ -60,13 +60,14 @@ type Response struct {
 func testCreate(ns, clusterName string) {
 	By(fmt.Sprintf("When create the TiDB cluster: %s/%s", ns, clusterName))
 	instanceName := getInstanceName(ns, clusterName)
-	err := createSecret(ns, clusterName)
-	Expect(err).NotTo(HaveOccurred())
 
 	cmdStr := fmt.Sprintf("helm install /charts/tidb-cluster -f /tidb-cluster-values.yaml"+
 		" -n %s --namespace=%s --set clusterName=%s,tidb.passwordSecretName=%s",
 		instanceName, ns, clusterName, ns+"-"+clusterName)
-	_, err = execCmd(cmdStr)
+	_, err := execCmd(cmdStr)
+	Expect(err).NotTo(HaveOccurred())
+
+	err = createSecret(ns, clusterName)
 	Expect(err).NotTo(HaveOccurred())
 
 	By("Then all members should running")

diff --git a/tests/e2e/test_helper.go b/tests/e2e/test_helper.go
@@ -101,7 +101,7 @@ func clearOperator() error {
 			return err
 		}
 		err = kubeCli.CoreV1().Secrets(fixture.ns).Delete(fixture.ns+"-"+fixture.clusterName, nil)
-		if err != nil && apierrs.IsNotFound(err) {
+		if err != nil && !apierrs.IsNotFound(err) {
 			return err
 		}
 	}