Skip to content

Commit 92eb366

Browse files
authored
Merge pull request #1129 from objectstack-ai/copilot/fix-ci-build-test-errors-yet-again
2 parents 6968ac6 + bd1aefe commit 92eb366

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
## [Unreleased]
99

10+
### Fixed
11+
12+
- **ObjectView ChatterPanel test assertion mismatch** (`apps/console`): Fixed the failing CI test in `ObjectView.test.tsx` where the RecordChatterPanel drawer test used `getByLabelText('Show discussion')` but the actual aria-label rendered by the component is `'Show Discussion (0)'` (from the i18n default translation `detail.showDiscussion` with count parameter). Updated the test assertion to match the correct aria-label.
13+
1014
### Changed
1115

1216
- **Upgrade @objectstack packages from v3.2.9 to v3.3.0**: Upgraded all `@objectstack/*` dependencies across the entire monorepo (root, `apps/console`, `examples/crm`, `examples/todo`, `examples/msw-todo`, `examples/kitchen-sink`, `packages/core`, `packages/types`, `packages/react`, `packages/data-objectstack`, `packages/plugin-map`, `packages/plugin-gantt`, `packages/plugin-timeline`) from `^3.2.9` to `^3.3.0`. All 42 build tasks pass and all 7,224 tests pass with no breaking changes.

apps/console/src/__tests__/ObjectView.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -918,7 +918,7 @@ describe('ObjectView Component', () => {
918918

919919
// The drawer should render a "Show Discussion" button (ChatterPanel is defaultCollapsed)
920920
await vi.waitFor(() => {
921-
const showBtn = screen.getByLabelText('Show discussion');
921+
const showBtn = screen.getByLabelText('Show Discussion (0)');
922922
expect(showBtn).toBeInTheDocument();
923923
// Verify items count shows (0) since items are initially empty
924924
expect(showBtn).toHaveTextContent('Show Discussion (0)');

0 commit comments

Comments
 (0)