golang/go

path/filepath: TestEvalSymlinks tests with "/" prefix fail in Windows

Open

#24.846 geöffnet am 13. Apr. 2018

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (19.008 Forks)batch import
NeedsInvestigationOS-Windowshelp wanted

Repository-Metriken

Stars
 (133.883 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

What version of Go are you using (go version)?

go version go1.10.1 windows/amd64

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

set GOARCH=amd64 set GOHOSTARCH=amd64 set GOHOSTOS=windows set GOOS=windows

What did you do?

Built go using "all.bat"

What did you expect to see?

while running tests ok path/filepath x.xxxs

What did you see instead?

--- FAIL: TestEvalSymlinks (0.04s) path_test.go:798: EvalSymlinks("C:\Users\varga\AppData\Local\Temp\evalsymlink400713581\test/linkabs") returns "C:\", want "/" FAIL FAIL path/filepath 1.552s

I have a code fix for this if this. Essentially just prepending the expected result with the volume name if we are on Windows and the path has a prefix of "/".

Does that sound reasonable?

Contributor Guide

Research-Richtung
Pruefe den fehlschlagenden Test und verstehe, wie EvalSymlinks Windows Pfade mit einem Schrägstrich Präfix behandelt. Die Korrektur besteht wahrscheinlich darin, dem erwarteten Ergebnis den Volume Namen (z. B. "C:") voranzustellen, wenn der Pfad unter Windows mit "/" beginnt.
Tech Stack
go
Domain
backendtesting
Issue Type
Fehler
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.
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
GoWindows
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70