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.

Sign up for GitHub

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

Jump to bottom

feat(connecteventmanager): block Connected() until accepted #435

Merged
merged 4 commits into from
Aug 17, 2023

fix(connecteventmanager): add test to confirm sync Connected() call flow

22fd90d
Select commit
Loading
Failed to load commit list.
Merged

feat(connecteventmanager): block Connected() until accepted #435

fix(connecteventmanager): add test to confirm sync Connected() call flow
22fd90d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Aug 17, 2023 in 1s

51.02% of diff hit (target 49.80%)

View this Pull Request on Codecov

51.02% of diff hit (target 49.80%)

Annotations

Check warning on line 45 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L45 

Added line #L45 was not covered by tests

Check warning on line 82 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L82 

Added line #L82 was not covered by tests

Check warning on line 105 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L101-L105 

Added lines #L101 - L105 were not covered by tests

Check warning on line 107 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L107 

Added line #L107 was not covered by tests

Check warning on line 109 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L109 

Added line #L109 was not covered by tests

Check warning on line 138 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L138 

Added line #L138 was not covered by tests

Check warning on line 160 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L149-L160 

Added lines #L149 - L160 were not covered by tests

Check warning on line 185 in bitswap/network/connecteventmanager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

bitswap/network/connecteventmanager.go#L185 

Added line #L185 was not covered by tests
View more details on Codecov