[SLOF] [PATCH 4/4] Add testprograms

Thomas Huth thuth at redhat.com
Thu Jul 1 16:38:06 AEST 2021


On 01/07/2021 06.11, Alexey Kardashevskiy wrote:
> 
> 
> On 30/06/2021 22:26, Stefan Berger wrote:
>>
>> On 6/29/21 11:10 PM, Alexey Kardashevskiy wrote:
>>> What does this actually test? It runs sha384/etc but it is not comparing 
>>> the outcome with known good values or anything like this.
>>>
>> I visually compared the results against the expected results from test 
>> vectors or results from  command line output of echo -n "..." | sha512sum 
>> for example.
> 
> if would make more sense if you hardcoded those outputs to the test and 
> compared, then the next reader won't have questions and will know exactly 
> what it tests.

Bonus points for turning these into proper unit tests that could be run in 
github actions or another CI system (Travis-CI provides free ppc64le runners 
if that's of any help).

  Thomas



More information about the SLOF mailing list