cypress icon indicating copy to clipboard operation
cypress copied to clipboard

CT: cypress doesn't recognize "+" symbol in the path to the Angular component

Open bobige opened this issue 1 year ago • 4 comments

Current behavior

Importing an angular component into a test file, when the component folder name starts with a "+" symbol, is not possible.

Desired behavior

Cypress imports components that have the "+" symbol at the start of the containing folder name.

Test code to reproduce

  1. Create a new Angular 15 project.
  2. Create a component in a folder that has the "+" sign in front of it
  3. Create a basic test file for the component and just mount it.
  4. See error

Cypress Version

13.9.0

Node version

v16.18.1

Operating System

Windows 10.0.19045 Build 19045

Debug Logs

No response

Other

I am configuring Cypress to test Angular 15 components. We have a naming convention to put the "+" sign in front of the folder name, containing the component files. When I mount the component, cypress gives a "No tests found" error, which according to the documentation could be caused by a syntax error.

It is easily reproduceable. Create a new Angular 15 project and create a component in a folder that has the "+" sign in front of it. I also use Chrome 124:

image

If I remove the "+" sign or even change the naming to have a different special character, such as "_", it works:

image

bobige avatar May 09 '24 08:05 bobige

@jennifer-shehane Hello ! Can I solve this problem ?

H0onnn avatar May 09 '24 23:05 H0onnn

@H0onnn Yes, please follow our contributing guidelines and write a test for the behavior and we could review.

jennifer-shehane avatar May 10 '24 15:05 jennifer-shehane

@jennifer-shehane Hello, is there a way to check if the test works properly with the modified code in my local environment?

H0onnn avatar May 14 '24 05:05 H0onnn

@H0onnn @jennifer-shehane Hello, is there any progress on this issue?

bobige avatar May 22 '24 12:05 bobige

@bobige No, this is open to contributors if someone wants to make a PR though

jennifer-shehane avatar May 23 '24 18:05 jennifer-shehane