EducationalTestingService / EducationalTestingService/rsmtool

Streamline the code for flagging warnings in the reports

Offen
#539 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
good first issue help wanted
Vorherrschende Sprache
Python
Sterne
71
Forks
21
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

When tests are run in STRICT mode, we check for any errors or warnings in the report.

We currently have two functions:

``test_utils.check_report`` https://github.com/EducationalTestingService/rsmtool/blob/main/rsmtool/test_utils.py#L767 checks for both errors and warnings

``collect_warning_messages_from_report`` https://github.com/EducationalTestingService/rsmtool/blob/main/rsmtool/test_utils.py#L737 once again collects warning messages.

The warnings that are excluded under ``check_report`` would still be flagged as errors by ``collect_warning_messages_from_report``.

We need to figure out why we have these two functions and streamline them.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Lies rsmtool/test_utils.py bei collect_warning_messages_from_report und check_report und prüfe die Funktionen. Vergleiche, wie jede Funktion mit Warnungen und Ausschlüssen umgeht, und untersuche dann den STRICT-Modus-Testpfad, der sie aufruft. Fertig ist die Änderung, wenn die Warnungsprüfungen eine klare, konsistente Zuständigkeit haben und die relevanten STRICT-Modus-Tests bestehen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
testing
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.