Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: change file extension #2086

Merged
merged 1 commit into from
May 5, 2024

Conversation

talkor
Copy link
Member

@talkor talkor commented Apr 21, 2024

.jest => .test

@talkor talkor requested a review from a team as a code owner April 21, 2024 17:11
Copy link
Contributor

@YossiSaadi YossiSaadi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had a few question

@@ -59,7 +59,7 @@ const commonExtends = ["plugin:react/recommended", "plugin:react-hooks/recommend
module.exports = {
overrides: [
{
files: ["*.jest.js", "jest.init.js"],
files: ["*.test.js", "jest.init.js"],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it does not take into consideration *.test.tsx or *.test.ts
what is the meaning of it not catching it?

},
{
type: "add",
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-tests.jest.js",
templateFile: "plop/general/component-tests-jest.txt"
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-tests.test.js",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the file name dropped the jest, not replaced it with test

@@ -53,13 +53,13 @@ module.exports = plop => {
},
{
type: "add",
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-snapshot-tests.jest.js",
templateFile: "plop/general/component-snapshot-tests-jest.txt"
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-snapshot-tests.test.js",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the file name dropped the jest, not replaced it with test

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you dropped the jest here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed it completely, we don't use it anyway, we use snapshot testing inside the test files

},
{
type: "add",
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-tests.jest.js",
templateFile: "plop/general/component-tests-jest.txt"
path: "src/components/{{properCase componentName}}/__tests__/{{camelCase componentName}}-tests.test.js",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the file name dropped the jest, not replaced it with test

@@ -39,7 +39,7 @@ const commonExtends = ['plugin:react/recommended', 'plugin:prettier/recommended'
module.exports = {
overrides: [
{
files: ['*.jest.js', 'jest.init.js'],
files: ['*.test.js', 'jest.init.js'],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as before, what's the meaning of it not catching ts{x}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed it as it has no effect

packages/storybook-blocks/package.json Show resolved Hide resolved
@talkor talkor merged commit a89364a into talko/test/test-infra May 5, 2024
10 checks passed
@talkor talkor deleted the talko/test/rename-test-files branch May 5, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants