rename filter helpers

MetaMask · Dec 17, 2024 · dc0283e · dc0283e
1 parent ff57ba3
commit dc0283e
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 17 deletions.
diff --git a/packages/multichain/src/index.test.ts b/packages/multichain/src/index.test.ts
@@ -25,7 +25,7 @@ describe('@metamask/multichain', () => {
         "KnownWalletNamespaceRpcMethods",
         "KnownNotifications",
         "KnownWalletScopeString",
-        "filterScopeObjectsSupported",
+        "getSupportedScopeObjects",
         "parseScopeString",
         "normalizeScope",
         "mergeScopeObject",

diff --git a/packages/multichain/src/index.ts b/packages/multichain/src/index.ts
@@ -33,7 +33,7 @@ export {
   KnownNotifications,
   KnownWalletScopeString,
 } from './scope/constants';
-export { filterScopeObjectsSupported } from './scope/filter';
+export { getSupportedScopeObjects } from './scope/filter';
 export type {
   ExternalScopeString,
   ExternalScopeObject,

diff --git a/packages/multichain/src/scope/filter.test.ts b/packages/multichain/src/scope/filter.test.ts
@@ -1,8 +1,8 @@
 import * as Assert from './assert';
 import {
-  filterScopesSupported,
+  getSupportedScopes,
   bucketScopesBySupport,
-  filterScopeObjectsSupported,
+  getSupportedScopeObjects,
 } from './filter';
 import * as Supported from './supported';
 
@@ -20,11 +20,11 @@ jest.mock('./supported', () => ({
 const MockSupported = jest.mocked(Supported);
 
 describe('filter', () => {
-  describe('filterScopesSupported', () => {
+  describe('getSupportedScopes', () => {
     const isChainIdSupported = jest.fn();
 
     it('checks if each scope is supported', () => {
-      filterScopesSupported(
+      getSupportedScopes(
         {
           'eip155:1': {
             methods: ['a'],
@@ -68,7 +68,7 @@ describe('filter', () => {
       });
 
       expect(
-        filterScopesSupported(
+        getSupportedScopes(
           {
             'eip155:1': {
               methods: ['a'],
@@ -175,9 +175,9 @@ describe('filter', () => {
     });
   });
 
-  describe('filterScopeObjectsSupported', () => {
+  describe('getSupportedScopeObjects', () => {
     it('checks if each scopeObject method is supported', () => {
-      filterScopeObjectsSupported({
+      getSupportedScopeObjects({
         'eip155:1': {
           methods: ['method1', 'method2'],
           notifications: [],
@@ -222,7 +222,7 @@ describe('filter', () => {
         },
       );
 
-      const result = filterScopeObjectsSupported({
+      const result = getSupportedScopeObjects({
         'eip155:1': {
           methods: ['method1', 'method2'],
           notifications: [],
@@ -250,7 +250,7 @@ describe('filter', () => {
     });
 
     it('checks if each scopeObject notification is supported', () => {
-      filterScopeObjectsSupported({
+      getSupportedScopeObjects({
         'eip155:1': {
           methods: [],
           notifications: ['notification1', 'notification2'],
@@ -295,7 +295,7 @@ describe('filter', () => {
         },
       );
 
-      const result = filterScopeObjectsSupported({
+      const result = getSupportedScopeObjects({
         'eip155:1': {
           methods: [],
           notifications: ['notification1', 'notification2'],
@@ -323,7 +323,7 @@ describe('filter', () => {
     });
 
     it('does not modify accounts', () => {
-      const result = filterScopeObjectsSupported({
+      const result = getSupportedScopeObjects({
         'eip155:1': {
           methods: [],
           notifications: [],

diff --git a/packages/multichain/src/scope/filter.ts b/packages/multichain/src/scope/filter.ts
@@ -51,7 +51,7 @@ export const bucketScopesBySupport = (
  * @param hooks.isChainIdSupported - A helper that returns true if an eth chainId is currently supported by the wallet.
  * @returns a NormalizedScopesObject with only scopes that are currently supported.
  */
-export const filterScopesSupported = (
+export const getSupportedScopes = (
   scopes: NormalizedScopesObject,
   {
     isChainIdSupported,
@@ -73,7 +73,7 @@ export const filterScopesSupported = (
  * @param scopeObject - The NormalizedScopeObject to filter.
  * @returns a NormalizedScopeObject with only methods and notifications that are currently supported.
  */
-const filterScopeObjectSupported = (
+const getSupportedScopeObject = (
   scopeString: InternalScopeString,
   scopeObject: NormalizedScopeObject,
 ) => {
@@ -100,12 +100,12 @@ const filterScopeObjectSupported = (
  * @param scopes - The NormalizedScopesObject to filter.
  * @returns a NormalizedScopesObject with only methods, and notifications that are currently supported.
  */
-export const filterScopeObjectsSupported = (scopes: NormalizedScopesObject) => {
+export const getSupportedScopeObjects = (scopes: NormalizedScopesObject) => {
   const filteredScopesObject: NormalizedScopesObject = {};
 
   for (const [scopeString, scopeObject] of Object.entries(scopes)) {
     assertIsInternalScopeString(scopeString);
-    filteredScopesObject[scopeString] = filterScopeObjectSupported(
+    filteredScopesObject[scopeString] = getSupportedScopeObject(
       scopeString,
       scopeObject,
     );