Skip to content

fix: run the new chain count test as the operator user - #79

Merged
Mateusz7410 merged 1 commit into
mainfrom
fix/timeline-test-run-as
Nov 7, 2026
Merged

Mateusz7410 merged 1 commit into
mainfrom
fix/timeline-test-run-as

Conversation

@Mateusz7410

Copy link
Copy Markdown
Collaborator

Summary

  • shouldCountAPageChainByTheRowsReadForThePage builds its AsyncJobTimelineChains snapshot inside System.runAs(operator()), like the other timeline tests.

Why

The v3.2.0 package version build failed on this test from #78. The build runs Apex tests as its own user, and the WITH USER_MODE chain query threw System.QueryException: Variable does not exist: tmpVar4 there. It never failed in a scratch org, from source or from the globalized copy. With the runAs, a package build from this branch succeeded (99% coverage).

Test-only change, no production code touched.

Test plan

  • bash dev/verify/verify.sh all passed (Jest 213/213, mock 66/66, full 441/441 at 97%, ns green)
  • Package version build from this branch succeeded

Files

  • force-app/main/default/classes/timeline/AsyncJobTimelineControllerTest.cls

The package version build runs tests as its own user, and the WITH USER_MODE chain query threw 'Variable does not exist: tmpVar4' there. Running the snapshot as the operator, like the other timeline tests, fixes the build.
@vercel

vercel Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
async-lib Ready Ready Preview Nov 7, 2026 3:37pm UTC

Request Review

@Mateusz7410
Mateusz7410 merged commit 97975d2 into main Nov 7, 2026
1 of 4 checks passed
@github-actions

github-actions Bot commented Nov 7, 2026

Copy link
Copy Markdown

🚀 Salesforce CI

⏭️ Deployment skipped

An earlier step failed (scratch org creation or Dev Hub authorization), so nothing was deployed. Check the workflow logs.

📦 Workflow run, logs and artifacts


🤖 Automated comment by Salesforce CI

This branch was successfully deployed

1 active deployment
Preview — de32ea66 Deployed Nov 7, 2026 by vercel[bot]
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.

1 participant