go-rpm-macros

Clone
Source Code
GIT

#20 Use --with-tests when running checks also.
Merged 2 years ago by qulogic. Opened 4 years ago by qulogic.
qulogic/go-rpm-macros check-tests  into  master

file modified
+1 -1 
@@ -391,7 +391,7 @@ 
 

  checks() {
 

  local goipath="${1}"
 

  GOPATH="${workroot}${GOPATH+:${GOPATH}}" GO111MODULE=off \
 

-   golist --provided --tests --package-path ${goipath} $(expandflags) |\
 

+   golist --provided --with-tests --package-path ${goipath} $(expandflags) |\
 

      while read -r -d $'\n' dir ; do
 

        pushd "${workroot}/src/${dir}" >/dev/null
 

          echo "${dir}"

Tests will be missed if they are in the same package as a main package.

For example, the package in bug 1747622 is not running any tests, but it should. They are ignored because --tests filters out the package itself, but the tests are there.

I'd like to build new golist and check that nothing broke soon, so I'd like to incorporate this in the check as well. If no-one has any objections in the next week or so, I'm just going to merge this.

Pull-Request has been merged by qulogic
2 years ago
Metadata
None
No Tags
Changes Summary 1
+1 -1
file changed
bin/go-rpm-integration
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.