search

parallaxsecond / rust-tss-esapi

TSS 2.0 Enhanced System API (ESAPI) Rust wrapper
https://docs.rs/tss-esapi/
Apache License 2.0
86 stars 51 forks source link

Adds strum as a integration test requirement. #423

Closed Superhepper closed 1 year ago

Superhepper commented 1 year ago

-Adds integration-test feature in order to conditionally add dependencies.

-Updates the TpmFormatZeroWarning so it is possible to iterate over all the items when creating integration tests.

Superhepper commented 1 year ago

Was it something like this that you had in mind @wiktor-k ?

wiktor-k commented 1 year ago

Yeah the test code got significantly smaller. There's a slight cost of setup that I think we'd pay only once and then all similar tests can just reuse what you did here. 👍

I see that the CI complains about MSRV thought :/

Superhepper commented 1 year ago

I see that the CI complains about MSRV thought :/

That was just because I thought I was going to be clever and update a dependency.