Skip to content

Add Repeater + TCP docker compose tests - #272

Merged
jparisu merged 2 commits into
mainfrom
feature/tcp-repeater-compose
Aug 12, 2022
Merged

Add Repeater + TCP docker compose tests#272
jparisu merged 2 commits into
mainfrom
feature/tcp-repeater-compose

Conversation

@rsanchez15

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: RaulSanchez <raul@eprosima.com>
@rsanchez15
rsanchez15 temporarily deployed to codecov August 12, 2022 08:33 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov August 12, 2022 08:33 Inactive

@jparisu jparisu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add this new test to the CMakeLists

Comment thread ddsrouter_test/compose/repeater_tcp/ddsrouter_edge_1.yaml Outdated
Comment thread ddsrouter_test/compose/repeater_tcp/ddsrouter_edge_2.yaml Outdated
Signed-off-by: RaulSanchez <raul@eprosima.com>
@rsanchez15
rsanchez15 temporarily deployed to codecov August 12, 2022 08:37 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov August 12, 2022 08:37 Inactive
@codecov

codecov Bot commented Aug 12, 2022

Copy link
Copy Markdown

Codecov Report

Merging #272 (b79f227) into main (d343373) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #272      +/-   ##
==========================================
- Coverage   58.73%   58.64%   -0.09%     
==========================================
  Files          99       99              
  Lines        3390     3390              
  Branches     1264     1264              
==========================================
- Hits         1991     1988       -3     
- Misses        629      631       +2     
- Partials      770      771       +1     
Impacted Files Coverage Δ
ddsrouter_core/src/cpp/communication/Track.cpp 60.86% <0.00%> (-2.90%) ⬇️
...ore/src/cpp/reader/implementations/rtps/Reader.cpp 65.00% <0.00%> (-1.00%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jparisu jparisu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jparisu
jparisu merged commit 563466f into main Aug 12, 2022
@jparisu
jparisu deleted the feature/tcp-repeater-compose branch August 12, 2022 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants