Refactor action

.github/workflows/test.yaml

@@ -4,6 +4,11 @@ on:
     branches: [ master ]
   pull_request:
     branches: [ master ]
+
+concurrency:
+  group: ${{ github.head_ref }}
+  cancel-in-progress: true
+
 jobs:
   go-client:
     runs-on: ubuntu-latest
@@ -14,10 +19,48 @@ jobs:
         with:
           go-version: '1.13'
       - name: Test with GO test
-        run: make ci
+        run: |
+          make up
+          sleep 10
+          make test
+      - name: down
+        if: always()
+        run: |
+          make down
+  go-client-ssl:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup go
+        uses: actions/setup-go@v1
+        with:
+          go-version: '1.13'
       - name: Test SSL connection with CA
-        run: make ssl-test
+        run: |
+          make up-ssl
+          make ssl-test
       - name: Test SSL connection self-signed
-        run: make ssl-test-self-signed
+        run: |
+          make up-ssl
+          make ssl-test-self-signed
+      - name: down
+        if: always()
+        run: |
+          make down
+  go-client-example:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup go
+        uses: actions/setup-go@v1
+        with:
+          go-version: '1.13'
       - name: Run examples
-        run: make run-examples
+        run: |
+          make up
+          sleep 10
+          make run-examples
+      - name: down
+        if: always()
+        run: |
+          make down  

Makefile

@@ -1,4 +1,4 @@
-.PHONY: build test fmt ci ssl-test run-examples
+.PHONY: build test fmt up up-ssl down ssl-test run-examples
 
 default: build
 
@@ -12,33 +12,25 @@ test:
 
 fmt:
 	go fmt
+up:
+	cd ./nebula-docker-compose && docker-compose up -d
 
-ci:
-	cd ./nebula-docker-compose && docker-compose up -d && \
-	sleep 10 && \
-	cd .. && \
-	go test -v -race; \
+up-ssl:
+	cd ./nebula-docker-compose && enable_ssl=true docker-compose up -d
+
+down:
 	cd ./nebula-docker-compose && docker-compose down -v
 
 ssl-test:
-	cd ./nebula-docker-compose && enable_ssl=true docker-compose up -d && \
-	sleep 10 && \
-	cd .. && \
-	go test -v -run TestSslConnection; \
-	cd ./nebula-docker-compose && docker-compose down -v
+	ssl_test=true go test -v -run TestSslConnection;
 
 ssl-test-self-signed:
 	cd ./nebula-docker-compose && enable_ssl=true docker-compose up -d && \
 	sleep 10 && \
 	cd .. && \
-	ssl_test=true go test -v -run TestSslConnection; \
-	cd ./nebula-docker-compose && docker-compose down -v
+	self_signed=true go test -v -run TestSslConnection;
 
 run-examples:
-	cd ./nebula-docker-compose && enable_ssl=true docker-compose up -d && \
-	sleep 10 && \
-	cd .. && \
 	go run basic_example/graph_client_basic_example.go && \
 	go run gorountines_example/graph_client_goroutines_example.go && \
-	go run json_example/parse_json_example.go && \
-	cd ./nebula-docker-compose && docker-compose down -v
+	go run json_example/parse_json_example.go

client_test.go

@@ -259,7 +259,8 @@ func TestServiceDataIO(t *testing.T) {
 	// test base type
 	{
 		query :=
-			"FETCH PROP ON person \"Bob\" YIELD person.name, person.age, person.grade," +
+			"FETCH PROP ON person \"Bob\" YIELD vertex as VertexID," +
+				"person.name, person.age, person.grade," +
 				"person.friends, person.book_num, person.birthday, " +
 				"person.start_school, person.morning, " +
 				"person.property, person.is_girl, person.child_name, " +

result_set_test.go

@@ -117,7 +117,7 @@ func TestAsDedupList(t *testing.T) {
 		UVal: &nebula.NSet{Values: valList},
 	}
 	valWrap := ValueWrapper{&value, testTimezone}
-	assert.Equal(t, "[\"elem1\", \"elem2\", \"elem3\"]", valWrap.String())
+	assert.Equal(t, "{\"elem1\", \"elem2\", \"elem3\"}", valWrap.String())
 	assert.Equal(t, true, valWrap.IsSet())
 
 	res, _ := valWrap.AsList()