Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

testing: surprising interaction of subtests with TempDir [1.16 backport] #50645

Closed
gopherbot opened this issue Jan 16, 2022 · 3 comments
Closed

testing: surprising interaction of subtests with TempDir [1.16 backport] #50645

gopherbot opened this issue Jan 16, 2022 · 3 comments
Labels
CherryPickApproved Used during the release process for point releases FrozenDueToAge
Milestone
Go1.16.14

Comments

@gopherbot
Copy link

@ianlancetaylor requested issue #46624 to be considered for backport to the next 1.16 minor release.

@gopherbot Please open backport to 1.16.
@gopherbot gopherbot added the CherryPickCandidate Used during the release process for point releases label Jan 16, 2022
@gopherbot gopherbot mentioned this issue Jan 16, 2022
Closed
@gopherbot gopherbot added this to the Go1.16.14 milestone Jan 16, 2022
@ianlancetaylor ianlancetaylor mentioned this issue Jan 16, 2022
Closed
@gopherbot
Copy link
Author

Change https://golang.org/cl/378914 mentions this issue: [release-branch.go1.16] testing: drop unusual characters from TempDir directory name

@dmitshur
Copy link
Contributor

Approving as a fix for testing behavior. Only 1.16 needs the backport, as 1.17 already has this fix.

@dmitshur dmitshur added CherryPickApproved Used during the release process for point releases and removed CherryPickCandidate Used during the release process for point releases labels Jan 26, 2022
@gopherbot
Copy link
Author

Closed by merging 8144f1d to release-branch.go1.16.

gopherbot pushed a commit that referenced this issue Jan 26, 2022
@tklauser @dmitshur
[release-branch.go1.16] testing: drop unusual characters from TempDir…
8144f1d 
… directory name

Only use safe characters of the test name for the os.MkdirTemp pattern.
This currently includes the alphanumeric characters and ASCII
punctuation characters known not to interact with globs.

For #46624
Fixes #50645

Change-Id: I402c34775b943fed9b97963c52f79245cc16dc1d
Reviewed-on: https://go-review.googlesource.com/c/go/+/326010
Trust: Tobias Klauser <tobias.klauser@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
(cherry picked from commit 97cee43)
Reviewed-on: https://go-review.googlesource.com/c/go/+/378914
Trust: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Tobias Klauser <tobias.klauser@gmail.com>
@golang golang locked and limited conversation to collaborators Jan 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
CherryPickApproved Used during the release process for point releases FrozenDueToAge
Projects
None yet
Milestone
Go1.16.14
Development

No branches or pull requests
2 participants
@dmitshur @gopherbot