Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: added basic integration tests for SRv6 Localsids #1573

Merged
merged 3 commits into from
Nov 29, 2019

Conversation

fgschwan
Copy link
Collaborator

Rewritten some CRUD robot test for SRv6 localsids (except of SR-proxy localsid due to more complicated approach for dumping data for SR-proxy localsids). This includes some localsid dump support on vppcalls level (tests needed verification of data written to VPP)

Signed-off-by: Filip Gschwandtner <filip.gschwandtner@pantheon.tech>
Signed-off-by: Filip Gschwandtner <filip.gschwandtner@pantheon.tech>
tests/integration/vpp/110_srv6_test.go Outdated Show resolved Hide resolved
tests/integration/vpp/110_srv6_test.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Nov 28, 2019

Codecov Report

Merging #1573 into dev will increase coverage by 1.86%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1573      +/-   ##
==========================================
+ Coverage   52.71%   54.58%   +1.86%     
==========================================
  Files         658      361     -297     
  Lines       78986    30979   -48007     
==========================================
- Hits        41638    16910   -24728     
+ Misses      34831    12743   -22088     
+ Partials     2517     1326    -1191
Flag Coverage Δ
#e2e ?
#unittests 54.58% <0%> (-0.75%) ⬇️

@fgschwan fgschwan changed the title Basic integration tests for SRv6 Localsids test: added basic integration tests for SRv6 Localsids Nov 28, 2019
Signed-off-by: Filip Gschwandtner <filip.gschwandtner@pantheon.tech>
@ondrej-fabry ondrej-fabry merged commit c40280a into ligato:dev Nov 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants