On 20.10.21 09:05, Michael Adler wrote:
> The bg_setenv/bg_printenv command-line interface is currently untested,
> which makes it error-prone to extend the current code.
> 
> This commit introduces basic tests for the bg_setenv/bg_printenv
> command-line interface.
> 
> The following new test dependencies are introduced (included as git
> submodules, as suggested in the docs [1]):
> 
> - bats-core [2]: MIT
> - bats-support [3]: CC0-1.0
> - bats-assert [4]: CC0-1.0
> 
> Strictly speaking, only bats-core is a must-have, but [3] and [4] make
> it much more friendly to use (and they are being used in this commit as
> well).
> 

Your argumentation why to introduce another test framework and why one
can't replace the other should go here as well. To some degree it was in
the cover letter, but here it would be easier to find later on via git.

BUT: You do not explain to us anywhere why installing bats from a
distro, Debian or Ubuntu in CI, is not sufficient. Why do we need to
import it?

Jan

-- 
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux

-- 
You received this message because you are subscribed to the Google Groups "EFI 
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/efibootguard-dev/b59cdba5-6a1f-e65d-3b31-45badcb95f57%40siemens.com.

Reply via email to