Restructure `epics/test_signals.py` - Githubissues

bluesky / ophyd-async

Hardware abstraction for bluesky written using asyncio

https://blueskyproject.io/ophyd-async

BSD 3-Clause "New" or "Revised" License

7 stars 21 forks source link

Restructure `epics/test_signals.py` #385

Open coretl opened 2 weeks ago

coretl commented 2 weeks ago

At the moment we have:

A parameterized fixture that provides an IOC, with records either prefixed CA or PVA
A few tests that use this and are parameterized to run on both
A lot of tests that skip the CA parts

I suggest we refactor so:

There is a single IOC that loads test_records twice with both CA and PVA prefixes
The tests that run with both have are further parameterized with CA and PVA param
The others just run using the IOC with PVA prefix

This will mean breaking the massive test_backend_get_put_monitor test matrix into two: PVA only tests, and CA and PVA tests