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

Create test for checksum code #44

Closed

Teque5 opened this issue Jan 4, 2024 · 0 comments · Fixed by #68

Collaborator

Teque5 commented Jan 4, 2024

Missing a test for the checksum. Verify ok and verify that bad checksum raises err.

note to self about chaining exceptions; may use in same PR:

try:
    do_something()
except KeyError as err:
    msg = 'something happened'
    log.error(msg)
    raise KeyError(msg) from err

Teque5 mentioned this issue

Allow skip_checksum on creation #46

Closed

Teque5 mentioned this issue

add test for bad checksum #68

Merged

Teque5 closed this as completed in #68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment