aws / aws/aws-encryption-sdk-c
Fix missing coverage for aws_cryptosdk_priv_fail_session
Open
maintenance
- Dominant language
- C
- Stars
- 63
- Forks
- 59
- PR merge metrics
- No merged PRs in 30d
Description
Nothing currently ends up calling this in the tests. Probably we can consolidate some of the session_process codepaths to call it instead of open-coding the same sort of logic.
Contributor guide
Research direction
Locate aws_cryptosdk_priv_fail_session and the session_process codepaths, then inspect the existing tests that exercise those paths. Consolidate the relevant path so the function is called rather than duplicating its logic, and verify that tests execute the function and improve coverage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- cryptography, security, testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100