Skip to content

Commit

Permalink
Merge pull request storybookjs#30310 from storybookjs/jeppe/fix-flaky…
Browse files Browse the repository at this point in the history
…-test-runner-tests

Tests: Disable flaky test-runner tests
  • Loading branch information
JReinhold authored Jan 21, 2025
2 parents f3c87f1 + 8768418 commit f3276d5
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 3 deletions.
3 changes: 2 additions & 1 deletion code/core/template/stories/args.stories.ts
Original file line number Diff line number Diff line change
Expand Up @@ -92,5 +92,6 @@ export const Events = {
await new Promise((resolve) => channel.once(STORY_ARGS_UPDATED, resolve));
await within(canvasElement).findByText(/updated/);
},
tags: ['!vitest'],
// this story can't be reliably tested because the args changes results in renderPhases disrupting test runs
tags: ['!vitest', '!test'],
};
3 changes: 2 additions & 1 deletion code/core/template/stories/decorators.stories.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,5 +67,6 @@ export const Hooks = {
});
await new Promise((resolve) => channel.once(STORY_ARGS_UPDATED, resolve));
},
tags: ['!vitest'],
// this story can't be reliably tested because the args changes results in renderPhases disrupting test runs
tags: ['!vitest', '!test'],
};
3 changes: 2 additions & 1 deletion code/core/template/stories/globals.stories.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,8 @@ export const Events = {
await channel.emit('updateGlobals', { globals: { foo: 'fooValue' } });
await within(canvasElement).findByText('fooValue');
},
tags: ['!vitest'],
// this story can't be reliably tested because the globals changes results in renderPhases disrupting test runs
tags: ['!vitest', '!test'],
};

export const Overrides1 = {
Expand Down
2 changes: 2 additions & 0 deletions code/core/template/stories/rendering.stories.ts
Original file line number Diff line number Diff line change
Expand Up @@ -105,4 +105,6 @@ export const ChangeArgs = {
await within(canvasElement).findByText(/New Text/);
await expect(button).toHaveFocus();
},
// this story can't be reliably tested because the args changes results in renderPhases disrupting test runs
tags: ['!vitest', '!test'],
};
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ const meta = {
</div>`,
});
},
// this story can't be reliably tested because the (inherited) args changes results in renderPhases disrupting test runs
tags: ['!vitest', '!test'],
} satisfies Meta<typeof Reactivity>;

export default meta;
Expand Down

0 comments on commit f3276d5

Please sign in to comment.