Re: [PR PATCH] [Updated] [RFC] build-styles/go.sh: run go test -v ${go_project:-go_import_path} in do_check

[bobertlo <bobertlo@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 807 bytes --]

There is an updated pull request by bobertlo against master on the void-packages repository

https://github.com/bobertlo/void-packages go-test
https://github.com/void-linux/void-packages/pull/34496

[RFC] build-styles/go.sh: run go test -v ${go_project:-go_import_path} in do_check
There are 313 build_style=go packages, and currently 8 of them have tests defined in the package template.

I have been testing the go packages with this. So far, I have rebuilt 183 packages and there have been no errors and 64 packages have run tests successfully.

Ideally, more tests would be added (because they are a lot more common on internal libraries than command clients) but this is probably the safest default option.

A patch file from https://github.com/void-linux/void-packages/pull/34496.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-go-test-34496.patch --]
[-- Type: text/x-diff, Size: 817 bytes --]

From d4162b34f865a3ebadad0113c5800b47434ff17f Mon Sep 17 00:00:00 2001
From: Robert Lowry <bobertlo@gmail.com>
Date: Sun, 12 Dec 2021 13:58:29 -0600
Subject: [PATCH] build-styles/go.sh: run go test -v
 ${go_project:-go_import_path}

---
 common/build-style/go.sh | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/common/build-style/go.sh b/common/build-style/go.sh
index d18116ee9633..5db34db9465c 100644
--- a/common/build-style/go.sh
+++ b/common/build-style/go.sh
@@ -2,6 +2,13 @@
 # This helper is for templates for Go packages.
 #
 
+do_check() {
+	go_package=${go_package:-$go_import_path}
+	if [ "${go_mod_mode}" != "off" ] && [ -f go.mod ]; then
+		go test -v ${go_package}
+	fi
+}
+
 do_configure() {
 	# $go_import_path must be set, or we can't link $PWD into $GOSRCPATH
 	# nor build from modules