Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

helper_lib test assumes the system temporary directory is outside a Git worktree

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#920 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
74/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
testing

Rechercherichtung

Beginne mit test_git_root_search_not_found in der helper_lib-Testsuite (Ziel //src/helper_lib:helper_lib_tests), lies dann die Implementierung von get_runfiles_dir() in helper_lib, um zu sehen, wie deren Suche nach einem Vorfahren mit .git funktioniert. Die Korrektur liegt im Test: Isoliere oder mocke die Grenze der Wurzelsuche (z. B. indem du das Startverzeichnis oder die Suchfunktion patchst), damit der No-Git-Root-Zweig unabhängig vom Layout von /tmp auf dem Host erreicht wird. Fertig bedeutet: Der Test besteht auf einem Host, auf dem ein Vorfahre von tmp_path ein .git enthält, und bazel test --lockfile_mode=error //src/helper_lib:helper_lib_tests ist grün.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Problem

test_git_root_search_not_found creates nowhere beneath pytest’s tmp_path and expects get_runfiles_dir() to fail because no .git ancestor exists. That assumption does not hold when the system temporary directory itself is inside a Git worktree.

On a Linux host where /tmp/.git exists, the test walks from /tmp/pytest-of-.../nowhere to /tmp, treats /tmp as the Git root, and then follows the runfiles fallback instead of raising SystemExit with Could not find git root. The target therefore fails even though the production behavior is consistent with its ancestor search.

Reproduction

  1. Run the helper-library tests on a host with a Git marker at an ancestor of pytest’s temporary directory (for example /tmp/.git).
  2. Run bazel test --lockfile_mode=error //src/helper_lib:helper_lib_tests.
  3. Observe test_git_root_search_not_found fail because the expected no-root branch is not reached.

This was reproduced while validating #919; 24 of 25 Bazel targets passed, with this target as the only failure.

Expected behavior

The test should deterministically exercise the no-Git-root path regardless of the host temporary-directory layout, for example by isolating or mocking the root search boundary rather than relying on tmp_path having no .git ancestor.

Vorherrschende Sprache
Python
Sterne
11
Forks
35
Ø Merge
1 T. 5 Std.
Gemergte PRs (30 T.)
77

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus eclipse-score/docs-as-code

Alle Issues in eclipse-score/docs-as-code

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

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