improve comment and add explanation for icon name setting

Logaqm · Sep 16, 2022 · b3ab1ba · b3ab1ba
1 parent b6d9615
commit b3ab1ba
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/jest/setup.js b/jest/setup.js
@@ -62,11 +62,10 @@ jest.mock('../src/components/Icon/Expensicons', () => {
     const reduce = require('underscore').reduce;
     const Expensicons = jest.requireActual('../src/components/Icon/Expensicons');
     return reduce(Expensicons, (prev, _curr, key) => {
+        // We set the name of the anonymous mock function here so we can dynamically build the list of mocks and access the
+        // "name" property to use in accessibility hints for element querying
         const fn = () => '';
-        Object.defineProperty(fn, 'name', {
-            value: key,
-            configurable: true,
-        });
+        Object.defineProperty(fn, 'name', {value: key});
         return {...prev, [key]: fn};
     }, {});
 });