search

eic / EICrecon

EIC Reconstruction - JANA based
https://eic.github.io/EICrecon
GNU Lesser General Public License v3.0
6 stars 29 forks source link

fix(ci): add src/tests plugins as CI jobs #1386

Closed wdconinc closed 6 months ago

wdconinc commented 6 months ago

Briefly, what does this PR introduce?

We don't test what's in src/tests, to a large degree (exception is the catch unit tests), and this leads to code issues, e.g. #1385.

This PR adds the test plugins in src/tests to a CI job. It won't actually catch #1385 until we stop silently eating exceptions... It catches #1385 now, https://github.com/eic/EICrecon/actions/runs/8726213766/job/23941043650#step:8:676.

What kind of change does this PR introduce?

Please check if this PR fulfills the following:

Does this PR introduce breaking changes? What changes might users need to make to their code?

No.

Does this PR change default behavior?

No.

github-actions[bot] commented 6 months ago

Capybara summary for PR 1386