Skip to content

Fix lint errors in Jest tests #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wimpyprogrammer merged 1 commit into from
May 12, 2021
Merged

Fix lint errors in Jest tests #16

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
109 changes: 52 additions & 57 deletions src/pattern.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1142,13 +1142,16 @@ describe('expand', () => {

it.each<[string, RegExp]>([
// From https://www.regular-expressions.info/refcharclass.html
['Character class subtraction', /[c-m-[j-z]]/],
['Character class intersection', /[a-i&&c-z]/],
['Character class nested intersection', /[a-i&&[c-z]]/],
])('does not recognize RegEx syntax: %s %p', (_: string, input: RegExp) => {
const result = expandAll(input);
expect(result).not.toEqual(['c', 'd', 'e', 'f', 'g', 'h', 'i']);
});
['subtraction', /[c-m-[j-z]]/],
['intersection', /[a-i&&c-z]/],
['nested intersection', /[a-i&&[c-z]]/],
])(
'does not recognize RegEx syntax: character class %s %p',
(_: string, input: RegExp) => {
const result = expandAll(input);
expect(result).not.toEqual(['c', 'd', 'e', 'f', 'g', 'h', 'i']);
}
);

[
// From https://www.regular-expressions.info/posixbrackets.html#class
Expand Down Expand Up @@ -1177,10 +1180,10 @@ describe('expand', () => {
'V', // vertical whitespace
].forEach((posixClass) => {
it.each([
['POSIX class', `[[:${posixClass}:]]`],
['negative POSIX class', `[[:^${posixClass}:]]`],
['', `[[:${posixClass}:]]`],
['negative', `[[:^${posixClass}:]]`],
])(
'does not recognize RegEx syntax: %s /%s/',
'does not recognize RegEx syntax: %s POSIX class /%s/',
(_: string, input: string) => {
const result = expandAll(input);
expect(result.pop()).toEqual(':]');
Expand All @@ -1205,12 +1208,9 @@ describe('expand', () => {
'Word',
'XDigit',
].forEach((javaPosixClass) => {
it.each([
['Java POSIX class', `\p{${javaPosixClass}}`],
['Java POSIX class', `\p{Is${javaPosixClass}}`],
])(
'does not recognize RegEx syntax: %s /%s/',
(_: string, input: string) => {
it.each([`\p{${javaPosixClass}}`, `\p{Is${javaPosixClass}}`])(
`does not recognize RegEx syntax: Java ${javaPosixClass} POSIX class /%s/`,
(input: string) => {
const result = expandAll(input);
expect(result).toHaveLength(1);
expect(result[0]).toMatch(/^p{/);
Expand Down Expand Up @@ -1396,7 +1396,7 @@ describe('expand', () => {
['', `\p{In${unicodeBlock}}`],
['lowercase', `\p{In${unicodeBlock.toLowerCase()}}`],
])(
'does not recognize RegEx syntax: Unicode block %s /%s/',
`does not recognize RegEx syntax: ${unicodeBlock} Unicode block %s /%s/`,
testUnicodeBlock
);

Expand All @@ -1406,7 +1406,7 @@ describe('expand', () => {
['hyphens for underscores', `\p{${unicodeBlock.replace('_', '-')}}`],
['spaces for underscores', `\p{${unicodeBlock.replace('_', ' ')}}`],
])(
'does not recognize RegEx syntax: Unicode block %s /%s/',
`does not recognize RegEx syntax: ${unicodeBlock} Unicode block %s /%s/`,
testUnicodeBlock
);
}
Expand All @@ -1417,7 +1417,7 @@ describe('expand', () => {
['underscores for hyphens', `\p{${unicodeBlock.replace('-', '_')}}`],
['spaces for hyphens', `\p{${unicodeBlock.replace('-', ' ')}}`],
])(
'does not recognize RegEx syntax: Unicode block %s /%s/',
`does not recognize RegEx syntax: ${unicodeBlock} Unicode block %s /%s/`,
testUnicodeBlock
);
}
Expand Down Expand Up @@ -1507,7 +1507,7 @@ describe('expand', () => {
['Longhand format', `\p{Is${unicodeCategory}}`],
['Longhand negative format', `\p{^${unicodeCategory}}`],
])(
'does not recognize RegEx syntax: Unicode category %s /%s/',
`does not recognize RegEx syntax: ${unicodeCategory} Unicode category %s /%s/`,
(_: string, input: string) => {
const result = expandAll(input);
expect(result).toHaveLength(1);
Expand All @@ -1519,7 +1519,7 @@ describe('expand', () => {
['Longhand negative format', `\P{${unicodeCategory}}`],
['Longhand double negative format', `\P{^${unicodeCategory}}`],
])(
'does not recognize RegEx syntax: Unicode category %s /%s/',
`does not recognize RegEx syntax: ${unicodeCategory} Unicode category %s /%s/`,
(_: string, input: string) => {
const result = expandAll(input);
expect(result).toHaveLength(1);
Expand All @@ -1530,18 +1530,18 @@ describe('expand', () => {

// From https://www.regular-expressions.info/unicode.html#category
['L', 'M', 'Z', 'S', 'N', 'P', 'C'].forEach((unicodeCategory) => {
it.each([['Shorthand format', `\p${unicodeCategory}`]])(
'does not recognize RegEx syntax: Unicode category %s /%s/',
(_: string, input: string) => {
it.each([`\p${unicodeCategory}`])(
'does not recognize RegEx syntax: Unicode category shorthand format /%s/',
(input: string) => {
const result = expandAll(input);
expect(result).toHaveLength(1);
expect(result[0]).toMatch(/^p/);
}
);

it.each([['Shorthand negative format', `\P${unicodeCategory}`]])(
'does not recognize RegEx syntax: Unicode category %s /%s/',
(_: string, input: string) => {
it.each([`\P${unicodeCategory}`])(
'does not recognize RegEx syntax: Unicode category shorthand negative format /%s/',
(input: string) => {
const result = expandAll(input);
expect(result).toHaveLength(1);
expect(result[0]).toMatch(/^P/);
Expand All @@ -1560,7 +1560,7 @@ describe('expand', () => {
expect(result).toEqual(['X']);
});

it.each<[string, RegExp]>([
it.each<[string, RegExp | string]>([
// From https://www.regular-expressions.info/refcharacters.html
['Escape sequence', /Qab abE/],
['Octal escape sequence', /\o{141}\o{142} \o{141}\o{142}/],
Expand All @@ -1586,59 +1586,54 @@ describe('expand', () => {
// From https://www.regular-expressions.info/refadv.html
['backreference in lookbehind', /(ab) (?<=\1)/],
['marker to ignore preceeding text', /ignore this \Kab ab/],
])('does not recognize RegEx syntax: %s %p', (_: string, input: RegExp) => {
const result = expandAll(input);
expect(result).not.toEqual(['ab ab']);
});

it.each([
// From https://www.regular-expressions.info/refext.html
['numbered backreference before group', '\\1 (ab)'],
['named backreference before group', '\\k<x> (?<x>ab)'],
['backreference', "(?<x>ab) \\k'x'"],
['backreference', '(?<x>ab) \\k{x}'],
['backreference', '(?<x>ab) \\g{x}'],
['failed backreference', '(?<x>ab)? \\k<x>'],
])(
'does not recognize RegEx syntax: %s /%s/',
(_: string, input: string) => {
const result = expandAll(input);
expect(result).not.toEqual(['ab ab']);
}
);
])('does not recognize RegEx syntax: %s %p', (_, input) => {
const result = expandAll(input);
expect(result).not.toEqual(['ab ab']);
});

it.each<[string, RegExp]>([
it.each<RegExp>([
// From https://www.regular-expressions.info/refrecurse.html
['recursion', /a\g<0>?z/],
['recursion', /ag'0'?z/],
])('does not recognize RegEx syntax: %s %p', (_: string, input: RegExp) => {
/a\g<0>?z/,
/ag'0'?z/,
])('does not recognize RegEx syntax: recursion %p', (input: RegExp) => {
const result = expandN(input, 3);
expect(result).not.toEqual(['az', 'aazz', 'aaazzz']);
});

it.each([
// From https://www.regular-expressions.info/refrecurse.html
['subroutine call', 'a(b\\g<1>?y)z'],
['subroutine call', "a(b\\g'1'?y)z"],
['relative subroutine call', 'a(b\\g<-1>?y)z'],
['relative subroutine call', "a(b\\g'-1'?y)z"],
['named subroutine call', 'a(?<x>b\\g<x>?y)z'],
['', 'a(b\\g<1>?y)z'],
['', "a(b\\g'1'?y)z"],
['relative', 'a(b\\g<-1>?y)z'],
['relative', "a(b\\g'-1'?y)z"],
['named', 'a(?<x>b\\g<x>?y)z'],
])(
'does not recognize RegEx syntax: %s /%s/',
'does not recognize RegEx syntax: %s subroutine call /%s/',
(_: string, input: string) => {
const result = expandN(input, 3);
expect(result).not.toEqual(['abyz', 'abbyyz', 'abbbyyyz']);
}
);

it.each<[string, RegExp]>([
it.each<RegExp>([
// From https://www.regular-expressions.info/refrecurse.html
['forward subroutine call', /\g<+1>x([ab])/],
['forward subroutine call', /\g'+1'x([ab])/],
])('does not recognize RegEx syntax: %s %p', (_: string, input: RegExp) => {
const result = expandAll(input);
expect(result).not.toEqual(['axa', 'axb', 'bxa', 'bxb']);
});
/\g<+1>x([ab])/,
/\g'+1'x([ab])/,
])(
'does not recognize RegEx syntax: forward subroutine call %p',
(input: RegExp) => {
const result = expandAll(input);
expect(result).not.toEqual(['axa', 'axb', 'bxa', 'bxb']);
}
);
});
});

Expand Down