refactor: code review changes

Author: mdjastrzebski

.gitignore

@@ -32,3 +32,4 @@ buck-out/
 lib/
 
 /.idea
+coverage/

src/__tests__/compiler.test.tsx

@@ -29,7 +29,7 @@ test('regex constructor', () => {
   expect(buildRegex('a').test('b')).toBeFalsy();
 });
 
-test('buildPattern escapes special characters', () => {
+test('"buildPattern" escapes special characters', () => {
   expect(buildPattern('.')).toBe('\\.');
   expect(buildPattern('*')).toBe('\\*');
   expect(buildPattern('+')).toBe('\\+');
@@ -44,4 +44,8 @@ test('buildPattern escapes special characters', () => {
   expect(buildPattern('\\')).toBe('\\\\');
 
   expect(buildPattern('*.*')).toBe('\\*\\.\\*');
+
+  expect(buildPattern(oneOrMore('.*'), zeroOrMore('[]{}'))).toBe(
+    '(?:\\.\\*)+(?:\\[\\]\\{\\})*'
+  );
 });

src/compiler.ts

@@ -2,6 +2,7 @@ import type { RegexElement } from './types';
 import { characterClasses, isCharacterClass } from './character-classes';
 import { baseQuantifiers, isBaseQuantifier } from './quantifiers/base';
 import { compileRepeat } from './quantifiers/repeat';
+import { escapeText } from './utils';
 
 /**
  * Generate RegExp object for elements.
@@ -52,8 +53,3 @@ function compileSingle(element: RegexElement): string {
   // @ts-expect-error User passed incorrect type
   throw new Error(`Unknown elements type ${element.type}`);
 }
-
-// Source: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_expressions#escaping
-function escapeText(text: string) {
-  return text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); // $& means the whole matched string
-}

src/utils.ts

@@ -7,3 +7,8 @@
 export function wrapGroup(regex: string): string {
   return regex.length === 1 ? regex : `(?:${regex})`;
 }
+
+// Source: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_expressions#escaping
+export function escapeText(text: string) {
+  return text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); // $& means the whole matched string
+}