commaai / commaai/opendbc

Enable 100% branch coverage check

Open
#2,557 11 comments 0 reactions 0 assignees View on GitHub
bounty CI / testing
Dominant language
Python
Stars
3.4k
Forks
2.3k
Avg merge
3d 2h
Merged PRs (30d)
38

Description

Your goal is to [enable](https://github.com/commaai/opendbc/pull/2555) the branch coverage check and carefully fix all the violations.

For each violation:
1. can we remove this branch?
2. can we rewrite this branch?
3. add test coverage

Our test coverage should be fairly complete, so this should mostly be removing and rewriting.

Here's where we're at now:
Image

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the linked PR 2555 and the current coverage output shown in the issue. Run the existing test and coverage checks, then work through each reported branch violation using the issue's remove, rewrite, or test-coverage questions. Done means the branch coverage check is enabled and all violations are resolved.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.