fixing AST comparison tests

prettier-solidity · Aug 25, 2024 · 453c6a1 · 453c6a1
1 parent 4ceae94
commit 453c6a1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/clean.ts b/src/clean.ts
@@ -1,5 +1,12 @@
 // Prettier offers a clean way to define ignored properties.
-const ignoredProperties = new Set(['loc', 'range', 'comments']);
+const ignoredProperties = new Set([
+  'loc',
+  'range',
+  'comments',
+  // this function is defined at constructor time so it won't pass AST
+  // comparisons.
+  'cleanModifierInvocationArguments'
+]);
 // eslint-disable-next-line @typescript-eslint/no-empty-function
 function clean(/* ast, newObj, parent */): void {}
 clean.ignoredProperties = ignoredProperties;

diff --git a/tests/config/run-format-test.js b/tests/config/run-format-test.js
@@ -358,9 +358,7 @@ async function runTest({
     if (isAstUnstableTest) {
       expect(formattedAst).not.toEqual(originalAst);
     } else {
-      expect(JSON.parse(JSON.stringify(formattedAst))).toEqual(
-        JSON.parse(JSON.stringify(originalAst)),
-      );
+      expect(formattedAst).toEqual(originalAst);
     }
   }