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

Fix file race on windows #19

Open
empijei opened this issue Oct 2, 2017 · 2 comments
Open

Fix file race on windows #19

empijei opened this issue Oct 2, 2017 · 2 comments

Comments

@empijei
Copy link
Owner

empijei commented Oct 2, 2017

Wapty is not tested on windows but at least running make testvr on windows should be done.

@empijei empijei added this to the Misc/Maybe milestone Oct 2, 2017
@empijei empijei self-assigned this Oct 2, 2017
@as27
Copy link
Contributor

as27 commented Oct 2, 2017

Running the go tests generates just 2 errors. Both are inside confi/save_test.go

--- FAIL: TestSaveAll (0.00s)
save_test.go:52: rename C:\Users\SCHROE~1\AppData\Local\Temp.status.zip C:\Users\ME\AppData\Local\Temp\status.zip: Der Prozess kann nicht auf
die Datei zugreifen, da sie von einem anderen Prozess verwendet wird.
save_test.go:66: open C:\Users\Me\AppData\Local\Temp\status.zip: Das System kann die angegebene Datei nicht finden.
FAIL
FAIL

First error of line 52 means, that status.zip can not be used, becaue it is used by another process.
Error in line 66: status.zip is missing.

@empijei
Copy link
Owner Author

empijei commented Oct 2, 2017

Thanks, I'll look into this as soon as I get the chance

@empijei empijei closed this as completed Oct 2, 2017
@empijei empijei added the bug label Oct 2, 2017
@empijei empijei reopened this Oct 2, 2017
@empijei empijei changed the title Run tests on windows Fix file race on windows Oct 2, 2017
@empijei empijei assigned Pandry and unassigned empijei Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants