FuelLabs / fuels-ts

Fuel Network Typescript SDK
https://docs.fuel.network/docs/fuels-ts/
Apache License 2.0
44.21k stars 1.34k forks source link

chore: increate test timeout for e2e script #2632

Closed Torres-ssf closed 3 months ago

Torres-ssf commented 3 months ago

I've seen this report 2 times already

maschad commented 3 months ago

@Torres-ssf the error being thrown seems related to devnet faucet, what leads you to believe a timeout increase would solve this?

Torres-ssf commented 3 months ago

@Torres-ssf the error being thrown seems related to devnet faucet, what leads you to believe a timeout increase would solve this?

@maschad There seems to be an issue with the devnet, the problem is not specific to the faucet.

I misinterpreted the issue initially, thinking it was caused by a timeout. However, the timeout occurred due to the devnet issue.

I will mark this PR as awaiting before closing it because I recall seeing this test fail a couple of times before due to timeout, but when the devnet was working properly, of course.

maschad commented 3 months ago

Sounds good @Torres-ssf , you may want to convert it to draft as well, or the stale automation may be disrupted if someone interacts with the PR mistakenly thinking it's ready for review. We can re-open for review when devnet is stabilized.

github-actions[bot] commented 3 months ago

Coverage Report:

Lines Branches Functions Statements
80.29%(+0%) 71.8%(+0%) 77.56%(+0%) 80.37%(+0%)
Changed Files: Coverage values did not change👌.