test(git): add testcase of function git.InstallSubCmd

before: only failure testcase
afeter: add testdata/testcmd file to test normal testcase

Author: robertzwj

Makefile

@@ -29,6 +29,7 @@ commit:
 	@go run main.go
 
 build: clean
+	@echo "========> Building binary"
 	go build ${GO_BUILD_FLAGS} -o ${BIN}
 
 dryRun: build

git/install_test.go

@@ -1,13 +1,15 @@
 /*
  * @Author: robert zhang <robertzhangwenjie@gmail.com>
  * @Date: 2022-08-06 16:52:22
- * @LastEditTime: 2022-08-07 10:55:53
+ * @LastEditTime: 2022-09-03 17:24:07
  * @LastEditors: robert zhang
  * @Description:
  */
 package git
 
 import (
+	"os"
+	"path/filepath"
 	"testing"
 )
 
@@ -20,7 +22,6 @@ func TestInstallSubCmd(t *testing.T) {
 	testcases := []struct {
 		name    string
 		args    args
-		want    string
 		wantErr bool
 	}{
 		{
@@ -29,18 +30,28 @@ func TestInstallSubCmd(t *testing.T) {
 				filePath:   "/123/456",
 				subCmdName: "123",
 			},
-			want:    "",
 			wantErr: true,
 		},
+		{
+			name: "normalFile",
+			args: args{
+				filePath:   filepath.Join("testdata", "testcmd"),
+				subCmdName: "testcmd",
+			},
+			wantErr: false,
+		},
 	}
 
 	for _, tc := range testcases {
 		t.Run(tc.name, func(t *testing.T) {
 			dstFilePath, err := InstallSubCmd(tc.args.filePath, tc.args.subCmdName)
-			if (err != nil) != tc.wantErr && dstFilePath != tc.want {
-				t.Errorf("InstallSubCmd() error = %v,wantErr %v", err, tc.wantErr)
+			if err == nil {
+				os.Remove(dstFilePath)
 			}
+			if (err != nil) != tc.wantErr {
+				t.Errorf("InstallSubCmd() error = %v,wantErr %v ", err, tc.wantErr)
+			}
+
 		})
 	}
-
 }

git/testdata/testcmd

@@ -0,0 +1,7 @@
+###
+ # @Author: robert zhang <robertzhangwenjie@gmail.com>
+ # @Date: 2022-09-03 16:50:19
+ # @LastEditTime: 2022-09-03 17:26:03
+ # @LastEditors: robert zhang
+ # @Description: testfile for testing installCmd
+###