[ ] ~I've added tests applicable for this pull request~
What does this Pull Request accomplish?
In a recent version of the NI-DMM, the reported serial of a simulated DMM was intentionally changed. This causes the nidmm system test test_vi_string_attribute to fail in our internal test suite and will cause failures in PR Checks when we update the version we test against.
This change updates the test to rely on instrument_model, instead of serial_number. They are both read-only properties and we do not expect the reported value for instrument_model to change in the future.
List issues fixed by this Pull Request below, if any.
Fix #2012
What testing has been done?
Ran the nidmm system tests on a local machine with a dev version NI-DMM 2023Q4 installed and confirmed that the test failed.
Updated the test and ran again, confirming that the test passed.
PR Checks will confirm that the updated test passes with the driver runtime that we currently test against
What does this Pull Request accomplish?
In a recent version of the NI-DMM, the reported serial of a simulated DMM was intentionally changed. This causes the nidmm system test
test_vi_string_attribute
to fail in our internal test suite and will cause failures in PR Checks when we update the version we test against.This change updates the test to rely on
instrument_model
, instead ofserial_number
. They are both read-only properties and we do not expect the reported value forinstrument_model
to change in the future.List issues fixed by this Pull Request below, if any.
What testing has been done?