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

Integration tests os.RemoveAll directory not empty #2605

Closed
2 of 7 tasks
daviian opened this issue Sep 25, 2017 · 3 comments
Closed
2 of 7 tasks

Integration tests os.RemoveAll directory not empty #2605

daviian opened this issue Sep 25, 2017 · 3 comments

Comments

@daviian
Copy link
Member

daviian commented Sep 25, 2017

  • Gitea version (or commit ref):
  • Git version:
  • Operating system: Windows 10
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
[Macaron] 2017-09-25 20:05:38: Started GET /user2 for 
[Macaron] 2017-09-25 20:05:38: Completed /user2 200 OK in 7.9266ms
PASS
os.RemoveAll: remove integrations/indexers-pgsql/issues.bleve: directory not empty
make: *** [Makefile:190: test-pgsql] Error 1

Description

Since merging of b0f7457 I get this log when running integration tests in docker container.
I am using the webhippie/golang:edge image.

The full command I use is as follows:
run -i --rm -v D:\Development\go\src\code.gitea.io\gitea:/srv/app/src/code.gitea.io/gitea -w /srv/app/src/code.gitea.io/gitea --link gitea-test-db:pgsql -e TAGS="bindata" webhippie/golang:edge make build test-pgsql

@lunny lunny added this to the 1.x.x milestone Sep 26, 2017
@ethantkoenig
Copy link
Member

That's strange, os.RemoveAll isn't supposed to fail for non-empty directories.

@daviian
Copy link
Member Author

daviian commented Sep 30, 2017

I guess it is golang/go#20841

@daviian
Copy link
Member Author

daviian commented Sep 30, 2017

And because it is an issue with WSL I will close this one.

@daviian daviian closed this as completed Sep 30, 2017
@lunny lunny removed this from the 1.x.x milestone Oct 1, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants