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

Document Temp Dir Creation #476

Closed
svozza opened this issue Apr 25, 2019 · 2 comments · Fixed by #477
Closed

Document Temp Dir Creation #476

svozza opened this issue Apr 25, 2019 · 2 comments · Fixed by #477

Comments

@svozza
Copy link
Contributor

svozza commented Apr 25, 2019

**Is your feature request related to a problem?
I recently ran into the the issue desribed in #329, this caused me to waste a significant amount of time trying to figure out why my tests were passing on my Mac but not on my build system running Ubuntu. I assumed the file system was blank and was creating the /tmp directory in my tests and of course this worked fine locally because the temp directory pyfakefs creates for MacOS is /var.

Describe the solution you'd like
Document that the filesystem is not blank and explain the difference between MacOs and Linux.

Describe alternatives you've considered
n/a

I'm willing to make the PR to if someone can tell me where this note should live.

@mrbean-bremen
Copy link
Member

mrbean-bremen commented Apr 25, 2019

Thanks - good point! That could definitely help other users.
I think this belongs somewhere in the usage chapter, though I'm not sure where exactly - probably add a subtopic for troubleshooting? Maybe we can think of other problems that have occurred, for example problems with patching some libraries.
We would definitely appreciate a PR for this!

@svozza
Copy link
Contributor Author

svozza commented Apr 25, 2019

Sounds good, I should be able to stay this tomorrow!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants