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

Package e2e testing #402

Merged
merged 9 commits into from Jan 23, 2023
Merged

Package e2e testing #402

merged 9 commits into from Jan 23, 2023

Conversation

jinmayamashita
Copy link
Collaborator

@jinmayamashita jinmayamashita commented Jan 23, 2023

Issue #391

What I did

Package the e2e test that is running now.

How to test

  • Is this testable with jest or e2e?
  • Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

@jinmayamashita jinmayamashita self-assigned this Jan 23, 2023
@jinmayamashita jinmayamashita changed the base branch from main to next-v1 January 23, 2023 08:39
@jinmayamashita jinmayamashita linked an issue Jan 23, 2023 that may be closed by this pull request
5 tasks
@@ -1,5 +1,6 @@
{
"$schema": "https://turbo.build/schema.json",
"globalEnv": ["CI"],
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"test": "turbo run build test --filter=!reactjs-boilerplate",
"test": "turbo run build --filter=create-codes test --filter=create-codes",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Only now, no other testing is involved except for the cli program testing.

"@types/node": "18.11.18",
"@types/react": "18.0.27",
"@types/react-dom": "18.0.10",
"@types/testing-library__jest-dom": "5.14.5",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since we do not plan to use jest globally in the future, we will avoid the type error by adding the primary type once.

refs. testing-library/jest-dom#123 (comment)

@jinmayamashita
Copy link
Collaborator Author

@jinmayamashita jinmayamashita marked this pull request as ready for review January 23, 2023 14:18
@jinmayamashita jinmayamashita merged commit 8e9cc4f into next-v1 Jan 23, 2023
@jinmayamashita jinmayamashita deleted the package-e2e-testing branch January 23, 2023 14:19
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.

Modularize the features provided by the current React boilerplate
1 participant