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 Coverage Improvement #2644

Open
62 tasks
ElderJames opened this issue Aug 27, 2022 · 7 comments · Fixed by #3477
Open
62 tasks

Test Coverage Improvement #2644

ElderJames opened this issue Aug 27, 2022 · 7 comments · Fixed by #3477
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Milestone

Comments

@ElderJames
Copy link
Member

ElderJames commented Aug 27, 2022

We plan to release the official 1.0 version when the coverage reaches the desired value > 90%.

The following are all of the components currently in ant-design. Let's add test for them together.

We can get the detail by the codecov report:

https://app.codecov.io/gh/ant-design-blazor/ant-design-blazor

@ElderJames ElderJames added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Aug 27, 2022
@ElderJames ElderJames pinned this issue Aug 27, 2022
@kooliokey
Copy link
Contributor

@ElderJames That PR was for Alert that just needs moved in the list from affix

@ElderJames
Copy link
Member Author

@kooliokey Fixed. Thank you.

@ElderJames
Copy link
Member Author

Hello @kooliokey , would you like to continue to help the improvement?

@ElderJames
Copy link
Member Author

Hello @kooliokey , we still need to improve coverage, we'd appreciate it if you were available to help! 😊

@ElderJames ElderJames linked a pull request Oct 31, 2023 that will close this issue
15 tasks
@ElderJames
Copy link
Member Author

We also can use Playwright to setup E2E tests for the complex components, do you have any experience with this? @kooliokey

@kooliokey
Copy link
Contributor

We also can use Playwright to setup E2E tests for the complex components, do you have any experience with this? @kooliokey

That sounds like a good idea. I haven't used playwright before but have wanted to try it. Would we be testing the docs site with it? You have to run an application not random code from what I remember.

@ElderJames
Copy link
Member Author

Yes, we need to add a standalone E2E project for testing, because the demos on the documentation site don't cover the full use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants