Skip to content

Remove catalog tests out of monorepo#3951

Open
richardhjtan wants to merge 2 commits intomainfrom
CS-10119-remove-catalog-tests
Open

Remove catalog tests out of monorepo#3951
richardhjtan wants to merge 2 commits intomainfrom
CS-10119-remove-catalog-tests

Conversation

@richardhjtan
Copy link
Contributor

To be merged after this PR cardstack/boxel-catalog#19 goes in

What is changing

  • remove catalog realm involved related tests
  • skip catalog realm when doing host test
  • the SKIP_CATALOG is still needed because boxel-catalog can remove the variable and run the catalog realm test in CI
  • rename start-all-except-experiment script -> start-all-except-optional

@richardhjtan richardhjtan marked this pull request as ready for review February 5, 2026 07:24
@richardhjtan richardhjtan requested review from a team February 5, 2026 07:24
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Preview deployments

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Host Test Results

 1 files  ±    0  1 suites  ±0   58s ⏱️ - 1h 43m 29s
21 tests  - 1 923  4 ✅  - 1 910  0 💤  - 17  15 ❌ +15  2 🔥  - 11 
21 runs   - 1 938  2 ✅  - 1 914  0 💤  - 17  17 ❌ + 4  2 🔥  - 11 

For more details on these failures and errors, see this check.

Results for commit 4a3ab98. ± Comparison against base commit cf3abb2.

This pull request removes 1938 and adds 15 tests. Note that renamed tests count towards both.
Chrome ‑ Acceptance | AI Assistant tests: "Add Same Skills" copies skill configuration to new session
Chrome ‑ Acceptance | AI Assistant tests: ai assistant panel width persists to localStorage
Chrome ‑ Acceptance | AI Assistant tests: auto-attached cards behaviour
Chrome ‑ Acceptance | AI Assistant tests: auto-attached file is not displayed in interact mode
Chrome ‑ Acceptance | AI Assistant tests: auto-attached spec card can be removed
Chrome ‑ Acceptance | AI Assistant tests: auto-attaches spec card when spec panel is open and spec is selected
Chrome ‑ Acceptance | AI Assistant tests: can display and remove auto attached file
Chrome ‑ Acceptance | AI Assistant tests: can open attach file modal
Chrome ‑ Acceptance | AI Assistant tests: can open attached card dropdown menu
Chrome ‑ Acceptance | AI Assistant tests: can send a newly created auto-attached card
…
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | AI Assistant tests: defaults to the system card default regardless of submode 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | Commands tests: a host command added from a skill can be executed when clicked on 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | Spec preview: view when there is a single spec instance 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | basic tests: visiting realm root 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode tests > multiple realms: default realm is the personal realm 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode | create-file tests > when user has permissions to both test realms: new file button has options to create card def, field def, card instance, and text files 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode | editor tests: card instance JSON displayed in monaco editor 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode | file def navigation: clicking embedded file in preview opens markdown file in code mode 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode | file-tree tests: can navigate file tree, file view mode is persisted in query parameter 
Chrome ‑ Global error: Uncaught Error: Failed to fetch realm info for http://localhost:4201/catalog/: 404 at http://localhost:7357/assets/chunk.f930c4ac6701265f6192.js, line 156670  While executing test: Acceptance | code submode | head format preview: head format preview updates when editing card json 
…

♻️ This comment has been updated with latest results.

@tintinthong
Copy link
Contributor

tintinthong commented Feb 5, 2026

@richardhjtan All test are failing. Is it still trying to fetch /catalog from test??

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.

2 participants