drivendataorg/cloudpathlib

Tests: `tmpdir` is deprecated and we should use `tmp_path` instead

Offen

#370 geöffnet am 09.10.2023

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (87 Forks)auto 404
good first issuetests

Repository-Metriken

Stars
 (624 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We use tmpdir throughout the tests, but it is deprecated: https://docs.pytest.org/en/stable/how-to/tmp_path.html#the-tmpdir-and-tmpdir-factory-fixtures

We should remove replace tmpdir fixture with tmp_path throughout the test suite.

Originally posted by @jayqi in https://github.com/drivendataorg/cloudpathlib/pull/361#discussion_r1304854288

Contributor Guide

Research-Richtung
Ersetzen Sie alle Vorkommen der `tmpdir` Fixture durch `tmp path` in den Testdateien. Die `tmp path` Fixture gibt ein `pathlib.Path` Objekt zurück, anstelle eines stringbasierten `py.path.local`. Passen Sie Code an, der den Pfad als String verwendet oder Methoden aufruft, die spezifisch für `py.path.local` sind. Weitere Details zur Migration finden Sie in der pytest Dokumentation.
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.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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
Pythonpytest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90