Draft: attempting to add inc_mw_com tests

MODULE.bazel

@@ -79,5 +79,7 @@ bazel_dep(name = "score-baselibs", version = "0.0.0")
 git_override(
     module_name = "score-baselibs",
     remote = "git@github.com:eclipse-score/baselibs.git",
-    commit = "f0a394a602986ddf7abac6a238b9d44535a4b597",
-)
+    commit = "9498064967db802c76b631b1b53d72eaaebd462a",
+)
+
+bazel_dep(name = "boost.system", version = "1.83.0")

score/mw/com/impl/bindings/lola/subscription_state_machine_states.h

@@ -14,6 +14,7 @@
 #define SCORE_MW_COM_IMPL_BINDINGS_LOLA_SUBSCRIPTION_STATE_MACHINE_STATES_H
 
 #include <string>
+#include <cstdint>
 
 namespace score::mw::com::impl::lola
 {

score/mw/com/message_passing/message.h

@@ -15,44 +15,45 @@
 
 #include <array>
 #include <cstdint>
+#include <sys/types.h>
 #include <type_traits>
 
 namespace score::mw::com::message_passing
 {
 
-/// \brief Identifies a message
-using MessageId = std::int8_t;
+  /// \brief Identifies a message
+  using MessageId = std::int8_t;
 
-/// \brief Represents the payload that is transmitted via a ShortMessage
-using ShortMessagePayload = std::uint64_t;
+  /// \brief Represents the payload that is transmitted via a ShortMessage
+  using ShortMessagePayload = std::uint64_t;
 
-/// \brief Represents the payload that is transmitted via a MediumMessage
-using MediumMessagePayload = std::array<uint8_t, 16>;
+  /// \brief Represents the payload that is transmitted via a MediumMessage
+  using MediumMessagePayload = std::array<uint8_t, 16>;
 
-static_assert(std::is_trivially_copyable<pid_t>::value, "pid_t must be TriviallyCopyable");
-static_assert(std::is_trivially_copyable<MessageId>::value, "MessageId must be TriviallyCopyable");
-static_assert(std::is_trivially_copyable<ShortMessagePayload>::value, "ShortMessagePayload must be TriviallyCopyable");
-static_assert(std::is_trivially_copyable<MediumMessagePayload>::value,
-              "MediumMessagePayload must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<pid_t>::value, "pid_t must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<MessageId>::value, "MessageId must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<ShortMessagePayload>::value, "ShortMessagePayload must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<MediumMessagePayload>::value,
+                "MediumMessagePayload must be TriviallyCopyable");
 
-/// \brief Base class of all messages, containing common members.
-///
-/// \details The pid value depends on the context. If a message is sent, it shall contain the PID of the target process.
-/// I.e. the caller of Sender::Send() shall fill in the PID before. In case of reception of a message, the Receiver
-/// shall fill in the PID from where he received the message, before calling the registered handler.
-///
-/// It depends on the OS specific implementation/optimization of Sender/Receiver, whether the PID has to be transmitted
-/// explicitly (therefore extending the payload) or whether it is transmitted implicitly. E.g., when using QNX messaging
-/// mechanisms, the PID gets transmitted implicitly!
-/// In case a message gets received
-// Suppress "AUTOSAR C++14 A12-8-6" rule finding. This rule states: "Copy and move constructors and copy assignment and
-// move assignment operators shall be declared protected or defined ""=delete"" in base class."
-// The rule is intended to protect against slicing. BaseMessage class is created for layout documentation purposes
-// and is not used in the contexts where slicing is possible: there are no final objects, pointers or references of
-// this type. Trying to follow the rule will produce source bloat in this and derived classes with no practical benefit.
-// coverity[autosar_cpp14_a12_8_6_violation]
-class BaseMessage
-{
+  /// \brief Base class of all messages, containing common members.
+  ///
+  /// \details The pid value depends on the context. If a message is sent, it shall contain the PID of the target process.
+  /// I.e. the caller of Sender::Send() shall fill in the PID before. In case of reception of a message, the Receiver
+  /// shall fill in the PID from where he received the message, before calling the registered handler.
+  ///
+  /// It depends on the OS specific implementation/optimization of Sender/Receiver, whether the PID has to be transmitted
+  /// explicitly (therefore extending the payload) or whether it is transmitted implicitly. E.g., when using QNX messaging
+  /// mechanisms, the PID gets transmitted implicitly!
+  /// In case a message gets received
+  // Suppress "AUTOSAR C++14 A12-8-6" rule finding. This rule states: "Copy and move constructors and copy assignment and
+  // move assignment operators shall be declared protected or defined ""=delete"" in base class."
+  // The rule is intended to protect against slicing. BaseMessage class is created for layout documentation purposes
+  // and is not used in the contexts where slicing is possible: there are no final objects, pointers or references of
+  // this type. Trying to follow the rule will produce source bloat in this and derived classes with no practical benefit.
+  // coverity[autosar_cpp14_a12_8_6_violation]
+  class BaseMessage
+  {
   public:
     // Suppress "AUTOSAR C++14 M11-0-1" rule findings. This rule states: "Member data in non-POD class types shall
     // be private.". We need these data elements to be organized into a coherent organized data structure.
@@ -61,50 +62,50 @@ class BaseMessage
     MessageId id{};
     // coverity[autosar_cpp14_m11_0_1_violation]
     pid_t pid{-1};
-};
+  };
 
-/// \brief A ShortMessage shall be used for Inter Process Communication that is Asynchronous and acts as control
-/// mechanism.
-///
-/// \details Different operating systems can implement short messages very efficiently. This is given by the fact that
-/// such messages fit into register spaces on the CPU and copying the information is highly efficient. On QNX this is
-/// for example the case with Pulses. By providing such an interface, we make it easy for our applications to exchange
-/// data as efficient as possible. It shall be noted that no real data shall be transferred over this communication
-/// method, it shall rather act as a way to control or notify another process. The efficiency is gained by not providing
-/// strong typing, meaning the payload needs to be serialized by the user of the API. If you are searching for a strong
-/// typed interface, please consider our mw::com implementation (or ARA::COM).
-class ShortMessage : public BaseMessage
-{
+  /// \brief A ShortMessage shall be used for Inter Process Communication that is Asynchronous and acts as control
+  /// mechanism.
+  ///
+  /// \details Different operating systems can implement short messages very efficiently. This is given by the fact that
+  /// such messages fit into register spaces on the CPU and copying the information is highly efficient. On QNX this is
+  /// for example the case with Pulses. By providing such an interface, we make it easy for our applications to exchange
+  /// data as efficient as possible. It shall be noted that no real data shall be transferred over this communication
+  /// method, it shall rather act as a way to control or notify another process. The efficiency is gained by not providing
+  /// strong typing, meaning the payload needs to be serialized by the user of the API. If you are searching for a strong
+  /// typed interface, please consider our mw::com implementation (or ARA::COM).
+  class ShortMessage : public BaseMessage
+  {
   public:
     // Suppress "AUTOSAR C++14 M11-0-1" rule findings. This rule states: "Member data in non-POD class types shall
     // be private.". We need these data elements to be organized into a coherent organized data structure.
     // In MISRA C++:2023 this is no longer an issue and it is covered by rule 14.1.1
     // coverity[autosar_cpp14_m11_0_1_violation]
     ShortMessagePayload payload{};
-};
+  };
 
-static_assert(std::is_trivially_copyable<ShortMessage>::value, "ShortMessage type must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<ShortMessage>::value, "ShortMessage type must be TriviallyCopyable");
 
-/// \brief A MediumMessage shall be used for Inter Process Communication that is Asynchronous and acts as control
-/// mechanism.
-///
-/// \details Opposed to short messages, the medium size message might not be implemented that efficiently on various OS.
-/// Its size being double that of short-messages might hinder solutions, where message payload gets exchanged only via
-/// registers after a context switch. Still the payload size is small enough, that no (heap) memory allocation takes
-/// place.
-/// Introduction of medium messages was driven by some LoLa needs, where short messages weren't sufficient.
-class MediumMessage : public BaseMessage
-{
+  /// \brief A MediumMessage shall be used for Inter Process Communication that is Asynchronous and acts as control
+  /// mechanism.
+  ///
+  /// \details Opposed to short messages, the medium size message might not be implemented that efficiently on various OS.
+  /// Its size being double that of short-messages might hinder solutions, where message payload gets exchanged only via
+  /// registers after a context switch. Still the payload size is small enough, that no (heap) memory allocation takes
+  /// place.
+  /// Introduction of medium messages was driven by some LoLa needs, where short messages weren't sufficient.
+  class MediumMessage : public BaseMessage
+  {
   public:
     // Suppress "AUTOSAR C++14 M11-0-1" rule findings. This rule states: "Member data in non-POD class types shall
     // be private.". We need these data elements to be organized into a coherent organized data structure.
     // In MISRA C++:2023 this is no longer an issue and it is covered by rule 14.1.1
     // coverity[autosar_cpp14_m11_0_1_violation]
     MediumMessagePayload payload{};
-};
+  };
 
-static_assert(std::is_trivially_copyable<MediumMessage>::value, "MediumMessage type must be TriviallyCopyable");
+  static_assert(std::is_trivially_copyable<MediumMessage>::value, "MediumMessage type must be TriviallyCopyable");
 
-}  // namespace score::mw::com::message_passing
+} // namespace score::mw::com::message_passing
 
-#endif  // SCORE_MW_COM_MESSAGE_PASSING_SHORT_MESSAGE_H
+#endif // SCORE_MW_COM_MESSAGE_PASSING_SHORT_MESSAGE_H

score/mw/com/test/api/BUILD

@@ -0,0 +1,49 @@
+# *******************************************************************************
+# Copyright (c) 2025 Contributors to the Eclipse Foundation
+#
+# See the NOTICE file(s) distributed with this work for additional
+# information regarding copyright ownership.
+#
+# This program and the accompanying materials are made available under the
+# terms of the Apache License Version 2.0 which is available at
+# https://www.apache.org/licenses/LICENSE-2.0
+#
+# SPDX-License-Identifier: Apache-2.0
+# *******************************************************************************
+
+load("//score/bazel/generators:middleware.bzl", "cc_middleware_com_frontend")
+load("//score/bazel/generators:unit_tests.bzl", "cc_gtest_unit_test", "cc_unit_test_suites_for_host_and_qnx")
+load("//score/mw:common_features.bzl", "COMPILER_WARNING_FEATURES")
+
+cc_middleware_com_frontend(
+    name = "api",
+    srcs = select({
+        # TODO String type not supported due to a bug in the LLVM STL for QNX: []
+        "@platforms//os:qnx": [
+            "SomeComponent_qnx_cp60.fcdl",
+            "simple_struct_qnx_cp60.fidl",
+        ],
+        "//conditions:default": [
+            "SomeComponent.fcdl",
+            "simple_struct.fidl",
+        ],
+    }),
+)
+
+cc_gtest_unit_test(
+    name = "generated_api_test",
+    srcs = ["generated_api_test.cpp"],
+    features = COMPILER_WARNING_FEATURES,
+    deps = [
+        ":api",
+       # disabled, dependency missing: "//score/mw/log",
+    ],
+)
+
+cc_unit_test_suites_for_host_and_qnx(
+    name = "unit_test_suite",
+    cc_unit_tests = [
+        ":generated_api_test",
+    ],
+    visibility = ["//score/mw/com:__pkg__"],
+)

score/mw/com/test/api/SomeComponent.fcdl

@@ -0,0 +1,25 @@
+// *******************************************************************************>
+// Copyright (c) 2024 Contributors to the Eclipse Foundation
+// See the NOTICE file(s) distributed with this work for additional
+// information regarding copyright ownership.
+// This program and the accompanying materials are made available under the
+// terms of the Apache License Version 2.0 which is available at
+// https://www.apache.org/licenses/LICENSE-2.0
+// SPDX-License-Identifier: Apache-2.0 #
+// *******************************************************************************
+
+
+
+package our.name_space
+
+import our.name_space.* from "platform:/resource/AASModel/platform/aas/mw/com/test/api/simple_struct.fidl"
+
+service component SomeComponent {
+	provides SomeInterface as something
+	requires SomeInterface as something_else
+}
+
+// workwaround
+root component Irgendwie {
+    contains SomeComponent as FooBar
+}

score/mw/com/test/api/SomeComponent_qnx_cp60.fcdl

@@ -0,0 +1,25 @@
+// *******************************************************************************>
+// Copyright (c) 2024 Contributors to the Eclipse Foundation
+// See the NOTICE file(s) distributed with this work for additional
+// information regarding copyright ownership.
+// This program and the accompanying materials are made available under the
+// terms of the Apache License Version 2.0 which is available at
+// https://www.apache.org/licenses/LICENSE-2.0
+// SPDX-License-Identifier: Apache-2.0 #
+// *******************************************************************************
+
+
+
+package our.name_space
+
+import our.name_space.* from "platform:/resource/AASModel/platform/aas/mw/com/test/api/simple_struct_qnx_cp60.fidl"
+
+service component SomeComponent {
+	provides SomeInterface as something
+	requires SomeInterface as something_else
+}
+
+// workwaround
+root component Irgendwie {
+    contains SomeComponent as FooBar
+}