Skip to content

Commit 54aa72f

Browse files
committed
testsuite: perform build before test
Otherwise people running the task locally will have a bad time when the test either fails due to the missing executable, or far worse when they get incorrect test results because they don't realize it is using whichever random executable happened to be sitting around from the last time they did a build.
1 parent d07fdb3 commit 54aa72f

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

.github/workflows/test-go-task.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,6 @@ jobs:
8989
repo-token: ${{ secrets.GITHUB_TOKEN }}
9090
version: 3.x
9191

92-
- name: Build arduino-cli
93-
run: task go:build
94-
9592
- name: Run tests
9693
run: task go:test
9794

Taskfile.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,9 @@ tasks:
9090
9191
# Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/test-go-task/Taskfile.yml
9292
go:test:
93-
desc: Run unit tests
93+
desc: Run unit tests and part of the integration tests
94+
deps:
95+
- task: go:build
9496
dir: '{{default "./" .GO_MODULE_PATH}}'
9597
cmds:
9698
- |

0 commit comments

Comments
 (0)