Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

test that large packets can be transmitted over weave #1121

Merged
merged 1 commit into from
Jul 13, 2015

Conversation

rade
Copy link
Member

@rade rade commented Jul 13, 2015

change in coverage:

$ diff /tmp/coverage-before.txt /tmp/coverage-after.txt 
372c372
< github.com/weaveworks/weave/router/connection_maker.go:161:   queryLoop           87.5%

---
> github.com/weaveworks/weave/router/connection_maker.go:161:   queryLoop           100.0%
411c411
< github.com/weaveworks/weave/router/ethernet_decoder.go:33:    sendICMPFragNeeded      0.0%

---
> github.com/weaveworks/weave/router/ethernet_decoder.go:33:    sendICMPFragNeeded      88.9%
419c419
< github.com/weaveworks/weave/router/forwarder.go:78:       forward             57.9%

---
> github.com/weaveworks/weave/router/forwarder.go:78:       forward             63.2%
549c549
< github.com/weaveworks/weave/router/router.go:147:     handleCapturedPacket        81.5%

---
> github.com/weaveworks/weave/router/router.go:147:     handleCapturedPacket        85.2%
609c609
< total:                                (statements)            82.2%

---
> total:                                (statements)            82.5%

(the connection_maker improvement is just noise)

tomwilkie added a commit that referenced this pull request Jul 13, 2015
test that large packets can be transmitted over weave
@tomwilkie tomwilkie merged commit 4eb6d14 into master Jul 13, 2015
@tomwilkie tomwilkie deleted the test_large_packets branch July 13, 2015 16:06
@rade rade modified the milestones: current, 1.1.0 Jul 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants