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.

Sign up for GitHub

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

os: add tests for process-wide pwd on plan9 #10035

Closed
bradfitz opened this issue Feb 28, 2015 · 0 comments
Closed

os: add tests for process-wide pwd on plan9 #10035

bradfitz opened this issue Feb 28, 2015 · 0 comments
Labels
FrozenDueToAge Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone
Go1.5

Comments

@bradfitz
Copy link
Contributor

We should add more tests for the process-wide pwd issue found in #9428

The CL https://go-review.googlesource.com/6350 fixes the build but we should have explicit tests for this, and not just test it on accident.

/cc @0intro

David, maybe you want to write these? Start N goroutines which call LockOSThread, and then Chdir in the test goroutine and ask each goroutine what their Getwd is?
@bradfitz bradfitz added Testing An issue that has been verified to require only test changes, not just a test failure. os-plan9 labels Feb 28, 2015
@rsc rsc removed the os-plan9 label Apr 10, 2015
@rsc rsc changed the title plan9: add tests for process-wide pwd os: add tests for process-wide pwd on plan9 Apr 10, 2015
@rsc rsc added this to the Unplanned milestone Apr 10, 2015
@0intro 0intro closed this as completed in ae740a4 Apr 12, 2015
@mikioh mikioh modified the milestones: Go1.5, Unplanned Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Milestone
Go1.5
Development

No branches or pull requests
4 participants
@bradfitz @mikioh @rsc @gopherbot