test parsing and summarisation of outputs (no errors only)

[mo-marqh]

Description

Some updates to the Github actions workflows, avoiding warnings for old actions/checkout versions and adding testing of output parsing and summarisation to the matrix of tests

Linked issues

Closes # (issue)

Type of change

• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• New tests have been added
• Tests have been modified to accommodate this change

Checklist:

• My code follows the style guidelines of this project
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes, for both debug and optimised builds

[mo-marqh]

I'm seeing sporadic action runner test failures unrelated to this change, that succeed on re-trigger.

I'm assuming there are some synchronisation issues in test running outstanding not related to this change

[mo-marqh]

i have only added the running of the summarise-vernier executable.
So, these tests will only fail if that tool raises an exception.

Do you think it's vital that we add correctness checking to these tests, beyond no errors?
I'm a bit unsure about how to do this without making fragile tests that fail too often, but i'm happy to try

[Andrew Coughtrie (andrewcoughtrie)]

Just one query but otherwise I'd say it's a good enough test for now to make sure the script at least runs.

[Andrew Coughtrie (andrewcoughtrie)]

Running this command locally I needed to put quotes around *vernier-output* otherwise I get the error

find: paths must precede expression: `drhook-vernier-output-1'
find: possible unquoted pattern after predicate `-name'?

[mo-marqh]

always happy to be more explicit with find syntax

[Andrew Coughtrie (andrewcoughtrie)]

All good now