EnergySystemsModellingLab/MUSE2

Add more tests for investment step

Open

#1,250 opened on Apr 16, 2026

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (3 forks)auto 404
good first issuemuse xiiitest

Repository metrics

Stars
 (7 stars)
PR merge metrics
 (PR metrics pending)

Description

While there are some unit tests in src/simulation/investment.rs, there are also many functions without dedicated unit tests. This is particularly true for simple utility functions (e.g. get_demand_portion_for_market) with clear inputs and outputs. For more complex functions, we should at least consider how we might test them, e.g. by breaking them up.

Relying on regression tests to detect problems with this part of the code is not a robust approach.

Contributor guide

Research direction
Explore the `src/simulation/investment.rs` file and identify functions without unit tests. Start with simple utility functions like `get demand portion for market`, write test cases covering typical inputs and edge cases. For complex functions, consider how to break them down into testable units.
Tech stack
rust
Domain
backend
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RustGitbasic testing knowledge
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75