Skip to content

Commit e6c0b9c

Browse files
More test fixes
1 parent 7958b70 commit e6c0b9c

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

test/test_server.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -113,62 +113,62 @@ def test_publish_empty_message_should_return_none():
113113
def test_subscribe_to_new_topic(mock_resolve_msg, mock_ros_subscriber):
114114
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
115115
result = SysCommands(server).subscribe("object_pos_topic", "pos")
116-
assert server.subscribers != {}
116+
assert server.subscribers_table != {}
117117
mock_ros_subscriber.assert_called_once
118118

119119

120120
@mock.patch.object(rospy, "Subscriber")
121121
@mock.patch.object(SysCommands, "resolve_message_name", return_value="unity_interfaces.msg/Pos")
122122
def test_subscribe_to_existing_topic(mock_resolve_msg, mock_ros_subscriber):
123123
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
124-
server.subscribers = {"object_pos_topic": mock.Mock()}
124+
server.subscribers_table = {"object_pos_topic": mock.Mock()}
125125
result = SysCommands(server).subscribe("object_pos_topic", "pos")
126-
assert server.subscribers["object_pos_topic"] is not None
126+
assert server.subscribers_table["object_pos_topic"] is not None
127127
mock_ros_subscriber.assert_called_once
128128

129129

130130
def test_subscribe_to_empty_topic_should_return_none():
131131
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
132132
result = SysCommands(server).subscribe("", "pos")
133133
assert result is None
134-
assert server.subscribers == {}
134+
assert server.subscribers_table == {}
135135

136136

137137
def test_subscribe_to_empty_message_should_return_none():
138138
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
139139
result = SysCommands(server).subscribe("test-topic", "")
140140
assert result is None
141-
assert server.subscribers == {}
141+
assert server.subscribers_table == {}
142142

143143

144144
@mock.patch.object(rospy, "ServiceProxy")
145145
@mock.patch.object(SysCommands, "resolve_message_name")
146146
def test_ros_service_new_topic(mock_resolve_msg, mock_ros_service):
147147
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
148148
result = SysCommands(server).ros_service("object_pos_topic", "pos")
149-
assert server.ros_services != {}
149+
assert server.ros_services_table != {}
150150
mock_ros_service.assert_called_once
151151

152152

153153
@mock.patch.object(rospy, "ServiceProxy")
154154
@mock.patch.object(SysCommands, "resolve_message_name")
155155
def test_ros_service_existing_topic(mock_resolve_msg, mock_ros_service):
156156
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
157-
server.ros_services = {"object_pos_topic": mock.Mock()}
157+
server.ros_services_table = {"object_pos_topic": mock.Mock()}
158158
result = SysCommands(server).ros_service("object_pos_topic", "pos")
159-
assert server.ros_services["object_pos_topic"] is not None
159+
assert server.ros_services_table["object_pos_topic"] is not None
160160
mock_ros_service.assert_called_once
161161

162162

163163
def test_ros_service_empty_topic_should_return_none():
164164
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
165165
result = SysCommands(server).ros_service("", "pos")
166166
assert result is None
167-
assert server.ros_services == {}
167+
assert server.ros_services_table == {}
168168

169169

170170
def test_ros_service_empty_message_should_return_none():
171171
server = TcpServer(node_name="test-tcp-server", tcp_ip="127.0.0.1", tcp_port=10000)
172172
result = SysCommands(server).ros_service("test-topic", "")
173173
assert result is None
174-
assert server.ros_services == {}
174+
assert server.ros_services_table == {}

0 commit comments

Comments
 (0)