refactor: - Reuse fieldRegex() for priorityRegex

claremacrae · claremacrae · commit 4284e310120f · 2024-11-21T14:06:28.000Z
diff --git a/src/TaskSerializer/DefaultTaskSerializer.ts b/src/TaskSerializer/DefaultTaskSerializer.ts
@@ -95,7 +95,7 @@ export const DEFAULT_SYMBOLS: DefaultTaskSerializerSymbols = {
         // The following regex's end with `$` because they will be matched and
         // removed from the end until none are left.
         // \uFE0F? allows an optional Variant Selector 16 on emojis.
-        priorityRegex: /([🔺⏫🔼🔽⏬])\uFE0F?$/u,
+        priorityRegex: fieldRegex('([🔺⏫🔼🔽⏬])\uFE0F?$', ''),
         startDateRegex: dateFieldRegex('🛫'),
         createdDateRegex: dateFieldRegex('➕'),
         scheduledDateRegex: dateFieldRegex('[⏳⌛]'),
diff --git a/tests/TaskSerializer/DefaultTaskSerializer.test.ts b/tests/TaskSerializer/DefaultTaskSerializer.test.ts
@@ -65,7 +65,7 @@ describe('validate emoji regular expressions', () => {
     it('regular expressions should have expected source', () => {
         expect(generateRegexApprovalTest()).toMatchInlineSnapshot(`
             "
-            priorityRegex: /([🔺⏫🔼🔽⏬])\\uFE0F?$/u
+            priorityRegex: /([🔺⏫🔼🔽⏬])\\ufe0f?$/u
             startDateRegex: /🛫 *(\\d{4}-\\d{2}-\\d{2})$/u
             createdDateRegex: /➕ *(\\d{4}-\\d{2}-\\d{2})$/u
             scheduledDateRegex: /[⏳⌛] *(\\d{4}-\\d{2}-\\d{2})$/u

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ describe('validate emoji regular expressions', () => {`
`65`	`65`	`it('regular expressions should have expected source', () => {`
`66`	`66`	expect(generateRegexApprovalTest()).toMatchInlineSnapshot(`
`67`	`67`	`"`
`68`		`- priorityRegex: /([🔺⏫🔼🔽⏬])\\uFE0F?$/u`
	`68`	`+ priorityRegex: /([🔺⏫🔼🔽⏬])\\ufe0f?$/u`
`69`	`69`	`startDateRegex: /🛫 *(\\d{4}-\\d{2}-\\d{2})$/u`
`70`	`70`	`createdDateRegex: /➕ *(\\d{4}-\\d{2}-\\d{2})$/u`
`71`	`71`	`scheduledDateRegex: /[⏳⌛] *(\\d{4}-\\d{2}-\\d{2})$/u`