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

Jump to bottom

feat: migrate over @fuels/assets package into the TS SDK #1730

Closed
wants to merge 71 commits into from
Closed

feat: migrate over @fuels/assets package into the TS SDK #1730

wants to merge 71 commits into from

Conversation

Dhaiwat10
Copy link
Member

Closes #1541

@Torres-ssf
made hdwallet part of wallet package
a9a2d19
@Torres-ssf
made mnemonic package part of wallet package
e368429
@Torres-ssf
made wordlists package part of wallet package
a19d379
@Torres-ssf
made signer package part of wallet package
f8b9451
@Torres-ssf
made wallet-manager package part of wallet package
bca8e88
@Torres-ssf
made predicate package part of wallet package
62e5b43
@Torres-ssf
rename base-unlocked-wallet file to base-wallet-unlocked
5bb1cc0
@Torres-ssf
move fixture wallet-spec to proper place
dbf12f2
@Torres-ssf
move GenerateOptions definition to Account
bf3ed3c
@Torres-ssf
renaming helpers
3871d22
@Torres-ssf
refact assembleTransferToContractScript
3699c75
@Torres-ssf
move file to utils
10962e6
@Torres-ssf
made unit test to be side by side with tested function
90a9c86
@Torres-ssf
moving test fixture to proper place
8f03426
@Torres-ssf
moving test fixture to proper place
ffd1c1b
@Torres-ssf
moving test fixture to proper place
ac81696
@Torres-ssf
rename package wallet to account
ccc2870
@Torres-ssf
fixing import
5f4e643
@Torres-ssf
fixing doc
d7ba558
@Torres-ssf
rename export
9e987e1
@Torres-ssf
move account related tests to proper place
c725eb8
@Torres-ssf
removing readme and changelog files
3cf635b
@Torres-ssf
Merge branch 'master' into st/refactor/restructure-account
c836901
@Torres-ssf
Merge branch 'master' into st/refactor/restructure-account
7a86215
@Torres-ssf
add changeset
091b684
@Torres-ssf
Merge branch 'master' into st/refactor/restructure-account
5fe7322
@Torres-ssf
made provider package part of account package
8c3283d
@Torres-ssf
remove provider entry from existent changeset to avoid CI error
9e44de6
@Torres-ssf
update eslintignore
55ae198
@Torres-ssf
lintfix
a194cb7
@Dhaiwat10 Dhaiwat10 added the feat Issue is a feature label Feb 7, 2024
@Dhaiwat10 Dhaiwat10 self-assigned this Feb 7, 2024
@Dhaiwat10 Dhaiwat10 changed the base branch from master to st/refactor/restructure-account February 7, 2024 23:55
@Dhaiwat10 Dhaiwat10 mentioned this pull request Feb 7, 2024
Closed
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 8, 2024

Coverage Report:

Lines Branches Functions Statements
79.55%(+0.8%) 69.61%(+0.94%) 78.62%(+1.3%) 79.59%(+0.88%)
Changed Files:
Ok File (✨=New File) Lines Branches Functions Statements
🔴 ✨ packages/account/src/account.ts 80%
(+80%)		 39.02%
(+39.02%)		 89.47%
(+89.47%)		 80%
(+80%)
✅ ✨ packages/account/src/configs.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/hdwallet/hdwallet.ts 94.44%
(+94.44%)		 84.37%
(+84.37%)		 100%
(+100%)		 94.5%
(+94.5%)
🔴 ✨ packages/account/src/mnemonic/mnemonic.ts 98.48%
(+98.48%)		 90.32%
(+90.32%)		 100%
(+100%)		 98.48%
(+98.48%)
🔴 ✨ packages/account/src/mnemonic/utils.ts 78.46%
(+78.46%)		 63.63%
(+63.63%)		 100%
(+100%)		 79.71%
(+79.71%)
🔴 ✨ packages/account/src/predicate/predicate.ts 31.37%
(+31.37%)		 25%
(+25%)		 27.27%
(+27.27%)		 31.37%
(+31.37%)
✅ ✨ packages/account/src/predicate/utils/getPredicateRoot.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/chains.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/coin-quantity.ts 100%
(+100%)		 93.75%
(+93.75%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/fuel-graphql-subscriber.ts 90%
(+90%)		 62.5%
(+62.5%)		 100%
(+100%)		 90%
(+90%)
🔴 ✨ packages/account/src/providers/memory-cache.ts 100%
(+100%)		 93.75%
(+93.75%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/provider.ts 81.36%
(+81.36%)		 68.9%
(+68.9%)		 86.15%
(+86.15%)		 81.91%
(+81.91%)
🔴 ✨ packages/account/src/providers/resource.ts 80%
(+80%)		 100%
(+100%)		 25%
(+25%)		 62.5%
(+62.5%)
🔴 ✨ packages/account/src/providers/generated/operations.ts 94.05%
(+94.05%)		 100%
(+100%)		 80.64%
(+80.64%)		 94.28%
(+94.28%)
✅ ✨ packages/account/src/providers/assets/index.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/assets/utils/network.ts 21.73%
(+21.73%)		 0%
(+0%)		 0%
(+0%)		 21.73%
(+21.73%)
🔴 ✨ packages/account/src/providers/assets/utils/resolveIconPath.ts 0%
(+0%)		 100%
(+100%)		 0%
(+0%)		 0%
(+0%)
🔴 ✨ packages/account/src/providers/assets/utils/url.ts 33.33%
(+33.33%)		 0%
(+0%)		 0%
(+0%)		 30%
(+30%)
🔴 ✨ packages/account/src/providers/transaction-request/create-transaction-request.ts 58.82%
(+58.82%)		 53.33%
(+53.33%)		 37.5%
(+37.5%)		 58.82%
(+58.82%)
🔴 ✨ packages/account/src/providers/transaction-request/errors.ts 0%
(+0%)		 100%
(+100%)		 0%
(+0%)		 0%
(+0%)
🔴 ✨ packages/account/src/providers/transaction-request/hash-transaction.ts 61.11%
(+61.11%)		 50%
(+50%)		 100%
(+100%)		 61.11%
(+61.11%)
🔴 ✨ packages/account/src/providers/transaction-request/input.ts 83.33%
(+83.33%)		 87.5%
(+87.5%)		 100%
(+100%)		 83.33%
(+83.33%)
🔴 ✨ packages/account/src/providers/transaction-request/output.ts 66.66%
(+66.66%)		 50%
(+50%)		 100%
(+100%)		 66.66%
(+66.66%)
🔴 ✨ packages/account/src/providers/transaction-request/script-transaction-request.ts 42.5%
(+42.5%)		 57.14%
(+57.14%)		 38.88%
(+38.88%)		 41.46%
(+41.46%)
🔴 ✨ packages/account/src/providers/transaction-request/scripts.ts 75%
(+75%)		 100%
(+100%)		 50%
(+50%)		 75%
(+75%)
🔴 ✨ packages/account/src/providers/transaction-request/storage-slot.ts 18.18%
(+18.18%)		 0%
(+0%)		 0%
(+0%)		 18.18%
(+18.18%)
🔴 ✨ packages/account/src/providers/transaction-request/transaction-request.ts 78.46%
(+78.46%)		 62.71%
(+62.71%)		 82.6%
(+82.6%)		 78.35%
(+78.35%)
🔴 ✨ packages/account/src/providers/transaction-request/utils.ts 87.5%
(+87.5%)		 85.71%
(+85.71%)		 100%
(+100%)		 87.5%
(+87.5%)
✅ ✨ packages/account/src/providers/transaction-request/witness.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-response/getDecodedLogs.ts 0%
(+0%)		 0%
(+0%)		 0%
(+0%)		 0%
(+0%)
🔴 ✨ packages/account/src/providers/transaction-response/transaction-response.ts 73.17%
(+73.17%)		 43.75%
(+43.75%)		 87.5%
(+87.5%)		 73.17%
(+73.17%)
🔴 ✨ packages/account/src/providers/transaction-summary/assemble-transaction-summary.ts 100%
(+100%)		 71.42%
(+71.42%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-summary/calculate-transaction-fee.ts 100%
(+100%)		 87.5%
(+87.5%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-summary/call.ts 90.47%
(+90.47%)		 50%
(+50%)		 100%
(+100%)		 90.47%
(+90.47%)
✅ ✨ packages/account/src/providers/transaction-summary/date.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-summary/get-transaction-summary.ts 0%
(+0%)		 0%
(+0%)		 0%
(+0%)		 0%
(+0%)
✅ ✨ packages/account/src/providers/transaction-summary/input.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-summary/operations.ts 98.18%
(+98.18%)		 81.13%
(+81.13%)		 94.28%
(+94.28%)		 97.39%
(+97.39%)
✅ ✨ packages/account/src/providers/transaction-summary/output.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/transaction-summary/receipt.ts 88.88%
(+88.88%)		 45.45%
(+45.45%)		 100%
(+100%)		 88.88%
(+88.88%)
🔴 ✨ packages/account/src/providers/transaction-summary/status.ts 100%
(+100%)		 90.9%
(+90.9%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/transaction-summary/types.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/utils/auto-retry-fetch.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/utils/block-explorer.ts 100%
(+100%)		 95.65%
(+95.65%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/utils/gas.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/utils/json.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/utils/merge-quantities.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/providers/utils/receipts.ts 94.44%
(+94.44%)		 83.33%
(+83.33%)		 100%
(+100%)		 94.54%
(+94.54%)
✅ ✨ packages/account/src/providers/utils/sleep.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/providers/utils/time.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/signer/signer.ts 96.55%
(+96.55%)		 80%
(+80%)		 100%
(+100%)		 96.55%
(+96.55%)
✅ ✨ packages/account/src/test-utils/generateTestWallet.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/test-utils/launchNode.ts 100%
(+100%)		 88.46%
(+88.46%)		 64.7%
(+64.7%)		 93.4%
(+93.4%)
🔴 ✨ packages/account/src/test-utils/seedTestWallet.ts 100%
(+100%)		 50%
(+50%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/utils/formatTransferToContractScriptData.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wallet/base-wallet-unlocked.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wallet/keystore-wallet.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wallet/wallet.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/wallet/wallets.ts 100%
(+100%)		 100%
(+100%)		 87.5%
(+87.5%)		 94.11%
(+94.11%)
🔴 ✨ packages/account/src/wallet-manager/types.ts 0%
(+0%)		 100%
(+100%)		 0%
(+0%)		 0%
(+0%)
✅ ✨ packages/account/src/wallet-manager/wallet-manager.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wallet-manager/storages/memory-storage.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
🔴 ✨ packages/account/src/wallet-manager/vaults/mnemonic-vault.ts 93.54%
(+93.54%)		 80%
(+80%)		 100%
(+100%)		 93.54%
(+93.54%)
✅ ✨ packages/account/src/wallet-manager/vaults/privatekey-vault.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wordlists/index.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)
✅ ✨ packages/account/src/wordlists/words/english.ts 100%
(+100%)		 100%
(+100%)		 100%
(+100%)		 100%
(+100%)

Base automatically changed from st/refactor/restructure-account to master February 10, 2024 13:50
@Dhaiwat10
Copy link
Member Author

Closing in favour of #1747. Too many merge conflicts :3

@Dhaiwat10 Dhaiwat10 closed this Feb 14, 2024
@arboleya arboleya deleted the dp/migrate-assets branch February 14, 2024 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@digorithm digorithm Awaiting requested review from digorithm digorithm will be requested when the pull request is marked ready for review digorithm is a code owner

@arboleya arboleya Awaiting requested review from arboleya arboleya will be requested when the pull request is marked ready for review arboleya is a code owner

@Torres-ssf Torres-ssf Awaiting requested review from Torres-ssf Torres-ssf will be requested when the pull request is marked ready for review Torres-ssf is a code owner

@danielbate danielbate Awaiting requested review from danielbate danielbate will be requested when the pull request is marked ready for review danielbate is a code owner

@nedsalk nedsalk Awaiting requested review from nedsalk nedsalk will be requested when the pull request is marked ready for review nedsalk is a code owner

@petertonysmith94 petertonysmith94 Awaiting requested review from petertonysmith94 petertonysmith94 will be requested when the pull request is marked ready for review petertonysmith94 is a code owner

@maschad maschad Awaiting requested review from maschad maschad will be requested when the pull request is marked ready for review maschad is a code owner

Assignees

@Dhaiwat10 Dhaiwat10

Labels
feat Issue is a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Export Asset/Networks Config Interface
3 participants
@Dhaiwat10 @Torres-ssf @danielbate