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

MacOS CI-Tests failing #264

Closed
gfrey opened this issue Aug 30, 2018 · 5 comments
Closed

MacOS CI-Tests failing #264

gfrey opened this issue Aug 30, 2018 · 5 comments
Labels

Comments

@gfrey
Copy link

gfrey commented Aug 30, 2018

Which operating system (GOOS) and version are you using?

#  sw_vers
ProductName:    Mac OS X
ProductVersion: 10.13.6
BuildVersion:   17G65
# go version
go version go1.10.3 darwin/amd64

Please describe the issue that occurred.

During CI-Build for pr #263 the tests on MacOS failed with a RaceCondition. I tried to reproduce locally and reran the tests around a dozen times and noticed two things:

  • TestExchangedataInWatchedDir and TestExchangedataInWatchedFile fail entirely on my machine ("integration_darwin_test.go:92: [1] exchangedata failed: operation not supported").
  • TestFsnotifyDeleteWatchedDir has flakes. It worked like 75% of the times in my tests. This is also the test that failed in the CI build.

Are you able to reproduce the issue? Please provide steps to reproduce and a code sample if possible.

I just ran go test -v --race ./... and the first issue shows all the time, the second only every now and then.

@nathany
Copy link
Contributor

nathany commented Aug 30, 2018

It is failing on 10.13+:

A exchangedata() function call appeared in Darwin 1.3.1 (Mac OS X version 10.0). It was deprecated in macOS 10.13. https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man2/exchangedata.2.html

The call to func Exchangedata(path1 string, path2 string, options int) (err error) from x/sys/unix.

@bep
Copy link
Member

bep commented Aug 30, 2018

I added that test etc. which was a way to support Sublime Text 3 etc. in Hugo.

@nathany
Copy link
Contributor

nathany commented Aug 30, 2018

Working on a skip test now.

@nathany
Copy link
Contributor

nathany commented Aug 30, 2018

@bep #267 does a skip of the test for now, though perhaps there is another way to write this test that doesn't depend on Exchangedata?

@arp242
Copy link
Member

arp242 commented Jul 22, 2022

does a skip of the test for now, though perhaps there is another way to write this test that doesn't depend on Exchangedata?

This test specifically tests Exchangedata, so if that's not available then I don't think there's anything to test?

@arp242 arp242 closed this as completed Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants