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

Better handle network errors #164

Merged
merged 8 commits into from
Jan 4, 2024

Improve coverage of tests

19726d1
Select commit
Loading
Failed to load commit list.
Merged

Better handle network errors #164

Improve coverage of tests
19726d1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 4, 2024 in 0s

88.60% of diff hit (target 91.00%)

View this Pull Request on Codecov

88.60% of diff hit (target 91.00%)

Annotations

Check warning on line 228 in pkcs11/src/backend/login.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/login.rs#L228

Added line #L228 was not covered by tests

Check warning on line 232 in pkcs11/src/backend/login.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/login.rs#L232

Added line #L232 was not covered by tests

Check warning on line 243 in pkcs11/src/backend/login.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/login.rs#L240-L243

Added lines #L240 - L243 were not covered by tests

Check warning on line 56 in pkcs11/src/backend/mod.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/mod.rs#L56

Added line #L56 was not covered by tests

Check warning on line 158 in pkcs11/src/backend/mod.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/mod.rs#L158

Added line #L158 was not covered by tests

Check warning on line 211 in pkcs11/src/backend/mod.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkcs11/src/backend/mod.rs#L211

Added line #L211 was not covered by tests