Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Wilco Fiers <WilcoFiers@users.noreply.github.com>
  • Loading branch information
straker and WilcoFiers committed Jul 26, 2023
1 parent a4a8e52 commit 564a89f
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions test/commons/text/unicode.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,8 @@ describe('text.hasUnicode', () => {
});

it('returns true when given string has format unicode', () => {
// the front of this string contains a zero-width character
const actual = axe.commons.text.hasUnicode('⁩Hello World', {
// zero-width spacer character U+200B
const actual = axe.commons.text.hasUnicode('\u200BHello World', {
nonBmp: true
});
assert.isTrue(actual);
Expand Down Expand Up @@ -181,8 +181,8 @@ describe('text.hasUnicode', () => {
});

it('returns true when given format unicode characters', () => {
// the front of this string contains a zero-width character
const actual = axe.commons.text.hasUnicode('⁩Hello World', {
// zero-width spacer character U+200B
const actual = axe.commons.text.hasUnicode('\u200BHello World', {
emoji: true,
nonBmp: true,
punctuations: true
Expand Down Expand Up @@ -264,8 +264,8 @@ describe('text.removeUnicode', () => {
});

it('returns string removing format unicode', () => {
// the front of this string contains a zero-width character
const actual = axe.commons.text.removeUnicode('⁩Hello World', {
// zero-width spacer character U+200B
const actual = axe.commons.text.removeUnicode('\u200BHello World', {
nonBmp: true
});
assert.equal(actual, 'Hello World');
Expand Down

0 comments on commit 564a89f

Please sign in to comment.