Skip to content

test: use the run token for metamorph in integration tests - #1206

Open
mvolfik wants to merge 2 commits into
masterfrom
fix/metamorph-test
Open

mvolfik wants to merge 2 commits into
masterfrom
fix/metamorph-test

Conversation

@mvolfik

@mvolfik mvolfik commented Nov 10, 2026

Copy link
Copy Markdown
Member

fix for breaking change from apify/apify-core#32513

@mvolfik mvolfik self-assigned this Nov 10, 2026
@mvolfik mvolfik added the adhoc Ad-hoc unplanned task added during the sprint. label Nov 10, 2026
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Nov 10, 2026
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Nov 10, 2026
@github-actions

github-actions Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

✅ The link checker did not find any broken links.

See more at https://github.com/apify/apify-client-js/actions/runs/42833494809#summary-129889504430

@mvolfik mvolfik changed the title fix: use run token in metamorph test, and metamorphing from outside is not allowed anymore fix: use run token in metamorph test, as metamorphing from outside is not allowed anymore Nov 10, 2026
@mvolfik mvolfik added the t-core-services Issues with this label are in the ownership of the core services team. label Nov 10, 2026

@apify-factory apify-factory left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, nothing blocking.

🏭 Factory review — ℹ️ not auto-approved

This change was reviewed but cannot be auto-approved: a gate failed, so the review cannot end in an approval.

check result
Static gates ❌ 15/16 — no-risky-content
Review findings ✅ none
Reviewer escalation ✅ none
Juror 1 — claude-sonnet-5-5, adversarial ⏭️ skipped — a gate failed
Why it was not auto-approved

Failed gates:

  • no-risky-content: test/integration/_fixtures.ts: environment variable access (env-access); test/integration/run.test.ts: environment variable access (env-access)

The comments above are the review itself; this section only explains the approval decision. Push a change and request @apify-factory again to re-run it, or ask a human to review.

Reviewed a2d32a881 · workflow run

@mvolfik
mvolfik requested a review from vdusek November 10, 2026 14:45
@vdusek vdusek changed the title fix: use run token in metamorph test, as metamorphing from outside is not allowed anymore test: use the run token for metamorph in integration tests Nov 11, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-core-services Issues with this label are in the ownership of the core services team. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants