MetOffice/CSET

Make constraints tests more robust

Open

#838 geöffnet am 17. Sept. 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (16 Forks)auto 404
cleanupgood first issue

Repository-Metriken

Stars
 (28 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I don't think checking the result of repr is sufficient for many of the tests in this file. In particular those where the repr just says a lambda or function. The relevant lambda or function could be changed to something that produces complete nonsense and these tests would never catch it. It's not really related to this PR though so I'd suggest opening a new issue.

Originally posted by @SGallagherMet in https://github.com/MetOffice/CSET/pull/765#discussion_r1740967262

We should make the tests in test_constraints.py more robust. This will likely involve making most of them into filter tests.

Contributor Guide

Research-Richtung
Untersuchen Sie die Einschränkungstests in test constraints.py und konvertieren Sie sie in Filtertests, um Robustheit zu gewährleisten. Stellen Sie sicher, dass die Tests tatsächlich die Einschränkungslogik überprüfen.
Tech Stack
python
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitPython
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70