Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow to write tests with pressure devices #2178

Open
t-b opened this issue Jul 11, 2024 · 1 comment
Open

Allow to write tests with pressure devices #2178

t-b opened this issue Jul 11, 2024 · 1 comment
Labels
CodeQuality Issue Review 2025 Review of all open issues PressureControl Testing Related to Testing itself

Comments

@t-b
Copy link
Collaborator

t-b commented Jul 11, 2024

We don't yet have tests for the pressure control code, the line coverage for MIES_PressureControl.ipf is therefore only 24% 1. As recent issues have shown #2124, and also the additional fix required for #2137 (comment), this ain't good.

For being able to write tests using pressure control we would need a NI USB 6001 at all CI machines with hardware attached. If something has to be bought we might also buy three to prepare for a future separate machine for ITC1600 and NI.

@t-b t-b added PressureControl CodeQuality Testing Related to Testing itself labels Jul 11, 2024
@timjarsky timjarsky added the Issue Review 2025 Review of all open issues label Feb 3, 2025
@timjarsky
Copy link
Collaborator

Tim to add NI USB 6001 to new CI machines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CodeQuality Issue Review 2025 Review of all open issues PressureControl Testing Related to Testing itself
Projects
None yet
Development

No branches or pull requests

2 participants