Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Storyのテストが一部通らなくなってるのを修正 #2363

Conversation

Hiroshiba
Copy link
Member

内容

の解決プルリクエストです。

QuestionDialogが、Storybook上でonHideがなぜか呼ばれなくなってるっぽい。
Storybook内でDialog.createしたりしたのですが、onHideが呼ばれない続きだったので諦めてmodelValueがfalseになることをテストするようにしました。

関連 Issue

fix #2362

その他

なぜCI上では呼ばれてるのか不明・・・・・・・。

@Hiroshiba Hiroshiba requested a review from a team as a code owner November 17, 2024 16:11
@Hiroshiba Hiroshiba requested review from sevenc-nanashi and removed request for a team November 17, 2024 16:11
@Hiroshiba
Copy link
Member Author

Hiroshiba commented Nov 17, 2024

onUpdate:modelValueが存在しない」みたいな感じでなぜか怒られる・・・。

たぶん型として

const emit = defineEmits<{
  "update:modelValue": [value: boolean];
}>();

みたいに定義すれば問題ないけど、quasarの

defineEmits({
  ...useDialogPluginComponent.emitsObject,
});

は実態として定義していて、どう混ぜればいいのかわからない・・・・・・。

@sevenc-nanashi さん、ちょっと型周り助言いただきたく・・・ 🙇

@sevenc-nanashi
Copy link
Member

ローカルでvue-tscとvueのバージョンを上げたら直りました。

@Hiroshiba
Copy link
Member Author

ほんとだ!! ありがとうございます!!

@voicevox-preview-pages
Copy link

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:ef4594b

@Hiroshiba
Copy link
Member Author

Storybookプレビューでもしっかりエラー出ずに通ってそう!
便利~~
https://voicevox.github.io/preview-pages/preview/pr-2363/storybook/?path=/story/components-dialog-textdialog-questiondialog--click-backdrop-with-cancel

@Hiroshiba
Copy link
Member Author

たぶん大丈夫なのと、mainブランチにいつまでも×が付いてるのが気になるので、マージします!

@Hiroshiba Hiroshiba merged commit a5874ec into VOICEVOX:main Nov 18, 2024
10 checks passed
@Hiroshiba Hiroshiba deleted the Storyのテストが一部通らなくなってるのを修正 branch November 18, 2024 12:11
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.

chromaticのCIが落ちている
2 participants