aws / s2n-tls

An implementation of the TLS/SSL protocols

Home Page:https://aws.github.io/s2n-tls/usage-guide/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

add "minimal test" option for binding generation

jmayclin opened this issue · comments

Problem:

The default developer experience for running the bindings generation script is failure. #4527 added a FIPs feature flag, so cargo test --all-features now requires a go dependency, and fails if it isn't installed. Most people don't have a go dependency installed, so most developers will see the ./generate.sh script fail.

Solution:

It is useful to be able to run some tests when doing local development for the generate script. We should add a --minimal-tests option or something similar to provide a rough sanity check that bindings were generated correctly.

Requirements

The default script run should either

  1. be successful (don't make --all-features the default) or
  2. have a very explicit error message "it seems like you don't have go installed, so you should run this script with --minimal-tests instead.