Commit c679381

committed

Add infrastructure for defining -ldflags for go test run

The `go:build` task has an `LDFLAGS` taskfile template variable, which is currently used by some projects to inject the versioning information while building via an `-ldflags` flag. `go test` has undocumented support for an `-ldflags` flag as well. Projects might find it useful to be able to set the value of string variables in the code while running the tests. To provide support by the task for this use case, a `TEST_LDFLAGS` taskfile template variable is added to the `go test` command in the `go:test` task, as well as empty definition of the variable. Even though this project doesn't currently have any need for the capability, it does no harm and brings the task into alignment with the standardized "template".

1 parent 7ab3a17 commit c679381Copy full SHA for c679381

File tree

1 file changed

-0

lines changed

Taskfile.yml

1 file changed

-0

lines changed

`‎Taskfile.yml‎`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,8 @@ vars:`
`39`	`39`	`-X {{.CONFIGURATION_PACKAGE}}.Commit={{.COMMIT}}`
`40`	`40`	`-X {{.CONFIGURATION_PACKAGE}}.Timestamp={{.TIMESTAMP}}`
`41`	`41`	`'`
	`42`	+ # `-ldflags` flag to use for `go test` command
	`43`	`+ TEST_LDFLAGS:`
`42`	`44`
`43`	`45`	`tasks:`
`44`	`46`	`build:`
`@@ -225,6 +227,7 @@ tasks:`
`225`	`227`	`-run '{{default ".*" .GO_TEST_REGEX}}' \`
`226`	`228`	`{{default "-timeout 10m -coverpkg=./... -covermode=atomic" .GO_TEST_FLAGS}} \`
`227`	`229`	`-coverprofile=coverage_unit.txt \`
	`230`	`+ {{.TEST_LDFLAGS}} \`
`228`	`231`	`{{default .DEFAULT_GO_PACKAGES .GO_PACKAGES}}`
`229`	`232`
`230`	`233`	`# Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/test-go-integration-task/Taskfile.yml`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit c679381

File tree

1 file changed

1 file changed

`‎Taskfile.yml‎`

0 commit comments