Skip to content

test: automate various quote operations#36351

Merged
kodiakhq[bot] merged 8 commits intodevelopfrom
quote-messages
Jul 11, 2025
Merged

test: automate various quote operations#36351
kodiakhq[bot] merged 8 commits intodevelopfrom
quote-messages

Conversation

@Harmeet221
Copy link
Contributor

@Harmeet221 Harmeet221 commented Jul 3, 2025

JIRA TASK: https://rocketchat.atlassian.net/browse/QA-84

Description:
Covered the following scenarios for quote message:

  • Quote a single message in channel
  • Edit a quoted message
  • Delete a quoted message
  • Quote message with emoji
  • Quote message with markdown formatting
  • Quote message with code blocks
  • Cancel quote preview
  • Quote message in Direct Message
  • Quote message in Discussion
  • Quote message with thread in DM

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@Harmeet221 Harmeet221 requested a review from a team as a code owner July 3, 2025 12:22
@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Jul 3, 2025

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is targeting the wrong base branch. It should target 7.9.0, but it targets 7.8.0

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Jul 3, 2025

⚠️ No Changeset found

Latest commit: f82700c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Harmeet221 Harmeet221 marked this pull request as draft July 3, 2025 12:22
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://RocketChat.github.io/Rocket.Chat/pr-preview/pr-36351/

Built to branch gh-pages at 2025-07-11 16:30 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link

codecov bot commented Jul 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.32%. Comparing base (8c951f8) to head (f82700c).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #36351      +/-   ##
===========================================
- Coverage    65.32%   65.32%   -0.01%     
===========================================
  Files         3156     3156              
  Lines       103991   103992       +1     
  Branches     19901    19905       +4     
===========================================
- Hits         67933    67928       -5     
- Misses       33377    33381       +4     
- Partials      2681     2683       +2     
Flag Coverage Δ
e2e 58.39% <100.00%> (+0.04%) ⬆️
unit 70.43% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Harmeet221 Harmeet221 marked this pull request as ready for review July 8, 2025 09:38
@Harmeet221 Harmeet221 requested a review from jessicaschelly July 9, 2025 14:40
@jessicaschelly jessicaschelly added this to the 7.9.0 milestone Jul 11, 2025
@jessicaschelly jessicaschelly added the stat: QA assured Means it has been tested and approved by a company insider label Jul 11, 2025
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge stat: QA assured Means it has been tested and approved by a company insider and removed stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge labels Jul 11, 2025
@dionisio-bot dionisio-bot bot added stat: QA assured Means it has been tested and approved by a company insider and removed stat: QA assured Means it has been tested and approved by a company insider labels Jul 11, 2025
@Harmeet221 Harmeet221 closed this Jul 11, 2025
@Harmeet221 Harmeet221 reopened this Jul 11, 2025
@Harmeet221 Harmeet221 added the stat: QA assured Means it has been tested and approved by a company insider label Jul 11, 2025
@Harmeet221 Harmeet221 added this to the 7.9.0 milestone Jul 11, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jul 11, 2025
@kodiakhq kodiakhq bot merged commit b0be3e5 into develop Jul 11, 2025
92 of 106 checks passed
@kodiakhq kodiakhq bot deleted the quote-messages branch July 11, 2025 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants