EnergySystemsModellingLab/MUSE2

Improve test coverage for `input` module

Ouverte

#1 248 ouverte le 16 avr. 2026

 (0 commentaire) (0 réaction) (0 personne assignée)Rust (3 forks)auto 404
good first issuemuse xiiitest

Métriques du dépôt

Stars
 (7 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

The test coverage could be better. Use cargo llvm-cov to find areas of the code that aren't covered (see relevant section of our docs on how to use this).

In most cases, it probably makes sense to use patched examples rather than writing traditional unit tests for it, but the main thing is that the tests are targeted, as opposed to many of the regression tests we have now (not all though!).

I just ran cargo llvm-cov and it seems most of the places that aren't covered are error paths for input files, which should be eminently testable 😄. Hopefully you should be able to write tests using the simple example, but, if that proves difficult, we may have to do #1080 first.

It's fine not to have 100% coverage for this module. This issue is just about improving things.

Guide contributeur