EnergySystemsModellingLab/MUSE2

Improve test coverage for `input` module

オープン

#1,248 opened on 2026/04/16

 (0 件のコメント) (0 件のリアクション) (0 人の担当者)Rust (3 件のフォーク)auto 404
good first issuemuse xiiitest

Repository metrics

Stars
 (7 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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.

コントリビューターガイド