-
Notifications
You must be signed in to change notification settings - Fork 48
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
Check IS-11 control advertisement like other test suites do #806
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ISSUE: "[IS-11] Fix test 0.1 to look at all the devices exposing the IS-11 API #791" was supposed to fix that. |
@alabou that's the reported issue, this resolves the issue, and in the same way as IS-05 and IS-08 test suites. Is there a problem with that? |
If there is only one good way of doing this check then the code should not be duplicated for each API but a common function should be used. The test plan has been updated last week to v0.9 and should be matching the IS-05, IS-08 code and Matrox IS-11 code. As discussed in the meeting Nikita will take care of using a common function in all the test suites without duplicating the code. |
I'd take correct code that fixes a demonstrated bug... |
Co-authored-by: N-Nagorny <nikiton-n@mail.ru>
Add common method to test Device control advertisements
Fixes the test that searches for IS-11 control advertisement in IS-04 APIs. For instance, this test fails with the following
controls
: