Skip to content

NO-ISSUE: Enable story tests - #388

Open
fantonangeli wants to merge 1 commit into
open-workflow-specification:mainfrom
fantonangeli:NO-ISSUE-enable-story-tests
Open

NO-ISSUE: Enable story tests#388
fantonangeli wants to merge 1 commit into
open-workflow-specification:mainfrom
fantonangeli:NO-ISSUE-enable-story-tests

Conversation

@fantonangeli

Copy link
Copy Markdown
Member

This is to re-enable story tests

Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Copilot AI lite review requested due to automatic review settings September 4, 2026 13:10
@netlify

netlify Bot commented Sep 4, 2026

Copy link
Copy Markdown

Deploy Preview for openworkflow-editor ready!

Name Link
🔨 Latest commit 7822406
🔍 Latest deploy log https://app.netlify.com/projects/openworkflow-editor/deploys/6a9ac3471820750009e3219d
😎 Deploy Preview https://deploy-preview-388--openworkflow-editor.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.

Copilot AI 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.

🟢 Approval recommended

The change cleanly re-enables previously excluded tests without altering runtime code paths or introducing configuration inconsistencies.

Pull request overview

Re-enables the diagram editor’s “story tests” by allowing *.story.test.{ts,tsx} files to run as part of the Vitest unit test project in @openworkflowspec/diagram-editor.

Changes:

  • Removes the Vitest exclude rule that previously skipped tests/**/*.story.test.{ts,tsx} from the unit project.
File summaries
File Description
packages/open-workflow-diagram-editor/vitest.config.ts Stops excluding story test files so they execute again under the unit test suite.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lornakelly lornakelly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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