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.

Sign up for GitHub

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

fix: flaky retry test #1654

Merged
merged 26 commits into from
Jan 18, 2024
Merged

fix: flaky retry test #1654

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
a59907b
chore: `throw` on lint warnings (#1645)
danielbate Jan 15, 2024
b7216e4
chore: output `tsc` errors in build script (#1652)
danielbate Jan 16, 2024
c46fda2
fix: flaky test
nedsalk Jan 16, 2024
806c452
Revert "chore: `throw` on lint warnings (#1645)"
nedsalk Jan 16, 2024
8957fac
Revert "chore: output `tsc` errors in build script (#1652)"
nedsalk Jan 16, 2024
7e7dae9
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
nedsalk Jan 16, 2024
d0da15f
fix: retry tests' mock verification via chainInfo
nedsalk Jan 16, 2024
2d68236
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
nedsalk Jan 17, 2024
706810f
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
arboleya Jan 17, 2024
2819a5e
fix: auto-retry fetch calls, take #2 (#1662)
arboleya Jan 17, 2024
b8ff4ff
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
nedsalk Jan 17, 2024
67cd61f
Update packages/providers/src/utils/auto-retry-fetch.ts
nedsalk Jan 17, 2024
848fcae
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
nedsalk Jan 17, 2024
0b33506
feat: consider only ECONNREFUSED errors as retriable
nedsalk Jan 17, 2024
1ead540
test: better assertion
nedsalk Jan 17, 2024
c97f998
Update packages/providers/src/utils/auto-retry-fetch.test.ts
nedsalk Jan 17, 2024
55143f6
wording, optional property
nedsalk Jan 17, 2024
784c831
fix: first call is not considered a retry
nedsalk Jan 17, 2024
eac0f3c
Update packages/providers/src/utils/auto-retry-fetch.test.ts
nedsalk Jan 18, 2024
8c28c01
Update packages/providers/src/utils/auto-retry-fetch.test.ts
nedsalk Jan 18, 2024
9f4f94d
Update packages/providers/src/utils/auto-retry-fetch.test.ts
nedsalk Jan 18, 2024
d7eda26
Update packages/providers/src/utils/auto-retry-fetch.ts
nedsalk Jan 18, 2024
564e52a
Update packages/providers/src/utils/auto-retry-fetch.ts
nedsalk Jan 18, 2024
b594a47
Update packages/providers/test/auto-retry-fetch.test.ts
nedsalk Jan 18, 2024
f59e20f
fix: remove browser group for now
nedsalk Jan 18, 2024
c3a5699
Merge branch 'rc/salamander' into ns/fix/flaky-retry-test
nedsalk Jan 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: remove browser group for now
  • Loading branch information
@nedsalk
nedsalk committed Jan 18, 2024
commit f59e20fc54ce2aa1e612e1cf7b83fe67ca3eae16
2 changes: 1 addition & 1 deletion packages/providers/test/auto-retry-fetch.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ const FUEL_NETWORK_URL = 'http://127.0.0.1:4000/graphql';

/**
* @group node
nedsalk marked this conversation as resolved.
Show resolved Hide resolved
* @group browser
* TODO: add browser group as well (https://github.com/FuelLabs/fuels-ts/pull/1654#discussion_r1456501593)
*/
describe('Provider correctly', () => {
afterEach(() => {
Expand Down
Loading