Add application test for pctpairprotons

[acoussat]

Following this comment.

[SimonRit]

👍

[SimonRit]

The Python test fails because uproot is missing. Can you add an optional dependency https://github.com/RTKConsortium/RTK/blob/main/pyproject.toml#L171? Add then the option --group test on the line https://github.com/acoussat/PCT/blob/pctpairprotons-test/.github/workflows/build-test-package.yml#L51, see https://docs.openrtk.org/en/latest/INSTALLATION.html#testing.

[acoussat]

The Python test fails because uproot is missing. Can you add an optional dependency https://github.com/RTKConsortium/RTK/blob/main/pyproject.toml#L171?

Done!

Add then the option --group test on the line https://github.com/acoussat/PCT/blob/pctpairprotons-test/.github/workflows/build-test-package.yml#L51,

We checked together with @axel-grc and it does not work, and it is not how it is currently done in RTK. I have decided to follow RTK and just copy the dependencies there. Is it something we ought to change in RTK?

see https://docs.openrtk.org/en/latest/INSTALLATION.html#testing.

I copied and adapted the text from RTK, but I noticed that the command pytest test/ does not work, it should either be pytest/*.py, or rename all the test files to match test_*. I went for the first option, and I believe the doc in RTK should be corrected. @axel-grc do you want me to do it?

[SimonRit]

Good as is but it might be worth considering uproot a mandatory in the future.