Skip to content

Comments

chore(repo): re-enable e2e tests disabled by api-extractor issue#34519

Merged
FrozenPandaz merged 2 commits intomasterfrom
revert-rush
Feb 20, 2026
Merged

chore(repo): re-enable e2e tests disabled by api-extractor issue#34519
FrozenPandaz merged 2 commits intomasterfrom
revert-rush

Conversation

@FrozenPandaz
Copy link
Collaborator

Current Behavior

14 e2e test suites were disabled (xdescribe) due to an ESM import issue in @microsoft/api-extractor@7.57.0 (see qmhc/unplugin-dts#461).

Expected Behavior

With the upstream issue resolved, all 14 e2e test suites are re-enabled (describe) and should pass normally.

Related Issue(s)

Reverts #34516

@FrozenPandaz FrozenPandaz requested review from a team and Coly010 as code owners February 19, 2026 19:22
@nx-cloud
Copy link
Contributor

nx-cloud bot commented Feb 19, 2026

View your CI Pipeline Execution ↗ for commit cc4a91c

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 41m 39s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 3m 55s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 8s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-20 16:22:24 UTC

@netlify
Copy link

netlify bot commented Feb 19, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit cc4a91c
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69987f6a20365d00089127fc
😎 Deploy Preview https://deploy-preview-34519--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 19, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit cc4a91c
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69987f6ac6837f0009978274
😎 Deploy Preview https://deploy-preview-34519--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@nx-cloud nx-cloud bot left a comment

Choose a reason for hiding this comment

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

Important

At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.

Nx Cloud has identified a possible root cause for your failed CI:

Our PR re-enabled 14 e2e test suites that were previously disabled due to an upstream @microsoft/api-extractor@7.57.0 ESM import issue. Unfortunately, the upstream issue has not been resolved, and all re-enabled tests are failing with the same module resolution error. We need to either wait for the upstream fix, update/pin the dependency version, or keep these tests disabled until the environment issue is resolved.

No code changes were suggested for this issue.

Trigger a rerun:

Rerun CI

Nx Cloud View detailed reasoning on Nx Cloud ↗


🎓 Learn more about Self-Healing CI on nx.dev

@FrozenPandaz FrozenPandaz enabled auto-merge (squash) February 20, 2026 15:36
@FrozenPandaz FrozenPandaz merged commit 391d23c into master Feb 20, 2026
24 checks passed
@FrozenPandaz FrozenPandaz deleted the revert-rush branch February 20, 2026 17:17
FrozenPandaz added a commit that referenced this pull request Feb 20, 2026
)

## Current Behavior

14 e2e test suites were disabled (`xdescribe`) due to an ESM import
issue in `@microsoft/api-extractor@7.57.0` (see
qmhc/unplugin-dts#461).

## Expected Behavior

With the upstream issue resolved, all 14 e2e test suites are re-enabled
(`describe`) and should pass normally.

## Related Issue(s)

Reverts #34516

(cherry picked from commit 391d23c)
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