Refs #21131: Apply Eliana's rev

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
eProsima · Jun 20, 2024 · e0e26c3 · e0e26c3
1 parent 94cea7a
commit e0e26c3
Show file tree

Hide file tree

Showing 19 changed files with 11 additions and 13 deletions.
diff --git a/include/fastdds/rtps/transport/test_UDPv4TransportDescriptor.h b/include/fastdds/rtps/transport/test_UDPv4TransportDescriptor.h
@@ -26,8 +26,7 @@ namespace fastdds {
 namespace rtps {
 
 struct CDRMessage_t;
-
-struct TestTransportOptions;
+struct TestUDPv4TransportOptions;
 
 /**
  * UDP v4 Test Transport configuration
@@ -41,7 +40,7 @@ struct test_UDPv4TransportDescriptor : public SocketTransportDescriptor
     typedef std::function<bool (const Locator& destination)> DestinationLocatorFilter;
 
     //! Test transport options
-    std::shared_ptr<TestTransportOptions> test_transport_options = std::make_shared<TestTransportOptions>();
+    std::shared_ptr<TestUDPv4TransportOptions> test_transport_options = std::make_shared<TestUDPv4TransportOptions>();
 
     //! Test shim parameters
     //! Percentage of data messages being dropped
@@ -153,10 +152,10 @@ struct test_UDPv4TransportDescriptor : public SocketTransportDescriptor
             const test_UDPv4TransportDescriptor& t) const;
 };
 
-struct TestTransportOptions
+struct TestUDPv4TransportOptions
 {
-    FASTDDS_EXPORTED_API TestTransportOptions() = default;
-    ~TestTransportOptions() = default;
+    FASTDDS_EXPORTED_API TestUDPv4TransportOptions() = default;
+    ~TestUDPv4TransportOptions() = default;
 
     std::atomic<bool> test_UDPv4Transport_ShutdownAllNetwork{false};
     // Handle to a persistent log of dropped packets. Defaults to length 0 (no logging) to prevent wasted resources.

diff --git a/src/cpp/rtps/transport/test_UDPv4Transport.h b/src/cpp/rtps/transport/test_UDPv4Transport.h
@@ -53,7 +53,7 @@ class test_UDPv4Transport : public UDPv4Transport
     virtual LocatorList NormalizeLocator(
             const Locator& locator) override;
 
-    std::shared_ptr<TestTransportOptions> test_transport_options;
+    std::shared_ptr<TestUDPv4TransportOptions> test_transport_options;
 
 protected:
 

diff --git a/test/blackbox/common/BlackboxTests.hpp b/test/blackbox/common/BlackboxTests.hpp
@@ -37,7 +37,7 @@
 #include "../types/HelloWorldPubSubTypes.h"
 #include "../types/KeyedData1mbPubSubTypes.h"
 #include "../types/KeyedHelloWorldPubSubTypes.h"
-#include "../types/rtps/core_typesPubSubTypes.h"
+#include "../types/core/core_typesPubSubTypes.h"
 #include "../types/StringTestPubSubTypes.h"
 #include "../types/UnboundedHelloWorldPubSubTypes.h"
 

diff --git a/test/blackbox/common/DDSBlackboxTestsDiscovery.cpp b/test/blackbox/common/DDSBlackboxTestsDiscovery.cpp
@@ -34,7 +34,6 @@
 #include <fastdds/rtps/common/Locator.h>
 #include <fastdds/rtps/participant/ParticipantDiscoveryInfo.h>
 #include <fastdds/rtps/transport/test_UDPv4TransportDescriptor.h>
-#include <fastdds/rtps/transport/test_UDPv4TransportDescriptor.h>
 #include <gtest/gtest.h>
 
 #include "../utils/filter_helpers.hpp"

diff --git a/test/blackbox/types/rtps/core_types.cxx → test/blackbox/types/core/core_types.cxx b/test/blackbox/types/rtps/core_types.cxx → test/blackbox/types/core/core_types.cxx
diff --git a/test/blackbox/types/rtps/core_types.h → test/blackbox/types/core/core_types.h b/test/blackbox/types/rtps/core_types.h → test/blackbox/types/core/core_types.h
diff --git a/test/blackbox/types/rtps/core_types.hpp → test/blackbox/types/core/core_types.hpp b/test/blackbox/types/rtps/core_types.hpp → test/blackbox/types/core/core_types.hpp
diff --git a/test/blackbox/types/rtps/core_types.idl → test/blackbox/types/core/core_types.idl b/test/blackbox/types/rtps/core_types.idl → test/blackbox/types/core/core_types.idl
@@ -13,7 +13,7 @@
 // limitations under the License.
 
 /**
- * @file test/blackbox/types/rtps/core_types.idl
+ * @file test/blackbox/types/core/core_types.idl
  */
 
 module eprosima {

diff --git a/.../blackbox/types/rtps/core_typesCdrAux.hpp → .../blackbox/types/core/core_typesCdrAux.hpp b/.../blackbox/types/rtps/core_typesCdrAux.hpp → .../blackbox/types/core/core_typesCdrAux.hpp
diff --git a/.../blackbox/types/rtps/core_typesCdrAux.ipp → .../blackbox/types/core/core_typesCdrAux.ipp b/.../blackbox/types/rtps/core_typesCdrAux.ipp → .../blackbox/types/core/core_typesCdrAux.ipp
diff --git a/...kbox/types/rtps/core_typesPubSubTypes.cxx → ...kbox/types/core/core_typesPubSubTypes.cxx b/...kbox/types/rtps/core_typesPubSubTypes.cxx → ...kbox/types/core/core_typesPubSubTypes.cxx
diff --git a/...ackbox/types/rtps/core_typesPubSubTypes.h → ...ackbox/types/core/core_typesPubSubTypes.h b/...ackbox/types/rtps/core_typesPubSubTypes.h → ...ackbox/types/core/core_typesPubSubTypes.h
diff --git a/...ckbox/types/rtps/core_typesTypeObject.cxx → ...ckbox/types/core/core_typesTypeObject.cxx b/...ckbox/types/rtps/core_typesTypeObject.cxx → ...ckbox/types/core/core_typesTypeObject.cxx
diff --git a/...lackbox/types/rtps/core_typesTypeObject.h → ...lackbox/types/core/core_typesTypeObject.h b/...lackbox/types/rtps/core_typesTypeObject.h → ...lackbox/types/core/core_typesTypeObject.h
diff --git a/...ypes/rtps/core_typesTypeObjectSupport.cxx → ...ypes/core/core_typesTypeObjectSupport.cxx b/...ypes/rtps/core_typesTypeObjectSupport.cxx → ...ypes/core/core_typesTypeObjectSupport.cxx
diff --git a/...ypes/rtps/core_typesTypeObjectSupport.hpp → ...ypes/core/core_typesTypeObjectSupport.hpp b/...ypes/rtps/core_typesTypeObjectSupport.hpp → ...ypes/core/core_typesTypeObjectSupport.hpp
diff --git a/test/blackbox/utils/filter_helpers.hpp b/test/blackbox/utils/filter_helpers.hpp
@@ -18,7 +18,7 @@
 #include <fastdds/dds/core/Entity.hpp>
 #include <fastdds/rtps/common/Types.h>
 
-#include "../types/rtps/core_typesPubSubTypes.h"
+#include "../types/core/core_typesPubSubTypes.h"
 
 namespace eprosima {
 namespace fastdds {

diff --git a/utils/scripts/update_generated_code_from_idl.sh b/utils/scripts/update_generated_code_from_idl.sh
@@ -41,7 +41,7 @@ files_needing_output_dir=(
     './thirdparty/dds-types-test/IDL/strings.idl|../../../test/dds-types-test'
     './thirdparty/dds-types-test/IDL/structures.idl|../../../test/dds-types-test'
     './thirdparty/dds-types-test/IDL/unions.idl|../../../test/dds-types-test'
-    './test/blackbox/types/rtps/core_types.idl|.'
+    './test/blackbox/types/core/core_types.idl|.'
     )
 
 files_needing_no_typesupport=(

diff --git a/versions.md b/versions.md
@@ -58,7 +58,7 @@ Forthcoming
   * New attribute in `SendBuffersAllocationAttributes` to configure allocation of `NetworkBuffer` vector.
   * `SenderResource` and Transport APIs now receive a collection of `NetworkBuffer` on their `send` method.
 * Migrate fastrtps namespace to fastdds
-* Make Blackbox tests not include any private API.
+* Remove all the private API include from Blackbox tests.
 
 Version 2.14.0
 --------------