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.

Sign up for GitHub

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

Use a valid command id in TC_IDM_1_4.py #33556

Merged
merged 2 commits into from
May 24, 2024
Merged

Use a valid command id in TC_IDM_1_4.py #33556

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
58a14b0
Use a valid command id in TC_IDM_1_4.py
Apollon77 May 22, 2024
65ef6bb
address review feedback
Apollon77 May 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/python_testing/TC_IDM_1_4.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,11 @@ async def test_TC_IDM_1_4(self):
cap_for_batch_commands = 100
number_of_commands_to_send = min(max_paths_per_invoke + 1, cap_for_batch_commands)

invalid_command_id = 0xffff_ffff
# Use a valid (according to MEI definition) command-id (in this case belonging to Test Vendor MC with prefix 0xfff4)
# which should never be existing on a prodiction device. We use this decodable id to prevent hitting issues with the
# specification being not clearly defined in respect to decoding vs processing the invoke requests.
invalid_command_id = 0xfff4_00ff
tehampson marked this conversation as resolved.
Show resolved Hide resolved

list_of_commands_to_send = []
for endpoint_index in range(number_of_commands_to_send):
tehampson marked this conversation as resolved.
Show resolved Hide resolved
# Using Toggle command to form the base as it is a command that doesn't take
Expand Down
Loading