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

add(test): add tests for chats several stuff #147

Merged
merged 1 commit into from
Sep 6, 2024
Merged

add(test): add tests for chats several stuff #147

merged 1 commit into from
Sep 6, 2024

Conversation

luisecm
Copy link
Collaborator

@luisecm luisecm commented Sep 6, 2024

What this PR does 📖

  • Add tests for pin indicator empty
  • Add tests for pin messages container elements correct (profile picture, sender, timestamp, online indicator, message content)
  • Add tests for adding pin indicator on local/remote messages
  • Add tests for removing pin indicator from context menu on local/remote pinned messages
  • Add tests for removing pin indicator from Unpin button on local/remote pinned messages
  • Add tests for going to pin messages from pinned messages container (go to button)
  • Add tests for scroll to bottom button

Which issue(s) this PR fixes 🔨

  • Resolve #

Special notes for reviewers 🗒️

Additional comments 🎤

Copy link

github-actions bot commented Sep 6, 2024

Automated Test Results

passed  122 passed
skipped  1 skipped

Details

stats  123 tests across 17 suites
duration  18 minutes, 34 seconds
commit  cd2f513

Skipped tests

Automated Tests on Chrome Desktop › 20-settings-developer.spec.ts › Settings Developer Tests › T4 - Clicking Clear State should clear users state

Copy link

github-actions bot commented Sep 6, 2024

Playwright test execution is complete! You can find the test results report here

@luisecm luisecm merged commit 3dec9b0 into dev Sep 6, 2024
5 checks passed
@luisecm luisecm deleted the luis/pinned branch September 6, 2024 00:28
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