Skip to content
This repository was archived by the owner on Jan 14, 2023. It is now read-only.

Commit 9d4fe89

Browse files
committed
Add @test to testInt8List and fix it
This makes sure it is run. This was caught by error-prone. The test was also failing, so fix the copy-paste error in PrimitiveFieldType that broke it.
1 parent 304e521 commit 9d4fe89

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

message_generation/src/main/java/org/ros/internal/message/field/PrimitiveFieldType.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ public Byte getDefaultValue() {
8181

8282
@Override
8383
public Field newVariableList(String name, int size) {
84-
return ChannelBufferField.newVariable(this, name, size);
84+
return ByteArrayField.newVariable(this, name, size);
8585
}
8686

8787
@Override

message_generation/src/test/java/org/ros/internal/message/MessageTest.java

+1
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ public void testConstantString() {
114114
assertEquals("Hello, ROS! # comment", rawMessage.getString("data"));
115115
}
116116

117+
@Test
117118
public void testInt8List() {
118119
topicDefinitionResourceProvider.add("foo/foo", "int8[] data");
119120
RawMessage rawMessage = messageFactory.newFromType("foo/foo");

0 commit comments

Comments
 (0)