-
-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Create main.spec.ts #1806
base: main
Are you sure you want to change the base?
Create main.spec.ts #1806
Conversation
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.
❌ Changes requested. Reviewed everything up to 228cc44 in 53 seconds
More details
- Looked at
222
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. __tests__/playwright-test/main.spec.ts:3
- Draft comment:
The test name 'Check on logo element exists' seems to be grammatically incorrect or unclear. Consider renaming it for clarity.
test('Check if logo element exists', async ({ page }) => {
- Reason this comment was not posted:
Marked as duplicate.
Workflow ID: wflow_4WcThNmRFZIfGqvF
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
}); | ||
|
||
|
||
test('Check if logo element exists', async ({ page }) => { |
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.
The test description 'Check if logo element exists' is potentially confusing as it is similar to the test on line 3. Consider specifying what each test is checking for more clearly to avoid confusion.
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Summary:
Added
__tests__/playwright-test/main.spec.ts
with multiple Playwright tests to verify the existence of various UI elements on the/chat
page.Key points:
__tests__/playwright-test/main.spec.ts
.flex.cursor-pointer
.mb-2
.bg-background
.flex.size-full.flex-col.items-center.justify-center
button.ring-offset-background
.ring-offset-background
.mr-1
#new-workspace-button
#userButtonOpenMenu
Generated with ❤️ by ellipsis.dev