#668 remove custom stringify function in test

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## main
 
 ### ✨ Features and improvements
+- remove custom `stringify` function in test [#668](https://github.com/maplibre/maplibre-style-spec/issues/668))
 - _...Add new stuff here..._
 
 ### 🐞 Bug fixes

test/integration/expression/expression.test.ts

@@ -12,7 +12,6 @@ import {
 import ExpressionParsingError from '../../../src/expression/parsing_error';
 import {Result} from '../../../src/util/result';
 import {getGeometry} from '../../lib/geometry';
-import {stringify} from '../../lib/util';
 import {deepEqual, stripPrecision} from '../../lib/json-diff';
 
 const DECIMAL_SIGNIFICANT_FIGURES =  6;
@@ -52,7 +51,7 @@ describe('expression', () => {
                 delete fixture.metadata;
 
                 const fname = path.join(__dirname, expressionTestFileName);
-                fs.writeFileSync(fname, `${stringify(fixture)}\n`);
+                fs.writeFileSync(fname, JSON.stringify(fixture));
                 return;
             }
 

test/lib/util.ts

@@ -1,4 +1,3 @@
-import compactStringify from 'json-stringify-pretty-compact';
 import Color from '../../src/util/color';
 
 /**
@@ -27,17 +26,3 @@ export function expectToMatchColor(toTest: Color, expectedSerialized: string, nu
 export function expectCloseToArray(toTest: number[], expected: number[], numDigits = 5) {
     expect(toTest).toEqual(expected.map(n => isNaN(n) ? n : expect.closeTo(n, numDigits)));
 }
-
-// we have to handle this edge case here because we have test fixtures for this
-// edge case, and we don't want UPDATE=1 to mess with them
-export function stringify(v) {
-    let s = compactStringify(v);
-
-    if (s.indexOf('\u2028') >= 0) {
-        s = s.replace(/\u2028/g, '\\u2028');
-    }
-    if (s.indexOf('\u2029') >= 0) {
-        s = s.replace(/\u2029/g, '\\u2029');
-    }
-    return s;
-}