[Rule] test name should be trimed

[aymericbouzy]

I don't like when people leave white spaces at the beginning and end of tests. It's weird.

Valid code example

it("is a valid test name", () => {
  // code
})

Invalid code example

it(" is not a valid test name", () => {
  // code
})

it("is not valid either ", () => {
  // code
})

[benkimpel]

@aymericbouzy I haven't seen this one much in the wild, but I agree that's weird!! Seems like it could fall within the purview of this plugin.

[dangreenisrael]

👍

@aymericbouzy Is this something you would be interested in taking on?

[aymericbouzy]

I've never written an ESlint rule before 😅 but I'm not afraid of trying 👍 Will keep you posted.

[aymericbouzy]

Ideally I would like an ESlint rule that corrects grammar mistakes in test names 😂 some of my interns are very creative with English ... just kidding it would be a nightmare without real benefit 😉

[benkimpel]

@aymericbouzy If you'd like any help or would like to bounce ideas or plans off someone just let me know. (This is probably a pretty good rule to start with!)

[erunion]

This case is already covered under the valid-title rule in eslint-plugin-jest for what it's worth.

https://github.com/jest-community/eslint-plugin-jest/blob/HEAD/docs/rules/valid-title.md

[hockeybuggy]

Oh nice! Good catch @erunion . Looks like valid-title already exists and has a superset of the functionality described.

I am gonna close this to avoid duplicating effort. 😀

[benkimpel]

Nice! Thanks @erunion.

[dangreenisrael]

Thanks @erunion!