Skip to content

Commit

Permalink
fix(frontend): fix Chromatic test fails (misskey-dev#13826)
Browse files Browse the repository at this point in the history
* fix: attempt to fix Chromatic test fails

* chore: add comment
  • Loading branch information
zyoshoka authored May 18, 2024
1 parent 9b0fc31 commit def7b8c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
5 changes: 4 additions & 1 deletion packages/frontend/src/components/MkModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -276,8 +276,11 @@ const align = () => {
const onOpened = () => {
emit('opened');
// NOTE: Chromatic テストの際に undefined になる場合がある
if (content.value == null) return;
// モーダルコンテンツにマウスボタンが押され、コンテンツ外でマウスボタンが離されたときにモーダルバックグラウンドクリックと判定させないためにマウスイベントを監視しフラグ管理する
const el = content.value!.children[0];
const el = content.value.children[0];
el.addEventListener('mousedown', ev => {
contentClicking = true;
window.addEventListener('mouseup', ev => {
Expand Down
8 changes: 7 additions & 1 deletion packages/frontend/src/components/global/MkAd.stories.impl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ import { i18n } from '@/i18n.js';

let lock: Promise<undefined> | undefined;

function sleep(ms: number) {
return new Promise(resolve => setTimeout(resolve, ms));
}

const common = {
render(args) {
return {
Expand Down Expand Up @@ -43,6 +47,8 @@ const common = {
lock = new Promise(r => resolve = r);

try {
// NOTE: sleep しないと何故か落ちる
await sleep(100);
const canvas = within(canvasElement);
const a = canvas.getByRole<HTMLAnchorElement>('link');
// await expect(a.href).toMatch(/^https?:\/\/.*#test$/);
Expand All @@ -53,7 +59,7 @@ const common = {
const i = buttons[0];
await expect(i).toBeInTheDocument();
await userEvent.click(i);
// await expect(canvasElement).toHaveTextContent(i18n.ts._ad.back);
await expect(canvasElement).toHaveTextContent(i18n.ts._ad.back);
await expect(a).not.toBeInTheDocument();
await expect(i).not.toBeInTheDocument();
buttons = canvas.getAllByRole<HTMLButtonElement>('button');
Expand Down

0 comments on commit def7b8c

Please sign in to comment.