Rename test interfaces package to make it a shorter name

Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>

rosbag2_storage_mcap/package.xml

@@ -23,7 +23,7 @@
   <test_depend>rosbag2_cpp</test_depend>
   <test_depend>rosbag2_test_common</test_depend>
   <test_depend>std_msgs</test_depend>
-  <test_depend>rosbag2_storage_mcap_test_fixture_interfaces</test_depend>
+  <test_depend>rosbag2_storage_mcap_test_msgs</test_depend>
 
   <export>
     <build_type>ament_cmake</build_type>

rosbag2_storage_mcap/test/rosbag2_storage_mcap/test_message_definition_cache.cpp

@@ -25,61 +25,58 @@ using ::testing::UnorderedElementsAre;
 
 TEST(test_message_definition_cache, can_find_idl_includes) {
   const char sample[] = R"r(
-#include "rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdlA.idl"
+#include "rosbag2_storage_mcap_test_msgs/msg/BasicIdlA.idl"
 
-#include <rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdlB.idl>
+#include <rosbag2_storage_mcap_test_msgs/msg/BasicIdlB.idl>
 
-module rosbag2_storage_mcap_test_fixture_interfaces {
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct ComplexIdl {
-      rosbag2_storage_mcap_test_fixture_interfaces::msg::BasicIdlA a;
-      rosbag2_storage_mcap_test_fixture_interfaces::msg::BasicIdlB b;
+      rosbag2_storage_mcap_test_msgs::msg::BasicIdlA a;
+      rosbag2_storage_mcap_test_msgs::msg::BasicIdlB b;
     };
   };
 };
 
 )r";
   std::set<std::string> dependencies = parse_dependencies(Format::IDL, sample, "");
-  ASSERT_THAT(dependencies,
-              UnorderedElementsAre("rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdlA",
-                                   "rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdlB"));
+  ASSERT_THAT(dependencies, UnorderedElementsAre("rosbag2_storage_mcap_test_msgs/msg/BasicIdlA",
+                                                 "rosbag2_storage_mcap_test_msgs/msg/BasicIdlB"));
 }
 
 TEST(test_message_definition_cache, can_find_msg_deps) {
   MessageDefinitionCache cache;
-  auto [format, content] =
-    cache.get_full_text("rosbag2_storage_mcap_test_fixture_interfaces/ComplexMsg");
+  auto [format, content] = cache.get_full_text("rosbag2_storage_mcap_test_msgs/ComplexMsg");
   ASSERT_EQ(format, Format::MSG);
   ASSERT_EQ(content,
-            R"r(rosbag2_storage_mcap_test_fixture_interfaces/BasicMsg b
+            R"r(rosbag2_storage_mcap_test_msgs/BasicMsg b
 
 ================================================================================
-MSG: rosbag2_storage_mcap_test_fixture_interfaces/BasicMsg
+MSG: rosbag2_storage_mcap_test_msgs/BasicMsg
 float32 c
 )r");
 }
 
 TEST(test_message_definition_cache, can_find_idl_deps) {
   MessageDefinitionCache cache;
-  auto [format, content] =
-    cache.get_full_text("rosbag2_storage_mcap_test_fixture_interfaces/msg/ComplexIdl");
+  auto [format, content] = cache.get_full_text("rosbag2_storage_mcap_test_msgs/msg/ComplexIdl");
   EXPECT_EQ(format, Format::IDL);
   EXPECT_EQ(content,
             R"r(================================================================================
-IDL: rosbag2_storage_mcap_test_fixture_interfaces/msg/ComplexIdl
-#include "rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdl.idl"
+IDL: rosbag2_storage_mcap_test_msgs/msg/ComplexIdl
+#include "rosbag2_storage_mcap_test_msgs/msg/BasicIdl.idl"
 
-module rosbag2_storage_mcap_test_fixture_interfaces {
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct ComplexIdl {
-      rosbag2_storage_mcap_test_fixture_interfaces::msg::BasicIdl a;
+      rosbag2_storage_mcap_test_msgs::msg::BasicIdl a;
     };
   };
 };
 
 ================================================================================
-IDL: rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdl
-module rosbag2_storage_mcap_test_fixture_interfaces {
+IDL: rosbag2_storage_mcap_test_msgs/msg/BasicIdl
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct BasicIdl {
         float x;
@@ -92,28 +89,28 @@ module rosbag2_storage_mcap_test_fixture_interfaces {
 TEST(test_message_definition_cache, can_resolve_msg_with_idl_deps) {
   MessageDefinitionCache cache;
   auto [format, content] =
-    cache.get_full_text("rosbag2_storage_mcap_test_fixture_interfaces/msg/ComplexMsgDependsOnIdl");
+    cache.get_full_text("rosbag2_storage_mcap_test_msgs/msg/ComplexMsgDependsOnIdl");
   EXPECT_EQ(format, Format::IDL);
   EXPECT_EQ(content,
             R"r(================================================================================
-IDL: rosbag2_storage_mcap_test_fixture_interfaces/msg/ComplexMsgDependsOnIdl
+IDL: rosbag2_storage_mcap_test_msgs/msg/ComplexMsgDependsOnIdl
 // generated from rosidl_adapter/resource/msg.idl.em
-// with input from rosbag2_storage_mcap_test_fixture_interfaces/msg/ComplexMsgDependsOnIdl.msg
+// with input from rosbag2_storage_mcap_test_msgs/msg/ComplexMsgDependsOnIdl.msg
 // generated code does not contain a copyright notice
 
-#include "rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdl.idl"
+#include "rosbag2_storage_mcap_test_msgs/msg/BasicIdl.idl"
 
-module rosbag2_storage_mcap_test_fixture_interfaces {
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct ComplexMsgDependsOnIdl {
-      rosbag2_storage_mcap_test_fixture_interfaces::msg::BasicIdl a;
+      rosbag2_storage_mcap_test_msgs::msg::BasicIdl a;
     };
   };
 };
 
 ================================================================================
-IDL: rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdl
-module rosbag2_storage_mcap_test_fixture_interfaces {
+IDL: rosbag2_storage_mcap_test_msgs/msg/BasicIdl
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct BasicIdl {
         float x;

rosbag2_storage_mcap_test_fixture_interfaces/CHANGELOG.rst → rosbag2_storage_mcap_test_msgs/CHANGELOG.rst

@@ -1,5 +1,5 @@
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-Changelog for package rosbag2_storage_mcap_test_fixture_interfaces
+Changelog for package rosbag2_storage_mcap_test_msgs
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 0.4.0 (2022-10-06)

rosbag2_storage_mcap_test_fixture_interfaces/CMakeLists.txt → rosbag2_storage_mcap_test_msgs/CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.8)
-project(rosbag2_storage_mcap_test_fixture_interfaces)
+project(rosbag2_storage_mcap_test_msgs)
 
 if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
   add_compile_options(-Wall -Wextra -Wpedantic)

rosbag2_storage_mcap_test_fixture_interfaces/msg/BasicIdl.idl → rosbag2_storage_mcap_test_msgs/msg/BasicIdl.idl

@@ -1,4 +1,4 @@
-module rosbag2_storage_mcap_test_fixture_interfaces {
+module rosbag2_storage_mcap_test_msgs {
   module msg {
     struct BasicIdl {
         float x;

rosbag2_storage_mcap_test_msgs/msg/ComplexIdl.idl

@@ -0,0 +1,9 @@
+#include "rosbag2_storage_mcap_test_msgs/msg/BasicIdl.idl"
+
+module rosbag2_storage_mcap_test_msgs {
+  module msg {
+    struct ComplexIdl {
+      rosbag2_storage_mcap_test_msgs::msg::BasicIdl a;
+    };
+  };
+};

rosbag2_storage_mcap_test_msgs/msg/ComplexMsg.msg

@@ -0,0 +1 @@
+rosbag2_storage_mcap_test_msgs/BasicMsg b

rosbag2_storage_mcap_test_msgs/msg/ComplexMsgDependsOnIdl.msg

@@ -0,0 +1 @@
+rosbag2_storage_mcap_test_msgs/BasicIdl a

rosbag2_storage_mcap_test_fixture_interfaces/package.xml → rosbag2_storage_mcap_test_msgs/package.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <package format="3">
-  <name>rosbag2_storage_mcap_test_fixture_interfaces</name>
+  <name>rosbag2_storage_mcap_test_msgs</name>
   <version>0.4.0</version>
   <description>message definition test fixtures for MCAP schema recording</description>
   <maintainer email="james@foxglove.dev">James Smith</maintainer>