diff options
| author | hathach <[email protected]> | 2026-09-06 23:21:05 +0700 |
|---|---|---|
| committer | hathach <[email protected]> | 2026-09-06 23:21:05 +0700 |
| commit | 831227a50876b25680a0d3fc4b5b0cc96e14f5bc (patch) | |
| tree | 394bdf3f19cde01dab170fcae17afb4b5a3bf9e8 /test | |
| parent | 16156990c404bff8304c8a2fa6f205c4728ca95a (diff) | |
tools: require documented driver coveragerework-metrics
Diffstat (limited to 'test')
| -rw-r--r-- | test/hil/test/test_drivers_coverage.py | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/test/hil/test/test_drivers_coverage.py b/test/hil/test/test_drivers_coverage.py index a159d2e94..eef508a46 100644 --- a/test/hil/test/test_drivers_coverage.py +++ b/test/hil/test/test_drivers_coverage.py @@ -64,16 +64,14 @@ class CheckerVerdicts(unittest.TestCase): self.addCleanup(os.unlink, tmp.name) return run_checker(tmp.name) - def test_missing_driver_becomes_a_warning_not_a_failure(self): - # drop every entry covering dcd_rp2040 and don't add it to uncovered: a - # coverage gap, not a validity error - it must not fail the run + def test_undocumented_missing_driver_fails(self): + # A new driver must be covered or carry an explicit uncovered reason. def mutate(d): for t in d['boards']: t['drivers'] = [x for x in t['drivers'] if x != 'dcd_rp2040'] r = self._mutated(mutate) - self.assertEqual(r.returncode, 0, r.stderr) - self.assertIn('WARNING: membrowse: dcd_rp2040 has no CI board', r.stdout) - self.assertEqual(r.stderr, '') + self.assertEqual(r.returncode, 1) + self.assertIn('dcd_rp2040 has no CI board and no uncovered entry', r.stderr) def test_unknown_driver_name_fails(self): r = self._mutated(lambda d: d['boards'][0]['drivers'].append('dcd_nonexistent')) |
