Skip to content

Conversation

@som-snytt
Copy link
Contributor

Fixes #24877

@som-snytt som-snytt force-pushed the issue/24877-array-range branch from cf7b113 to 548ebca Compare January 8, 2026 19:55
@som-snytt som-snytt marked this pull request as ready for review January 8, 2026 19:55
@som-snytt som-snytt requested a review from a team as a code owner January 8, 2026 19:55
@som-snytt
Copy link
Contributor Author

Ha! Scala.js was bug-compatible! I believe I just deleted the skip directive from the test.

image

@som-snytt
Copy link
Contributor Author

Probably there is a procedure for mitigating flaky tests. This is a repeat offender. Timeout tests are inherently flaky and need at least some gesture toward retry with backoff.

[123-0] + requests.ModelTests.multipart file uploads should contain application/octet-stream content type 209ms  
[123-0] Tests: 1, Passed: 1, Failed: 0
[123-1] + requests.RequestTests.matchingMethodWorks 459ms  
[123-1] + requests.RequestTests.params.get 39ms  
[123-1] + requests.RequestTests.multipart 20ms  
[123-1] + requests.RequestTests.cookies.session 21ms  
[123-1] + requests.RequestTests.cookies.raw 18ms  
[123-1] + requests.RequestTests.cookies.space 20ms  
[123-1] + requests.RequestTests.redirects.max 77ms  
[123-1] + requests.RequestTests.redirects.maxRelative 76ms  
[123-1] + requests.RequestTests.test_reproduction 3ms  
[123-1] + requests.RequestTests.streaming 13ms  
[123-1] + requests.RequestTests.timeouts.read 3023ms  requests.TimeoutException: Request to http://localhost:32777/delay/3 timed out. (readTimeout: 2000, connectTimout: 10000)
[123-1] + requests.RequestTests.timeouts.connect 38ms  requests.TimeoutException: Request to https://httpbin.org/delay/1 timed out. (readTimeout: 10000, connectTimout: 1)
[123-1] X requests.RequestTests.failures 537ms 
[123-1]   requests.RequestFailedException: Request to https://doesnt-exist.com/ failed with status code 403

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Array.range throw ArrayIndexOutOfBoundsException if overflow. not consists with other collections

1 participant