vitest/lower-case-title autofix is broken

[qeleb]

vitest/lower-case-title autofix is broken

[veritem]

Would you mind sending a repro?

Here there is different use cases for this I can add yours if it's not covered

[qeleb]

I haven’t got time to send over a repro today & my issue is in a private repo.

this test is passing? i’ve had this same bug since 0.0.20.

I’ve also found that expect-expect errors on run (something about length of property undefined), and no-identical-title seems not to do anything

[veritem]

@qeleb, those were fixed a few weeks ago. update the package to the latest version

[qeleb]

i’m using latest

[veritem]

@qeleb I would love it if you could share a sample code for something that's not working. This way, I can look into it and fix it as soon as possible.

[veritem]

Hi, let me know if you're still experiencing the same issue. If that's the case, feel free to re-open this

[qeleb]

Hi! Yes, please reopen this issue.

Unfortunately, all of these problems still exist in 0.0.30.
I've just done a test this morning to verify.

It has improved massively since I got this eslint plugin weeks ago, but some are still not working, so my eslint config looks like this:

env: { 'vitest-globals/env': true },
rules: {
  'vitest/consistent-test-it': ['error', { fn: 'it' }],
  //'vitest/expect-expect': 'error', // broken
  //'vitest/lower-case-title': 'warn', //autofix broken
  'vitest/max-nested-describe': ['error', { max: 3 }],
  'vitest/no-conditional-tests': 'error',
  'vitest/no-focused-tests': 'warn',
  //'vitest/no-identical-title': 'error', // broken
  'vitest/no-skipped-tests': 'warn',
},

vitest/expect-expect & vitest/no-identical-title don't work at all, but lower-case-title is really close to working.
It correctly identifies tests with uppercase letters, but autofix litters the titles with extra spaces.

Example:
[GET] /404: missing route => [ g e t] /404: missing route

[veritem]

Please use the latest version, I'm not sure why this is still occurring for you here is my example. It works perfectly https://github.com/veritem/eslint-plugin-vitest/tree/main/fixtures

[qeleb]

@veritem
try making it say “[TEST]: This is a Test”
i think some characters like the square bracket are breaking it.

[veritem]

fyi, It capitalizes the first character. If it can't be capitalized let's say it's a number or special character it leave it as it is.

[qeleb]

then why is the current version adding a bunch of spaces on autofix? something is not adding up…

[veritem]

I haven't seen this may be an example would help me fix it asp. I have tried this [TEST]: This is a Test it doesn't do anything to the text.