From 9d4fe89d34bff7de8db64d3dbe99062ccd5e5bf2 Mon Sep 17 00:00:00 2001
From: Rodrigo Queiro <rodrigoq@google.com>
Date: Tue, 27 Feb 2018 15:19:28 +0100
Subject: [PATCH] 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.
---
 .../java/org/ros/internal/message/field/PrimitiveFieldType.java | 2 +-
 .../src/test/java/org/ros/internal/message/MessageTest.java     | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/message_generation/src/main/java/org/ros/internal/message/field/PrimitiveFieldType.java b/message_generation/src/main/java/org/ros/internal/message/field/PrimitiveFieldType.java
index bfbe835..53fbdfa 100644
--- a/message_generation/src/main/java/org/ros/internal/message/field/PrimitiveFieldType.java
+++ b/message_generation/src/main/java/org/ros/internal/message/field/PrimitiveFieldType.java
@@ -81,7 +81,7 @@ public enum PrimitiveFieldType implements FieldType {
 
     @Override
     public Field newVariableList(String name, int size) {
-      return ChannelBufferField.newVariable(this, name, size);
+      return ByteArrayField.newVariable(this, name, size);
     }
 
     @Override
diff --git a/message_generation/src/test/java/org/ros/internal/message/MessageTest.java b/message_generation/src/test/java/org/ros/internal/message/MessageTest.java
index fe80bb4..798efea 100644
--- a/message_generation/src/test/java/org/ros/internal/message/MessageTest.java
+++ b/message_generation/src/test/java/org/ros/internal/message/MessageTest.java
@@ -114,6 +114,7 @@ public class MessageTest {
     assertEquals("Hello, ROS! # comment", rawMessage.getString("data"));
   }
 
+  @Test
   public void testInt8List() {
     topicDefinitionResourceProvider.add("foo/foo", "int8[] data");
     RawMessage rawMessage = messageFactory.newFromType("foo/foo");
-- 
GitLab