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

Create main.spec.ts #1806

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

sharma-shray
Copy link
Contributor

@sharma-shray sharma-shray commented Jul 5, 2024

🚀 This description was created by Ellipsis for commit 228cc44

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:

  • File Added: __tests__/playwright-test/main.spec.ts
  • Tests Added:
    • Check for logo element with class .flex.cursor-pointer
    • Check for logo element with class .mb-2
    • Check for background element with class .bg-background
    • Check for main div with class .flex.size-full.flex-col.items-center.justify-center
    • Check for side menu button with class button.ring-offset-background
    • Check for 'start chatting' element with class .ring-offset-background
    • Check for 'New chat' button with class .mr-1
    • Check for input with placeholder 'Search workspaces...'
    • Check for input with placeholder 'Search presets...'
    • Check for input with placeholder 'Search prompts...'
    • Check for input with placeholder 'Search models...'
    • Check for input with placeholder 'Search files...'
    • Check for input with placeholder 'Ask anything. Type "@" for assistants, "/" for prompts, "#" for files, and "!" for tools.'
    • Check for input with placeholder 'Search collections...'
    • Check for input with placeholder 'Search assistants...'
    • Check for input with placeholder 'Search tools...'
    • Check for 'New Workspace' button
    • Delete workspace test
    • Check for button with id #new-workspace-button
    • Check for profile button with test id #userButtonOpenMenu
    • Check for 'Cancel' and 'Save' buttons
    • Check for help information with text 'Show Help' and 'Show Workspaces'

Generated with ❤️ by ellipsis.dev

Copy link

@ellipsis-dev ellipsis-dev bot left a 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 in 1 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 }) => {
Copy link

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.

__tests__/playwright-test/main.spec.ts Outdated Show resolved Hide resolved
__tests__/playwright-test/main.spec.ts Outdated Show resolved Hide resolved
sharma-shray and others added 2 commits July 6, 2024 12:10
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>
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.

1 participant