CleverRaven/Cataclysm-DDA

Overmap coverage test needs to account for overmaps that spawn only in alternate Dimensions

Open

#84 557 ouverte le 8 janv. 2026

Voir sur GitHub
 (0 commentaires) (2 réactions) (0 assignés)C++ (3 991 forks)batch import
<Suggestion / Discussion>Code: TestsHelp WantedMap / Mapgen

Métriques du dépôt

Stars
 (9 264 stars)
Métriques de merge PR
 (Merge moyen 4j 17h) (378 PRs mergées en 30 j)

Description

Is your feature request related to a problem? Please describe.

When adding overmaps that only spawn in alternate dimensions (the terrains: "standing_stones_highlands", "highlands" can serve as an example here) you must forcibly add their entries to the overmap_terrain_coverage_whitelist to allow the test to pass.

Effectively this means that we dont check for special spawning errors in alternate dimensions

Solution you would like.

The test is expanded to cover alternate dimensions.

Describe alternatives you have considered.

No response

Additional context

No response

Guide contributeur

Direction de recherche
Examinez le test de couverture du terrain overmap pour comprendre comment il fonctionne actuellement. Étudiez le système des dimensions alternatives et comment les overmaps y apparaissent. Concevez un moyen de générer des données de test pour les overmaps dans des dimensions alternatives et modifiez le test pour les inclure.
Stack technique
Aucun
Domaine
testing
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
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
C++Cataclysm DDA development setup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30