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

TestClientRetryWaitCallbackSwitchToDefault fails on 32-bit systems #386

Closed
creekorful opened this issue Nov 2, 2020 · 1 comment · Fixed by #388
Closed

TestClientRetryWaitCallbackSwitchToDefault fails on 32-bit systems #386

creekorful opened this issue Nov 2, 2020 · 1 comment · Fixed by #388

Comments

@creekorful
Copy link
Contributor

creekorful commented Nov 2, 2020

Hello there,

I've packaged your library on Debian, and some tests are failing:

The test TestClientRetryWaitCallbackSwitchToDefault pass on 64-bit systems, but fails on 32-bit systems, such as i386, armhf. It has been reported on Debian.

One can easily reproduce the error locally by doing something like this:

$ GOARCH=386 go test -v ./... -run TestClientRetryWaitCallbackSwitchToDefault -count=1

Error log:

=== RUN   TestClientRetryWaitCallbackSwitchToDefault
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    resty_test.go:41: Method: GET
    resty_test.go:42: Path: /set-retrywaittime-test
    retry_test.go:477: Client has slept 1.329608 seconds before retry 3
--- FAIL: TestClientRetryWaitCallbackSwitchToDefault (9.74s)

See:

creekorful added a commit to Debian/resty that referenced this issue Nov 10, 2020
creekorful added a commit to Debian/resty that referenced this issue Nov 10, 2020
@creekorful
Copy link
Contributor Author

Hello,

I have uploaded my patch and now the package builds fine on i386, but not armhf.
You can view build status here.

I'll take a look this week,

Cheers

jeevatkm pushed a commit that referenced this issue Jan 10, 2021
DomenicoSola pushed a commit to DomenicoSola/resty that referenced this issue Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant