chore: Enable typescript coverage for all the files

.eslintrc.js

@@ -190,5 +190,5 @@ module.exports = {
     radix: 0,
   },
 
-  ignorePatterns: ['wdio.conf.js', 'app/util/termsOfUse/termsOfUseContent.js'],
+  ignorePatterns: ['wdio.conf.js', 'app/util/termsOfUse/termsOfUseContent.ts'],
 };

.github/workflows/ci.yml

@@ -25,13 +25,23 @@ jobs:
         env:
           PACKAGE_READ_TOKEN: ${{ secrets.PACKAGE_READ_TOKEN }}
       - run: yarn setup:node
+      - name: Ingore changes in specific file
+        shell: bash
+        run: |
+          git update-index --assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
+      - run: yarn setup:node
       - name: Require clean working directory
         shell: bash
         run: |
           if ! git diff --exit-code; then
             echo "Working tree dirty at end of job"
             exit 1
           fi
+      - run: yarn setup:node
+      - name: Revert ignore changes in specific file
+        shell: bash
+        run: |
+          git update-index --no-assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
   dedupe:
     runs-on: ubuntu-20.04
     needs: setup
@@ -70,13 +80,23 @@ jobs:
           cache: yarn
       - run: yarn setup:node
       - run: yarn ${{ matrix['scripts'] }}
+      - run: yarn setup:node
+      - name: Ingore changes in specific file
+        shell: bash
+        run: |
+          git update-index --assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
       - name: Require clean working directory
         shell: bash
         run: |
           if ! git diff --exit-code; then
             echo "Working tree dirty at end of job"
             exit 1
           fi
+      - run: yarn setup:node
+      - name: Revert ignore changes in specific file
+        shell: bash
+        run: |
+          git update-index --no-assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
   unit-tests:
     runs-on: ubuntu-20.04
     needs: setup
@@ -104,13 +124,23 @@ jobs:
           name: coverage-${{ matrix.shard }}
           path: ./tests/coverage/coverage-${{ matrix.shard }}.json
           if-no-files-found: error
+      - run: yarn setup:node
+      - name: Ingore changes in specific file
+        shell: bash
+        run: |
+          git update-index --assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
       - name: Require clean working directory
         shell: bash
         run: |
           if ! git diff --exit-code; then
             echo "Working tree dirty at end of job"
             exit 1
           fi
+      - run: yarn setup:node
+      - name: Revert ignore changes in specific file
+        shell: bash
+        run: |
+          git update-index --no-assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
   merge-unit-tests:
     runs-on: ubuntu-20.04
     needs: unit-tests
@@ -135,20 +165,33 @@ jobs:
           name: coverage
           path: ./tests/merged-coverage/lcov.info
           if-no-files-found: error
+      - run: yarn setup:node
+      - name: Ingore changes in specific file
+        shell: bash
+        run: |
+          git update-index --assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
       - name: Require clean working directory
         shell: bash
         run: |
           if ! git diff --exit-code; then
             echo "Working tree dirty at end of job"
             exit 1
           fi
+      - run: yarn setup:node
+      - name: Revert ignore changes in specific file
+        shell: bash
+        run: |
+          git update-index --no-assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
   sonar-cloud:
     runs-on: ubuntu-20.04
     needs: merge-unit-tests
     steps:
       - uses: actions/checkout@v3
         with:
           fetch-depth: 0 # SonarCloud needs a full checkout to perform necessary analysis
+      - uses: actions/setup-node@v3
+        with:
+          node-version-file: '.nvmrc'
       - uses: actions/download-artifact@v3
         with:
           name: coverage
@@ -163,13 +206,23 @@ jobs:
         env:
           HAVE_SONAR_TOKEN: ${{ secrets.SONAR_TOKEN != '' }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+      - run: yarn setup:node
+      - name: Ingore changes in specific file
+        shell: bash
+        run: |
+          git update-index --assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
       - name: Require clean working directory
         shell: bash
         run: |
           if ! git diff --exit-code; then
             echo "Working tree dirty at end of job"
             exit 1
           fi
+      - run: yarn setup:node
+      - name: Revert ignore changes in specific file
+        shell: bash
+        run: |
+          git update-index --no-assume-unchanged app/util/termsOfUse/termsOfUseContent.ts
   check-workflows:
     name: Check workflows
     runs-on: ubuntu-latest

.gitignore

@@ -120,5 +120,5 @@ ccache
 html-report/
 
 # terms of use
-app/util/termsOfUse/termsOfUseContent.js
+app/util/termsOfUse/termsOfUseContent.ts
 docs/assets/termsOfUse.html

app/components/UI/AssetOverview/TokenDetails/TokenDetails.tsx

@@ -30,12 +30,12 @@ export interface TokenDetails {
 
 export interface MarketDetails {
   marketCap: string | null;
-  totalVolume: number | null;
+  totalVolume: string | null;
   volumeToMarketCap: string | null;
-  circulatingSupply: number | null;
+  circulatingSupply: string | null;
   allTimeHigh: string | null;
   allTimeLow: string | null;
-  fullyDiluted: number | null;
+  fullyDiluted: string | null;
 }
 
 interface TokenDetailsProps {

app/components/UI/Notification/NotificationMenuItem/Icon.test.tsx

@@ -29,7 +29,7 @@ describe('NotificationIcon', () => {
   it('matches snapshot when icon is provided', () => {
     const { toJSON } = renderWithProvider(
       <NotificationIcon
-        image={{ url: walletNotification.imageUrl }}
+        image={{ url: walletNotification.imageUrl.name }}
         badgeIcon={walletNotification.badgeIcon}
       />,
       { state: mockInitialState },

app/components/UI/Notification/NotificationMenuItem/__snapshots__/Icon.test.tsx.snap

@@ -38,7 +38,7 @@ exports[`NotificationIcon matches snapshot when icon is provided 1`] = `
           }
           source={
             {
-              "uri": "SvgMock",
+              "uri": "MockImage",
             }
           }
           style={

app/components/UI/ProfileSyncing/ProfileSyncing.test.tsx

@@ -31,7 +31,11 @@ const handleSwitchToggle = jest.fn();
 describe('ProfileSyncing', () => {
   it('should render correctly', () => {
     const { toJSON } = renderWithProvider(
-      <ProfileSyncingComponent handleSwitchToggle={handleSwitchToggle} />,
+      <ProfileSyncingComponent
+        handleSwitchToggle={handleSwitchToggle}
+        isBasicFunctionalityEnabled={false}
+        isProfileSyncingEnabled={false}
+      />,
     );
     expect(toJSON()).toMatchSnapshot();
   });

app/components/Views/ImportPrivateKey/index.tsx

@@ -6,6 +6,7 @@ import {
   Text,
   View,
   ActivityIndicator,
+  DimensionValue,
 } from 'react-native';
 import { useNavigation } from '@react-navigation/native';
 import StyledButton from '../../UI/StyledButton';
@@ -26,7 +27,7 @@ import { ImportAccountFromPrivateKeySelectorsIDs } from '../../../../e2e/selecto
 const ImportPrivateKey = () => {
   const [privateKey, setPrivateKey] = useState('');
   const [loading, setLoading] = useState(false);
-  const [inputWidth, setInputWidth] = useState(
+  const [inputWidth, setInputWidth] = useState<DimensionValue | undefined>(
     Device.isAndroid() ? '99%' : undefined,
   );
   const navigation = useNavigation();

app/components/Views/NetworkSelector/NetworkSelector.tsx

@@ -309,7 +309,7 @@ const NetworkSelector = () => {
     networkName: string,
   ) => {
     const searchResult: ExtendedNetwork[] = networks.filter(({ name }) =>
-      name.toLowerCase().includes(networkName.toLowerCase()),
+      name?.toLowerCase().includes(networkName.toLowerCase()),
     );
 
     return searchResult;

app/components/Views/SDK/SDKSessionsManager/SDKSessionItem.test.tsx

@@ -61,6 +61,7 @@ describe('SDKSessionItem', () => {
         icon: 'http://icon-url.com',
         platform: 'test',
         connected: true,
+        dappId: '123',
       },
     },
     trigger: 0,

app/components/Views/Settings/NetworksSettings/NetworkSettings/CustomNetworkView/CustomNetwork.types.ts

@@ -18,7 +18,7 @@ export interface Network {
 }
 
 export interface ExtendedNetwork extends Network {
-  name: string;
+  name?: string;
   formattedRpcUrl?: string | null;
 }
 

app/components/hooks/useAccounts/useAccounts.types.ts

@@ -1,9 +1,9 @@
 // Third party dependencies.
 import { KeyringTypes } from '@metamask/keyring-controller';
+import { Hex } from '@metamask/utils';
 
 // External dependencies.
-import { AvatarGroupToken } from '../../../component-library/components/Avatars/AvatarGroup/AvatarGroup.types';
-import { Hex } from '@metamask/utils';
+import { AvatarTokenProps } from '../../../component-library/components/Avatars/Avatar/variants/AvatarToken/AvatarToken.types';
 
 /**
  * Asset information associated with the account, which includes both the fiat balance and owned tokens.
@@ -16,7 +16,7 @@ export interface Assets {
   /**
    * Tokens owned by this account.
    */
-  tokens?: AvatarGroupToken[];
+  tokens?: AvatarTokenProps[];
 }
 
 /**

app/declarations/index.d.ts

@@ -277,7 +277,7 @@ declare module '@metamask/contract-metadata' {
   export default content;
 }
 
-declare module './util/termsOfUse/termsOfUseContent' {
+declare module './util/termsOfUse/termsOfUseContent.ts' {
   const content: string;
   export default content;
 }

app/util/termsOfUse/termsOfUseContent.ts

@@ -0,0 +1 @@
+export default '';

package.json

@@ -15,7 +15,7 @@
     "clean-android": "yarn clean:android && yarn clean:node",
     "lint": "eslint '**/*.{js,ts,tsx}'",
     "lint:fix": "eslint '**/*.{js,ts,tsx}' --fix",
-    "lint:tsc": "tsc --project ./tsconfig.lint.json",
+    "lint:tsc": "tsc --project ./tsconfig.json",
     "format": "prettier '**/*.{js,ts,tsx,json,feature}' --write",
     "setup": "yarn clean && node scripts/setup.mjs",
     "setup:node": "yarn clean:node && yarn allow-scripts && yarn patch-package",

scripts/setup.mjs

@@ -212,7 +212,7 @@ const generateTermsOfUseContentTask = {
   task: async () => {
     const termsOfUsePath = path.resolve('./docs/assets/termsOfUse.html');
     const outputDir = path.resolve('./app/util/termsOfUse');
-    const outputPath = path.join(outputDir, 'termsOfUseContent.js'); 
+    const outputPath = path.join(outputDir, 'termsOfUseContent.ts'); 
 
     let termsOfUse = '';
     try {

tsconfig.json

@@ -63,10 +63,10 @@
     // "emitDecoratorMetadata": true,         /* Enables experimental support for emitting type metadata for decorators. */
   },
   "include": [
+    "app/declarations/index.d.ts",
     "app/**/*",
     "node_modules/expect-webdriverio",
-    "node_modules/expect-webdriverio/jest",
-    "app/components/UI/QRHardware/AnimatedQRScanner.tsx"
+    "node_modules/expect-webdriverio/jest"
   ],
   "exclude": [
     "node_modules",