Merge pull request #8387 from cfpb/ans_remove_string_utils

Remove `stringEscape` and `stringMatch`

cfgov/unprocessed/js/modules/util/strings.js

@@ -29,23 +29,4 @@ function formatTimestamp(totalSeconds) {
   return timestamp;
 }
 
-/**
- * Escapes a string.
- * @param {string} s - The string to escape.
- * @returns {string} The escaped string.
- */
-function stringEscape(s) {
-  return s.replace(/[-\\^$*+?.()|[\]{}]/g, '\\$&');
-}
-
-/**
- * Tests whether a string matches another.
- * @param {string} x - The control string.
- * @param {string} y - The comparison string.
- * @returns {boolean} True if `x` and `y` match, false otherwise.
- */
-function stringMatch(x, y) {
-  return RegExp(stringEscape(y.trim()), 'i').test(x);
-}
-
-export { formatTimestamp, stringEscape, stringMatch };
+export { formatTimestamp };

test/unit_tests/js/modules/util/strings-spec.js

@@ -1,10 +1,4 @@
-import {
-  formatTimestamp,
-  stringEscape,
-  stringMatch,
-} from '../../../../../cfgov/unprocessed/js/modules/util/strings.js';
-let string;
-let control;
+import { formatTimestamp } from '../../../../../cfgov/unprocessed/js/modules/util/strings.js';
 
 describe('Strings formatTimestamp()', () => {
   it('should convert 23 seconds into 00:23 timestamp', () => {
@@ -22,46 +16,3 @@ describe('Strings formatTimestamp()', () => {
     expect(formatTimestamp(seconds)).toBe('04:26:41');
   });
 });
-
-describe('Strings stringEscape()', () => {
-  it('should escape a url', () => {
-    string = 'https://google.com';
-
-    expect(stringEscape(string)).toBe('https://google\\.com');
-  });
-
-  it('should escape a hyphenated name', () => {
-    string = 'Miller-Webster';
-
-    expect(stringEscape(string)).toBe('Miller\\-Webster');
-  });
-
-  it('should escape a comma', () => {
-    string = 'Students, Parents, and Teachers';
-
-    expect(stringEscape(string)).toBe('Students, Parents, and Teachers');
-  });
-});
-
-describe('Strings stringMatch()', () => {
-  it('should return true when testing matching strings', () => {
-    string = 'Test String';
-    control = 'Test String';
-
-    expect(stringMatch(control, string)).toBe(true);
-  });
-
-  it('should return true when testing matching strings with differing casing', () => {
-    string = 'test string';
-    control = 'Test String';
-
-    expect(stringMatch(control, string)).toBe(true);
-  });
-
-  it('should return false when testing differing strings', () => {
-    string = 'Test String';
-    control = 'Result String';
-
-    expect(stringMatch(control, string)).toBe(false);
-  });
-});