EnergySystemsModellingLab/MUSE2

Add more tests for investment step

Ouverte

#1 250 ouverte le 16 avr. 2026

Voir sur GitHubGuide contributeur
 (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

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.

Guide contributeur

Direction de recherche
Examinez le fichier `src/simulation/investment.rs` et identifiez les fonctions sans tests unitaires. Commencez par des fonctions utilitaires simples comme `get demand portion for market`, en écrivant des cas de test couvrant les entrées typiques et les cas limites. Pour les fonctions complexes, envisagez comment les décomposer en unités testables.
Stack technique
rust
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGitbasic testing knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75