fix: number formatting on swap + send tx detail

[bergeron]

Description

Fixes an issue when rendering details for a swap + send transaction.

Token amounts less than 1 with >= 6 leading zeroes were formatted in scientific notation.

toString and toPrecision both render scientific in that case, so this PR uses toFixed instead.

Related issues

Fixes: #25876

Manual testing steps

1. Do a swap + send transaction with a source and/or destination amount < .000001
2. Click activity tab
3. Click on the transaction
4. Source and destination amounts should not be in scientific notation

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[greptile-apps]

PR Summary

(updates since last review)

This pull request addresses the issue of scientific notation in swap and send transaction details for token amounts less than 1 with six or more leading zeroes.

• shared/lib/transaction-controller-utils.test.js: Enhanced precision handling for token amounts in swap transactions, ensuring getSwapsTokensReceivedFromTxMeta respects precision arguments.
• ui/pages/swaps/swaps.util.ts: Updated formatSwapsValueForDisplay to handle small values without scientific notation.
• ui/components/app/transaction-breakdown/transaction-breakdown.test.js: Added test cases to ensure TransactionBreakdown correctly renders small token amounts without scientific notation.

These changes ensure accurate rendering of transaction details, especially for small token amounts.

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings}

[greptile-apps]

PR Summary

(updates since last review)

This pull request addresses the issue of scientific notation in swap and send transaction details for token amounts less than 1 with six or more leading zeroes.

• shared/lib/transaction-controller-utils.test.js: Enhanced precision handling for token amounts in swap transactions, ensuring getSwapsTokensReceivedFromTxMeta respects precision arguments.
• ui/pages/swaps/swaps.util.ts: Updated formatSwapsValueForDisplay to handle small values without scientific notation.
• ui/components/app/transaction-breakdown/transaction-breakdown.test.js: Added test cases to ensure TransactionBreakdown correctly renders small token amounts without scientific notation.

These changes ensure accurate rendering of transaction details, especially for small token amounts.

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings}

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[metamaskbot]

Builds ready [5b0eb3d]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5
  • common: 0, 1, 10, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (163 ± 196 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	72	299	109	47	23
		domContentLoaded	9	68	28	16	8
		load	38	1939	163	408	196
		domInteractive	9	68	28	16	8

Bundle size diffs [🚨 Warning! Bundle size has increased!]

• background: 0 Bytes (0.00%)
• ui: 45 Bytes (0.00%)
• common: 47 Bytes (0.00%)

[codecov]

Codecov Report

Attention: Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 69.67%. Comparing base (f861486) to head (5b0eb3d).

Files	Patch %	Lines
shared/lib/transactions-controller-utils.js	88.89%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #26029   +/-   ##
========================================
  Coverage    69.67%   69.67%           
========================================
  Files         1405     1405           
  Lines        49728    49733    +5     
  Branches     13748    13751    +3     
========================================
+ Hits         34646    34651    +5     
  Misses       15082    15082           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.