Skip to content

Commit

Permalink
Update tests to actually try to serialize a char
Browse files Browse the repository at this point in the history
Depends on ros2/rosidl#781

Signed-off-by: Arjo Chakravarty <[email protected]>
  • Loading branch information
arjo129 committed Jan 4, 2024
1 parent 82af486 commit 8a04d49
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/rosidl_runtime_py/test_set_message.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,13 +122,13 @@ def test_set_message_fields_none():
def test_set_message_fields_partial():
original_msg = message_fixtures.get_msg_basic_types()[0]
original_msg.bool_value = False
original_msg.char_value = 3
original_msg.char_value = chr(3)
original_msg.int32_value = 42

modified_msg = copy.copy(original_msg)
values = {}
values['bool_value'] = True
values['char_value'] = 1
values['char_value'] = chr(1)
values['int32_value'] = 24
set_message_fields(modified_msg, values)

Expand Down Expand Up @@ -213,7 +213,7 @@ def test_set_message_fields_nested_type():
msg0 = message_fixtures.get_msg_nested()[0]

msg0.basic_types_value.bool_value = False
msg0.basic_types_value.char_value = 3
msg0.basic_types_value.char_value = chr(3)
msg0.basic_types_value.int32_value = 42

assert msg0.basic_types_value != msg_basic_types
Expand Down

0 comments on commit 8a04d49

Please sign in to comment.