Chore: enable eslint-plugin/test-case-shorthand-strings.

[aladdin-add]

What is the purpose of this pull request? (put an "X" next to item)

[x] Other, please explain:

What changes did you make? (Give an overview)
enable eslint-plugin/test-case-shorthand-strings.

Is there anything you'd like reviewers to focus on?
no.

[eslintbot]

LGTM

[mention-bot]

@aladdin-add, thanks for your PR! By analyzing the history of the files in this pull request, we identified @mysticatea, @not-an-aardvark and @cschuller to be potential reviewers.

[eslintbot]

LGTM

[not-an-aardvark]

It would be nice to make the code example aligned with the unIndent call here:

// currently:

unIndent`
        foo;
    `

// after:
unIndent`
    foo
`

(Unfortunately, indent doesn't check this because the template string actually contains spaces. However, the unIndent function ignores the spaces.)

However, it seems like manually changing everything would probably be very tedious. Separately, maybe we should add something to lib/internal-rules to enforce "indentation" of code samples inside template literals.

[eslintbot]

LGTM

[not-an-aardvark]

Looks good to me, thanks!

It looks like there is a merge conflict.

[aladdin-add]

copy it. I'll fix it asap.

fixed. @not-an-aardvark

[eslintbot]

LGTM

[jsf-clabot]

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Aladdin-ADD
❌ 薛定谔的猫

---

薛定谔的猫 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

[eslintbot]

LGTM

[eslintbot]

LGTM

[not-an-aardvark]

Thanks for contributing!