8.27.3

[bimac]

• tested on dev rig
• does not affect task logic
• passes unit-tests
• reviewed

[coveralls]

coverage: 47.388% (-0.3%) from 47.663%
when pulling 1b5c220 on iblrigv8dev
into b6bd70f on iblrigv8.

[oliche]

It's hard to assess the difficulty/usefulness of testing the pyspin functions, but there is no test here, is there ?

Also we saw that the entrypoints need testing, maybe with some form of integration test.

[bimac]

It's hard to assess the difficulty/usefulness of testing the pyspin functions, but there is no test here, is there ?

There are currently no tests for the PySpin functions, correct. It may be possible to do this with plenty of mocking, but since PySpin is mostly a wrapper to the Spinnaker C++ SDK it would be very cumbersome and the benefits questionable. PySpin doesn't currently affect the core functionality of IBLRIG, so I would give testing a lower priority in this case.

Also we saw that the entrypoints need testing, maybe with some form of integration test.

Yes, I'll look into this.

[bimac]

Also we saw that the entrypoints need testing, maybe with some form of integration test.

Moving to pydantic_settings should make it easier to test the CLI argument parsing: #770