test: Replace magic tag names in e2e tests #37932

dougfabris · 2025-12-22T20:03:44Z

Proposed changes (including videos or screenshots)

We've been creating some tests and using tags with magical specific names in multiples test files. This can cause some undesired operations in tags such as edits or removals that is not desirable in others, causing intermittences

Issue(s)

Steps to test or reproduce

Further comments

Summary by CodeRabbit

Tests
- Refactored omnichannel tag management test suites for improved code organization and maintainability.
- Enhanced tag creation utilities with automatic fallback value generation for increased test data consistency and reliability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

changeset-bot · 2025-12-22T20:03:48Z

⚠️ No Changeset found

Latest commit: 59835e3

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

coderabbitai · 2025-12-22T20:03:50Z

Walkthrough

This pull request refactors omnichannel tag tests by replacing array-based tag storage with individual tag variables across multiple test files, and updates the tag creation utility to use faker for generating random fallback values for tag names and descriptions.

Changes

Cohort / File(s)	Change Summary
Tag test refactoring `apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts`, `apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts`, `apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts`	Replaced single aggregated tags array with individual tag variables; updated all creation, cleanup, and usage points to reference tag objects and their `data.name` properties instead of hard-coded names or array iteration.
Tag utility update `apps/meteor/tests/e2e/utils/omnichannel/tags.ts`	Added faker import; removed default parameter values and integrated faker-based fallback generation for tag name (8-character random string) and description (16-character random string).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

RocketChat/Rocket.Chat#36869: Directly modifies the same omnichannel tag test files and related tag handling utilities with similar structural changes.

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

ggazzo
aleksandernsilva

Poem

🐰 Tags once bundled in arrays so tight,
Now stand alone, individual and bright,
Faker weaves random strings with care,
Tests run cleaner, organized, fair,
Omnichannel hops with refactored delight! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: replacing hardcoded tag names in e2e tests with dynamic variables to prevent test intermittence.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch test/magic-names-intermitence

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 3dc9e2b and 59835e3.

📒 Files selected for processing (4)

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts

🧰 Additional context used

📓 Path-based instructions (4)

**/*.{ts,tsx,js}

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

**/*.{ts,tsx,js}: Write concise, technical TypeScript/JavaScript with accurate typing in Playwright tests
Avoid code comments in the implementation

Files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

**/*.spec.ts

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

**/*.spec.ts: Use descriptive test names that clearly communicate expected behavior in Playwright tests
Use .spec.ts extension for test files (e.g., login.spec.ts)

Files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

apps/meteor/tests/e2e/**/*.spec.ts

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

apps/meteor/tests/e2e/**/*.spec.ts: All test files must be created in apps/meteor/tests/e2e/ directory
Avoid using page.locator() in Playwright tests - always prefer semantic locators such as page.getByRole(), page.getByLabel(), page.getByText(), or page.getByTitle()
Use test.beforeAll() and test.afterAll() for setup/teardown in Playwright tests
Use test.step() for complex test scenarios to improve organization in Playwright tests
Group related tests in the same file
Utilize Playwright fixtures (test, page, expect) for consistency in test files
Prefer web-first assertions (toBeVisible, toHaveText, etc.) in Playwright tests
Use expect matchers for assertions (toEqual, toContain, toBeTruthy, toHaveLength, etc.) instead of assert statements in Playwright tests
Use page.waitFor() with specific conditions instead of hardcoded timeouts in Playwright tests
Implement proper wait strategies for dynamic content in Playwright tests
Maintain test isolation between test cases in Playwright tests
Ensure clean state for each test execution in Playwright tests
Ensure tests run reliably in parallel without shared state conflicts

Files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

apps/meteor/tests/e2e/**/*.{ts,spec.ts}

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

apps/meteor/tests/e2e/**/*.{ts,spec.ts}: Store commonly used locators in variables/constants for reuse
Follow Page Object Model pattern consistently in Playwright tests

Files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

🧠 Learnings (13)

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Ensure tests run reliably in parallel without shared state conflicts

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Utilize Playwright fixtures (`test`, `page`, `expect`) for consistency in test files

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Group related tests in the same file

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.{ts,spec.ts} : Follow Page Object Model pattern consistently in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-12-10T21:00:54.909Z

Learnt from: KevLehman
Repo: RocketChat/Rocket.Chat PR: 37091
File: ee/packages/abac/jest.config.ts:4-7
Timestamp: 2025-12-10T21:00:54.909Z
Learning: Rocket.Chat monorepo: Jest testMatch pattern '<rootDir>/src/**/*.spec.(ts|js|mjs)' is valid in this repo and used across multiple packages (e.g., packages/tools, ee/packages/omnichannel-services). Do not flag it as invalid in future reviews.

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/page-objects/**/*.ts : Utilize existing page objects pattern from `apps/meteor/tests/e2e/page-objects/`

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Maintain test isolation between test cases in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Use `test.step()` for complex test scenarios to improve organization in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.{ts,spec.ts} : Store commonly used locators in variables/constants for reuse

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Ensure clean state for each test execution in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/utils/omnichannel/tags.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Prefer web-first assertions (`toBeVisible`, `toHaveText`, etc.) in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Use `test.beforeAll()` and `test.afterAll()` for setup/teardown in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts
apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

📚 Learning: 2025-11-24T17:08:17.065Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-11-24T17:08:17.065Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Use `expect` matchers for assertions (`toEqual`, `toContain`, `toBeTruthy`, `toHaveLength`, etc.) instead of `assert` statements in Playwright tests

Applied to files:

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts

🧬 Code graph analysis (3)

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts (1)

apps/meteor/tests/e2e/utils/omnichannel/tags.ts (1)

createTag (15-36)

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts (1)

apps/meteor/tests/e2e/utils/omnichannel/tags.ts (1)

createTag (15-36)

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts (1)

apps/meteor/tests/e2e/utils/omnichannel/tags.ts (1)

createTag (15-36)

🔇 Additional comments (18)

apps/meteor/tests/e2e/utils/omnichannel/tags.ts (2)

1-1: LGTM: Faker import for random test data.

The addition of faker improves test isolation by generating unique tag names when not explicitly provided, preventing interference between parallel tests.

19-20: LGTM: Dynamic fallback values prevent tag name collisions.

Using faker.string.alpha for default names (8 chars) and descriptions (16 chars) ensures each tag gets a unique identifier, addressing the PR's goal of eliminating "magic" tag names that caused test intermittence.

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-chats.spec.ts (4)

26-27: LGTM: Individual tag variables improve test clarity.

Replacing the array with distinct variables (tagA, tagB) makes the test setup more explicit and aligns with the PR's goal of preventing tag name collisions.

66-69: LGTM: Tag creation updated for unique identifiers.

Individual tag creation ensures each tag receives a unique faker-generated name, eliminating the previous hardcoded 'tagA'/'tagB' magic strings that caused test interference.

101-106: LGTM: Dynamic tag names eliminate hardcoded values.

Using tagA.data.name and tagB.data.name instead of literal strings 'tagA' and 'tagB' is the core fix for test intermittence. Each test run now gets unique tag identifiers, preventing cross-test conflicts.

127-127: LGTM: Cleanup aligned with new tag structure.

The cleanup correctly deletes individual tags using the updated variable structure.

apps/meteor/tests/e2e/omnichannel/omnichannel-contact-center-filters.spec.ts (6)

29-30: LGTM: Consistent with tag isolation pattern.

Replacing the array with individual variables aligns with changes across other test files and improves test clarity.

72-75: LGTM: Individual tag creation ensures uniqueness.

Each tag now receives a unique faker-generated identifier, preventing the "magic name" collisions that caused intermittent test failures.

123-128: LGTM: Room updates use dynamic tag identifiers.

Replacing hardcoded tag strings with tag.data.name prevents tests from interfering with each other when running in parallel.

269-281: LGTM: Filter interactions updated for dynamic tags.

All tag selection and removal operations now use tagA.data.name and tagB.data.name, ensuring tests interact with their own unique tags rather than shared "magic" names.

317-317: LGTM: Applied filters use dynamic tag names.

The filter application correctly references the unique tag identifier, maintaining consistency with the PR's isolation improvements.

142-142: LGTM: Cleanup updated for individual tags.

Tag deletion correctly targets the new individual variables.

apps/meteor/tests/e2e/omnichannel/omnichannel-assign-room-tags.spec.ts (6)

24-27: LGTM: Four distinct tag variables for comprehensive testing.

The separation into tagA, tagB, globalTag, and sharedTag clearly communicates each tag's department association and prevents naming conflicts across tests.

46-51: LGTM: Tag creation with department associations.

Each tag receives a unique faker-generated name while maintaining its department configuration. This eliminates hardcoded names while preserving the test's department-specific logic.

68-68: LGTM: Cleanup handles all four tags.

The cleanup correctly deletes all tags, maintaining proper test isolation.

90-102: LGTM: Tag visibility checks use dynamic names.

All visibility assertions now reference tag.data.name, ensuring tests interact with their own unique tags rather than shared identifiers that could cause interference.

106-114: LGTM: Tag selection and verification updated.

Tag operations consistently use the dynamic tag names from the created tag objects, completing the migration away from magic strings.

127-131: LGTM: Department-specific visibility checks use dynamic tags.

The assertions correctly verify department-based tag visibility using the unique identifiers.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

codecov · 2025-12-22T20:19:58Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.77%. Comparing base (3dc9e2b) to head (59835e3).
⚠️ Report is 3 commits behind head on release-8.0.0.

Additional details and impacted files

@@                Coverage Diff                @@
##           release-8.0.0   #37932      +/-   ##
=================================================
+ Coverage          71.73%   71.77%   +0.03%     
=================================================
  Files               1453     1453              
  Lines              76689    76689              
  Branches           11133    11153      +20     
=================================================
+ Hits               55013    55040      +27     
+ Misses             21249    21220      -29     
- Partials             427      429       +2

Flag	Coverage Δ
unit	`71.77% <ø> (+0.03%)`	⬆️

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.

github-actions · 2025-12-22T20:31:12Z

📦 Docker Image Size Report

📈 Changes

Service	Current	Baseline	Change
sum of all images	1.1GiB	1.1GiB	+53MiB
rocketchat	355MiB	345MiB	+11MiB
ddp-streamer-service	168MiB	126MiB	+43MiB
omnichannel-transcript-service	132MiB	132MiB	-246B
queue-worker-service	132MiB	132MiB	+61B
account-service	113MiB	113MiB	+343B
authorization-service	111MiB	111MiB	-911B
presence-service	111MiB	111MiB	+5B

📊 Historical Trend

---
config:
  theme: "dark"
  xyChart:
    width: 900
    height: 400
---
xychart
  title "Image Size Evolution by Service (Last 30 Days + This PR)"
  x-axis ["11/15 22:28", "11/16 01:28", "11/17 23:50", "11/18 22:53", "11/19 23:02", "11/21 16:49", "11/24 17:34", "11/27 22:32", "11/28 19:05", "12/01 23:01", "12/02 21:57", "12/03 21:00", "12/04 18:17", "12/05 21:56", "12/08 20:15", "12/09 22:17", "12/10 23:26", "12/11 21:56", "12/12 22:45", "12/13 01:34", "12/15 22:31", "12/16 22:18", "12/17 21:04", "12/18 23:12", "12/19 23:27", "12/20 21:03", "12/22 18:54", "12/22 20:31 (PR)"]
  y-axis "Size (GB)" 0 --> 0.5
  line "account-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "authorization-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "ddp-streamer-service" [0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.16]
  line "omnichannel-transcript-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13]
  line "presence-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "queue-worker-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13, 0.13]
  line "rocketchat" [0.36, 0.36, 0.35, 0.35, 0.35, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.34, 0.35]

Statistics (last 27 days):

📊 Average: 1.5GiB
⬇️ Minimum: 1.2GiB
⬆️ Maximum: 1.6GiB
🎯 Current PR: 1.1GiB

ℹ️ About this report

This report compares Docker image sizes from this build against the develop baseline.

Tag: pr-37932
Baseline: develop
Timestamp: 2025-12-22 20:31:11 UTC
Historical data points: 27

Updated: Mon, 22 Dec 2025 20:31:11 GMT

test: replace magic tag names in e2e tests

59835e3

dougfabris added this to the 8.0.0 milestone Dec 22, 2025

dougfabris marked this pull request as ready for review December 22, 2025 20:57

dougfabris requested a review from a team as a code owner December 22, 2025 20:57

ggazzo approved these changes Dec 23, 2025

View reviewed changes

ggazzo added the stat: QA assured Means it has been tested and approved by a company insider label Dec 23, 2025

ggazzo merged commit 8796753 into release-8.0.0 Dec 23, 2025
44 of 45 checks passed

ggazzo deleted the test/magic-names-intermitence branch December 23, 2025 00:43

gaolin1 pushed a commit to gaolin1/medsense.webchat that referenced this pull request Jan 6, 2026

test: Replace magic tag names in e2e tests (RocketChat#37932)

44351f5

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test: Replace magic tag names in e2e tests #37932

test: Replace magic tag names in e2e tests #37932

Uh oh!

dougfabris commented Dec 22, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

changeset-bot bot commented Dec 22, 2025

Uh oh!

coderabbitai bot commented Dec 22, 2025 •

edited

Loading

Uh oh!

codecov bot commented Dec 22, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Dec 22, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

test: Replace magic tag names in e2e tests #37932

test: Replace magic tag names in e2e tests #37932

Uh oh!

Conversation

dougfabris commented Dec 22, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Summary by CodeRabbit

Uh oh!

changeset-bot bot commented Dec 22, 2025

⚠️ No Changeset found

Uh oh!

coderabbitai bot commented Dec 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

codecov bot commented Dec 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-actions bot commented Dec 22, 2025

📦 Docker Image Size Report

📈 Changes

📊 Historical Trend

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

dougfabris commented Dec 22, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 22, 2025 •

edited

Loading

codecov bot commented Dec 22, 2025 •

edited

Loading