lowRISC / sonata-system

A full micro-controller system utilizing the CHERIoT Ibex core, part of the Sunburst project funded by UKRI
Apache License 2.0
29 stars 17 forks source link

Check results of all hyperram tests, and not just the last one #205

Closed AlexJones0 closed 1 month ago

AlexJones0 commented 1 month ago

The hyperram test check fix only checked the result of the last test, and not each individual test. As a result, the hyperram test would only fail on CI if the last test specifically failed.

I've opted to track test failure via a separate Boolean variable rather than incrementing the failures variable each test, so that we retain per-test failure granularity, as otherwise failures would cascade through tests and provide less useful information in CI.

Also fixes a typo.