Fix tests on 32bit archs - Githubissues

uber-go / ratelimit

A Go blocking leaky-bucket rate limit implementation

MIT License

4.32k stars 300 forks source link

Fix tests on 32bit archs #129

Open creekorful opened 1 month ago

creekorful commented 1 month ago

Hello there,

While packaging your library on Debian I noticed that some tests were failing on 32bit archs because of an integer overflow. This PR fix the tests to ensure correct execution on both 32 and 64bit arch.

It has been tested on Debian architecture and now the package build fine.

Previous test log: https://ci.debian.net/packages/g/golang-uber-ratelimit/testing/armhf/51192820/
After the patch: https://ci.debian.net/packages/g/golang-uber-ratelimit/testing/armhf/51214247/

Package details: https://tracker.debian.org/pkg/golang-uber-ratelimit

Cheers,