Plugin doesn't properly support tests with trailing spaces

[danon]

When I name my test like this: (with a trailing space)

import test from 'ava';

test('name ',async  t => {
  t.is('bar', 'bar');
});

and I try to run it with ava -m "name " it doesn't find the test

  × Couldn’t find any matching tests

To properly run it, you need to pass it without the space: ava -m "name". The plugin should remove the spaces from the test name.

Additionally, when there are two spaces in the test name:

test('foo   bar',async  t => {
  t.is('bar', 'bar');
});

You need to call it with just one space: ava -m "foo bar". Calling it as-is, ava -m "foo bar", then it will fail too.

[eirikb]

Thank you for the issue

Do you think a simple

.trim().replace("  +".toRegex(), " "))

would suffice? Feels a bit hacky, but whatever works?

[danon]

Thank you for the issue

Do you think a simple

.trim().replace("  +".toRegex(), " "))

would suffice? Feels a bit hacky, but whatever works?

For the cases I have found, yes - it will suffice.

I am not entirely positive there aren't any more cases I haven't stumbled upon though.

[eirikb]

Is it just me, or does it seem like weird handling of AVA?

Anyway I will make a PR soon.

[danon]

Is it just me, or does it seem like weird handling of AVA?

Anyway I will make a PR soon.

Well, kinda.

For example these tests:

test('one  one', t => t.is(true, true));
test('one one',  t => t.is(true, true));

will raise

   22: test('one one', t => {
   23:   t.is(true, true);   

  Error: Duplicate test title: one one   

I guess Ava normalizes some names to its "ava form", and that's what you have to reffer the tests with.