refactor: add external_jsonlib_test into go.work and CI

bytedance · Sep 20, 2022 · 60fa111 · 60fa111
1 parent a324525
commit 60fa111
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 12 deletions.
diff --git a/.github/workflows/benchmark-linux-amd64.yml b/.github/workflows/benchmark-linux-amd64.yml
@@ -23,8 +23,11 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-go-
 
-      - name: Benchmark
+      - name: Benchmark sonic
         run: sh bench.sh
 
+      # - name: Benchmark third-party
+      #   run: go test -benchmem -run=^$ -bench . -v ./generic_test
+
       # - name: Diff
       #   run: ./bench.py -b '"^Benchmark.*Sonic"' -c
diff --git a/.github/workflows/push-check-go118.yml b/.github/workflows/push-check-go118.yml
@@ -21,10 +21,9 @@ jobs:
             ${{ runner.os }}-go-
 
       - name: Unit Test
-        run: GOMAXPROCS=4 go test -v -gcflags=-d=checkptr=0 -race ./...
+        run: |
+          GOMAXPROCS=4 go test -v -gcflags=-d=checkptr=0 -race ./...
+          GOMAXPROCS=4 go test -v -gcflags=-d=checkptr=0 -race ./external_jsonlib_test/...
       
       - name: Generic Test
-        run: GOMAXPROCS=4 go test -v -gcflags=-d=checkptr=0 -race ./generic_test
-
-      - name: Benchmark
-        run: go test -benchmem -run=^$ -bench . -v ./generic_test
+        run: GOMAXPROCS=4 go test -v -gcflags=-d=checkptr=0 -race ./generic_test
diff --git a/.github/workflows/push-check-linux-amd64.yml b/.github/workflows/push-check-linux-amd64.yml
@@ -24,4 +24,7 @@ jobs:
             ${{ runner.os }}-go-
 
       - name: Unit Test
-        run: go test -v -gcflags=-d=checkptr=0 ./...
+        run: |
+          go test -v -gcflags=-d=checkptr=0 ./...
+          cd ./external_jsonlib_test
+          go test -v -gcflags=-d=checkptr=0 ./...
diff --git a/external_jsonlib_test/go.work b/external_jsonlib_test/go.work
diff --git a/go.work b/go.work
@@ -4,4 +4,5 @@ use (
 	.
 	./generic_test
 	./fuzz
+	./external_jsonlib_test
 )