bug: CLI args of test names not propagated to Jest

[duhem-s]

Stencil version:

 @stencil/core@2.19.x

Current behavior:
I used to do npm run test:unit -- mg-form to only test my mg-form component but since the version 2.19.x I'm not able to do so, all my unit tests are launched.

My test:unit script run stencil test --spec.

Expected behavior:
Be able to run a specific test file.

GitHub Reproduction Link:
https://github.com/duhem-s/stencil-cli

Other information:

With the command: npm run test -- second on the GH reproduction :

With stencil@2.18.1

With stencil@2.19.0

[rwaskiewicz]

Thanks @duhem-s! I confirmed this was introduced in v2.19.0. I'm going to label this for the team to take a closer look

[alicewriteswrongs]

Is it possible this is a duplicate of #3785?

[alicewriteswrongs]

I just confirmed that this is the same issue as #3785, I have a fix which fixes the problem in both the reproduction for that issue and this one. I'm going to open a PR for that shortly, but I'm going to close this issue for bookkeeping purposes.

[duhem-s]

Hi @alicewriteswrongs 👋

I opened a new one #3828 do you think your PR will also fix this ?

[alicewriteswrongs]

Unfortunately that does look like a separate issue - at present we have a manually-defined set of short aliases like that which we support, and -u is, sadly, not among them at present 😢

[alicewriteswrongs]

Ok so I just realized, belatedly, that #3785 is not the same issue as this one, so I'm going to re-open this issue so we can keep track of making sure that our next release fixes it.

Sorry for the noise!

[rwaskiewicz]

@duhem-s 👋

I was able to verify that as of Stencil v2.20.0, this issue has been fixed using your reproduction case:

I'm going to close this as a result. Thanks again for the detailed reproduction case, it was helpful both in triaging the issue and verifying the fix worked!

Should this bug reappear, please feel free to open a new issue. Thanks!