-
Notifications
You must be signed in to change notification settings - Fork 0
feat: connect @testplane/testing-library @KuznetsovRoman #45
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
Conversation
efb7f56 to
6e6d47c
Compare
6e6d47c to
35c135b
Compare
src/utils/index.ts
Outdated
| // Find by role | ||
| const fileSearchResultsItems = await fileSearchResults.findAllByRole("option"); | ||
| expect(fileSearchResultsItems.length).toBeGreaterThan(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
лучше везде перед expect добавлять await, чтобы не думать, что вот в этой проверке он не нужен, а в expect(fileSearchResults).toBeDisabled(); обязателен. Лучше сразу пример генерить с await, чтобы никто даже не думал, что можно по-другому
| // Find by placeholder | ||
| const fileSearchInput = await browser.findByPlaceholderText("Search docs"); | ||
| await fileSearchInput.setValue("config"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
перед вводом текста я бы еще проверил на видимость элемента - await fileSearchInput.isDisplayedInViewport();
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"isDisplayedInViewport" просто вернет true / false
Добавил "waitForDisplayed"
17e1742 to
27d37d7
Compare
No description provided.