Fast-DDS icon indicating copy to clipboard operation
Fast-DDS copied to clipboard

[20424] Dynamic language binding tests

Open JLBuenoLopez opened this issue 1 year ago • 3 comments

Description

This PR adds types that test the Dynamic Language Binding.

Contributor Checklist

  • [ ] Commit messages follow the project guidelines.
  • [ ] The code follows the style guidelines of this project.
  • [ ] Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • [ ] Any new/modified methods have been properly documented using Doxygen.
  • [ ] Changes are ABI compatible.
  • [ ] Changes are API compatible.
  • [ ] New feature has been added to the versions.md file (if applicable).
  • [ ] New feature has been documented/Current behavior is correctly described in the documentation.
  • [ ] Applicable backports have been included in the description.

Reviewer Checklist

  • [ ] The PR has a milestone assigned.
  • [ ] The title and description correctly express the PR's purpose.
  • [ ] Check contributor checklist is correct.
  • [ ] Check CI results: changes do not issue any warning.
  • [ ] Check CI results: failing tests are unrelated with the changes.

JLBuenoLopez avatar Mar 26 '24 13:03 JLBuenoLopez

The following suggestions should be taken into account:

  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530329728
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530333580
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530336679
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530337888
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530339299
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530342154
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530344592
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530350103
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530355716
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530359796
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530360783
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530361408
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530397186
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530400651
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530404940
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530430656
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530447837
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530451324
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530488263
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530493561
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530511011
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530523513
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530527949
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530532421
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530537828
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530552599
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530568082
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530571363
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530580830
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530581643
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530587789
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530593573
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1530633988
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531559573
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531561686
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531592276
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531729809
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531731867
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531801684
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531904195
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531906336
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531946858
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531948372
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1531951670
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532083995
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532091161
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532102761
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532115013
  • [ ] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532235320
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532236764
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532252577
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532292553
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532299927
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532305288
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532306754
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1532307352
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1533325623
  • [x] https://github.com/eProsima/Fast-DDS/pull/4190#discussion_r1533328793

JLBuenoLopez avatar Mar 26 '24 13:03 JLBuenoLopez

This PR should fix inheritance (both structure and bitsets) when loading a Dynamic Type from a XML profiles file.

JLBuenoLopez avatar Apr 09 '24 06:04 JLBuenoLopez

XML structure and bitset inheritance must be fixed. Please, open also a documentation PR fixing the corresponding snippets.

JLBuenoLopez avatar Apr 16 '24 08:04 JLBuenoLopez