Migrated Install SDK functional tests to Vitest+Playwright+MSW

[carterworks]

Changed Packages

• core
• reactor-extension

Description

Migrates the Install SDK functional tests to the new Vitest+Playwright+MSW harness. C2579 (multi-instance) and C1338399 (NPM entry point) are preserved as documented test.skip with explanations.

Related Issue

Part of the functional test → integration test migration. See packages/browser/test/FUNCTIONAL_MIGRATION_PLAN.md.

Motivation and Context

The existing TestCafe functional test suite is being migrated to Vitest+Playwright+MSW to enable faster, more reliable CI testing without a running server. This PR is part of a stacked series — each PR migrates one test file.

Functional tests replaced:

• packages/browser/test/functional/specs/Install SDK/C2560.js
• packages/browser/test/functional/specs/Install SDK/C2579.js
• packages/browser/test/functional/specs/Install SDK/C1338399.js

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Improvement (non-breaking change which does not add functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have added a Changeset file with a consumer-facing description of my changes.
• I have signed the Adobe Open Source CLA or I'm an Adobe employee.
• I have made any necessary test changes and all tests pass.
• I have run the Sandbox successfully.

Stack

1. Added MSW handlers, mock fixtures, and Vitest+Playwright test harness for functional test migration #1532
2. Migrated Install SDK functional tests to Vitest+Playwright+MSW #1533 👈 current

[changeset-bot]

🦋 Changeset detected

Latest commit: 945ac88

The changes in this PR will be included in the next version bump.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

Greptile Summary

This PR adds a new Vitest+Playwright+MSW integration spec (installSdk.spec.js) that replaces the three TestCafe files in Install SDK/. C2560 is ported as an active test; C2579 and C1338399 are preserved as documented test.skip stubs with clear rationale for why the current harness cannot support them.

• C2560 verifies typeof window.alloy === "function" after the alloy auto-fixture loads the SDK — a stronger assertion than the original !!window.alloy boolean check.
• C2579 (multi-instance isolation) is skipped because initializeStandalone() reads __alloyNS only once and a second load would conflict with the existing instance; the skip comment recommends unit tests as coverage.
• C1338399 (NPM entry point) is skipped because the harness always loads the standalone build, not the alloyCreateInstance NPM artifact.

Confidence Score: 5/5

Safe to merge — the change adds a single new test file with no modifications to production code or shared test infrastructure.

Only one file is added. The active test (C2560) correctly uses the auto-fixture harness and makes a stronger assertion than the original TestCafe test. The two skipped tests include thorough explanations and are not regressions. No shared helpers or production code are touched.

No files require special attention.

Important Files Changed

Filename	Overview
packages/browser/test/integration/specs/Install SDK/installSdk.spec.js	New integration spec migrating three TestCafe tests (C2560 active, C2579 and C1338399 skipped with documented rationale) to the Vitest+Playwright+MSW harness — no logic defects found.

Sequence Diagram

sequenceDiagram
    participant Vitest
    participant extend.js (auto fixtures)
    participant MSW Worker
    participant setupBaseCode
    participant setupAlloy
    participant Test (C2560)

    Vitest->>extend.js (auto fixtures): run worker fixture (auto)
    extend.js (auto fixtures)->>MSW Worker: worker.start() (once)
    MSW Worker-->>extend.js (auto fixtures): started

    Vitest->>extend.js (auto fixtures): run alloy fixture (auto)
    extend.js (auto fixtures)->>extend.js (auto fixtures): clear kndctr_/AMCV_ cookies
    extend.js (auto fixtures)->>setupBaseCode: setupBaseCode()
    setupBaseCode-->>extend.js (auto fixtures): window.__alloyNS set
    extend.js (auto fixtures)->>setupAlloy: setupAlloy()
    setupAlloy-->>extend.js (auto fixtures): window.alloy (function)

    extend.js (auto fixtures)->>Test (C2560): use(alloy)
    Test (C2560)->>Test (C2560): expect(typeof window.alloy).toBe("function")
    Test (C2560)->>Test (C2560): expect(alloy).toBeDefined()

    Vitest->>extend.js (auto fixtures): teardown alloy fixture
    extend.js (auto fixtures)->>extend.js (auto fixtures): cleanAlloy()
    Vitest->>extend.js (auto fixtures): teardown worker fixture
    extend.js (auto fixtures)->>MSW Worker: worker.resetHandlers()

Loading

_{Reviews (2): Last reviewed commit: "test(integration): migrate install sdk f..." | Re-trigger Greptile}

[greptile-apps]

carterworks has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.