Add low-level register based tests for VISA. - Githubissues

ni / grpc-device

gRPC server providing remote access to NI device driver APIs.

MIT License

69 stars 47 forks source link

Add low-level register based tests for VISA. #1013

Closed dmondrik closed 11 months ago

dmondrik commented 11 months ago

What does this Pull Request accomplish?

Add peek/poke system tests for VISA.
Modify the test setup to use ASSERT macros so the tests are skipped if the setup fails.
Modify the gRPC error message to include optional text such as the VISA resource string.
- In addition to being helpful in general, we really need it to get a better idea of how the CI is failing now.

Why should this Pull Request be merged?

Final testing for User Story 2540593: [grpc_device_server] implement the register based methods

What testing has been done?

Ran the system tests locally with --gtest_filter=*Visa*:

[==========] 41 tests from 4 test suites ran. (3279 ms total)
[  PASSED  ] 41 tests.