Skip to content

test(preset): raise beforeEach hook timeout for interactive create-workspace#900

Merged
cogwirrel merged 1 commit into
mainfrom
fix/create-workspace-hook-timeout
Jul 7, 2026
Merged

test(preset): raise beforeEach hook timeout for interactive create-workspace#900
cogwirrel merged 1 commit into
mainfrom
fix/create-workspace-hook-timeout

Conversation

@nx-plugin-for-aws

Copy link
Copy Markdown
Collaborator

Description

Follow-up to #898. The interactive create-workspace smoke test's beforeEach calls createTestWorkspace, which retries workspace creation up to 3× on flaky installs. Each attempt can take ~40s, so the total can exceed vitest's default 120s hookTimeout, failing the setup before the test body runs.

Raise the hook timeout to 10 minutes (matching the it timeouts used elsewhere in the e2e suite).

Verification

Confirmed the interactive preset prompt works end-to-end via a PTY: enquirer.prompt renders, accepts the typed engagementId, the generator exits 0, and the value is persisted to aws-nx-plugin.config.mts as tags: ['MyEngagementId'].

Type of change

  • Bug fix (test infra)

…rkspace

createTestWorkspace retries workspace creation up to 3x, which can
exceed vitest's default 120s hookTimeout.
@codecov-commenter

codecov-commenter commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.36%. Comparing base (5480cca) to head (d7e55f9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #900   +/-   ##
=======================================
  Coverage   87.36%   87.36%           
=======================================
  Files         155      155           
  Lines        5388     5388           
  Branches     1263     1263           
=======================================
  Hits         4707     4707           
  Misses        359      359           
  Partials      322      322           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cogwirrel
cogwirrel merged commit 3c7fe20 into main Jul 7, 2026
53 checks passed
@cogwirrel
cogwirrel deleted the fix/create-workspace-hook-timeout branch July 7, 2026 06:16
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.

3 participants