javascript icon indicating copy to clipboard operation
javascript copied to clipboard

Published 20 hours ago • verified publisher icon Yoast

Move A11yNotice out of the tests folder

Open igorschoester opened this issue 6 years ago • 3 comments

Explanation

A11yNotice is a styled component, not a test helper. It is located in the src/tests folder and should be just in src.

Technical decisions

Feedback?

igorschoester avatar Apr 17 '19 14:04 igorschoester

I think it has to do with the fact that they have been moved to separate packages.

My hypothesis is that jest-styled-components behaves differently in regard to relative imports than in regard to @yoast/* imports. It would make sense that jest-styled-components doesn't care about "external" packages.

abotteram avatar Apr 18 '19 07:04 abotteram

Good point, maybe it is fine. Still would be nice to be confirmed though. And the moving out of the test folder still remains.

igorschoester avatar Apr 18 '19 08:04 igorschoester

As discussed with Anton, repurposing this PR for only the first point (moving from the test folder).

The second (no A11yNotice styled components output in snapshots) is no longer important enough. It might be expected behavior as Alexander says above. Or it may even be fixed within jest-styled-components itself if it is a styled-components v4 issue.

igorschoester avatar Apr 18 '19 11:04 igorschoester