[Bug]: step-cli uses gotestsum but does not build it

[dermotbradley]

Steps to Reproduce

Hi. I'm the Alpine package maintainer for step-cli (and step-certificates).

When step-cli 0.23.0 was released and I packaged it up I found that "make test" was failing whereas it had worked fine in previous releases.

This was due to the change to use "gotestsum". However in a "make build", "make test" sequence gotestsum is never built and that's why the tests fail.

I had to make a 2 line change (including remove a $(GOFLAGS) reference) to make/common.mk to fix this: https://git.alpinelinux.org/aports/tree/community/step-cli/01-fix-tests.patch

Your Environment

• OS - Alpine Linux
• step CLI Version - 0.23.0

Expected Behavior

"make test" to work as in previous releases.

Actual Behavior

"make test" failed as gotestsum was not present to run.

Additional Context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

[dopey]

Hey @dermotbradley 👋. Thanks for opening the issue!

And apologies that it has taken us so long to get back to you :[

Would it be possible to call make bootstrap -> make build -> make test (adding make bootstrap)? We prefer to keep the necessary install in the bootstrap target, if possible.