CleverRaven/Cataclysm-DDA

Tests: `guy != nullptr` randomly failing

Open

#73.352 geöffnet am 28. Apr. 2024

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (3.991 Forks)batch import
<Bug>Code: TestsHelp Wanted[C++]

Repository-Metriken

Stars
 (9.264 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4T 17h) (378 gemergte PRs in 30 T)

Beschreibung

Describe the bug

In all listed cases are also some armor display tests failing and this may be related. This specific random failure appears to be comparatively rare.

Unfortunately I lack the privileges in this repo to search through all workflow logs but I feel like I have seen this before. Can't prove it with additional links though.

Attach save file

Steps to reproduce

  1. Run tests.
  2. Win the rng lottery.
  3. Witness above failure.

Expected behavior

guy != nullptr

Screenshots

No response

Versions and configuration

n/a

Additional context

No response

Contributor Guide

Research-Richtung
Untersuche den fehlschlagenden Test 'guy != nullptr' und verfolge die Bedingungen, die zum Nullzeiger führen. Suche nach Race Conditions oder fehlenden Zuweisungen in der Spiellogik.
Tech Stack
ccpp
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40