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

Issue #371 - Add h2spec container test (inspired by qdr-image tests from @fgiorgetti) #381

Merged

Conversation

jiridanek
Copy link
Contributor

No description provided.

@jiridanek jiridanek linked an issue Apr 27, 2022 that may be closed by this pull request
@jiridanek jiridanek force-pushed the jd_2022_04_22_fernandos_container_tests branch 2 times, most recently from f8e34b3 to 0d0d885 Compare April 28, 2022 12:28
@jiridanek jiridanek changed the title Issue #371 - Add h2spec container test (inspired by qdr tests from @fgiorgetti) Issue #371 - Add h2spec container test (inspired by qdr-image tests from @fgiorgetti) Apr 28, 2022
@jiridanek jiridanek marked this pull request as ready for review April 28, 2022 14:45
@jiridanek jiridanek added this to the 2.1.0 milestone Apr 30, 2022
@jiridanek jiridanek force-pushed the jd_2022_04_22_fernandos_container_tests branch 2 times, most recently from 81caacd to 8eb3b3f Compare May 2, 2022 20:24
@jiridanek jiridanek force-pushed the jd_2022_04_22_fernandos_container_tests branch from 8eb3b3f to 87d9789 Compare May 9, 2022 19:45
@jiridanek jiridanek force-pushed the jd_2022_04_22_fernandos_container_tests branch from 87d9789 to 6ab9ff8 Compare May 13, 2022 08:17
@codecov-commenter
Copy link

codecov-commenter commented May 18, 2022

Codecov Report

Merging #381 (0923a26) into main (3111aaf) will increase coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #381      +/-   ##
==========================================
+ Coverage   25.04%   25.29%   +0.24%     
==========================================
  Files         114      114              
  Lines       28609    28601       -8     
  Branches     4646     4648       +2     
==========================================
+ Hits         7165     7234      +69     
+ Misses      20487    20406      -81     
- Partials      957      961       +4     
Flag Coverage Δ
unittests 25.29% <ø> (+0.24%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/router_core/modules/mobile_sync/mobile.c 0.76% <0.00%> (-0.01%) ⬇️
src/dispatch.c 62.29% <0.00%> (ø)
src/router_node.c 5.91% <0.00%> (+0.10%) ⬆️
src/policy.c 25.77% <0.00%> (+0.14%) ⬆️
src/router_core/address_watch.c 79.74% <0.00%> (+1.26%) ⬆️
src/connection_manager.c 19.72% <0.00%> (+15.23%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3111aaf...0923a26. Read the comment docs.

@jiridanek
Copy link
Contributor Author

jiridanek commented May 18, 2022

@ganeshmurthy Added a comment with manual commands to run the test by hand in three terminal windows. I solemnly promise to be ready to triage any problems with the test that might appear, so that this test addition is not causing any extra work for you.

@jiridanek jiridanek merged commit 380eff4 into skupperproject:main May 18, 2022
@jiridanek jiridanek deleted the jd_2022_04_22_fernandos_container_tests branch May 18, 2022 15:46
@jiridanek jiridanek self-assigned this Jun 16, 2022
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.

3 participants