Merge pull request #1297 from objectstack-ai/copilot/fix-ci-build-test-errors-please-work

Author: hotlong

CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- **CI: `ReportViewer` conditional-formatting test.** Made the
+  background-color assertion in
+  `packages/plugin-report/src/__tests__/ReportViewer.test.tsx`
+  tolerant of both `rgb(...)` and hex (`#rrggbb`) representations.
+  Newer jsdom versions preserve the original hex value on
+  `HTMLElement.style.backgroundColor` instead of normalizing to
+  `rgb(...)`, which broke the previous strict equality check
+  (CI run #25539753515).
+
 ### Added
 
 - **Page-mode record forms (`editMode: 'page'`).** New per-object

packages/plugin-report/src/__tests__/ReportViewer.test.tsx

@@ -408,7 +408,9 @@ describe('ReportViewer', () => {
     const cells = container.querySelectorAll('td');
     const wonCell = Array.from(cells).find((td) => td.textContent === 'Won');
     expect(wonCell).toBeDefined();
-    expect(wonCell?.style.backgroundColor).toBe('rgb(187, 247, 208)');
+    // jsdom may normalize colors to `rgb(...)` or preserve the original hex
+    // depending on the version, so accept either representation.
+    expect(wonCell?.style.backgroundColor).toMatch(/^(rgb\(187,\s*247,\s*208\)|#bbf7d0)$/i);
 
     // The cell with 'Lost' should NOT have background styling
     const lostCell = Array.from(cells).find((td) => td.textContent === 'Lost');