search

FuelLabs / fuels-ts

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

chore: export test and cli utilities in `fuels` umbrella package #2637

Closed nedsalk closed 15 hours ago

nedsalk commented 5 days ago

I exported the utils via the fuels umbrella package and changed imports to import from fuels accordingly. The changes in this PR are exclusively tied to that and running pnpm lint:fix afterwards.

nedsalk commented 1 day ago

@maschad you searched for /test-utils which is showing results for fuels/test-utils, @fuel-ts/utils/test-utls, @fuel-ts/account/test-utils, etc. This consolidates it all to use import from fuels/test-utils.

maschad commented 1 day ago

@maschad you searched for /test-utils which is showing results for fuels/test-utils, @fuel-ts/utils/test-utls, @fuel-ts/account/test-utils, etc. This consolidates it all to use import from fuels/test-utils.

Thanks for the clarification, if you update the search to be explicit for @fuel-ts/errors/test-utils @fuel-ts/account/test-utils and @fuel-ts/utils/test-utils you will see quite a few files. I've elaborated more here

nedsalk commented 18 hours ago

@danielbate is right in https://github.com/FuelLabs/fuels-ts/pull/2637#pullrequestreview-2153497159.

As for the snippets and some leftovers that I didn't fix, I fixed them in https://github.com/FuelLabs/fuels-ts/pull/2637/commits/f980df50428e24509449dd11ee6dfd8109aa82f7 and https://github.com/FuelLabs/fuels-ts/pull/2637/commits/5131778d863af12360e13ebb5fbd0aba5cd9d957.

github-actions[bot] commented 15 hours ago

Coverage Report:

Lines Branches Functions Statements
79.54%(+0%) 71.37%(+0%) 76.91%(+0%) 79.62%(+0%)
Changed Files: Coverage values did not change👌.