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

(Test/Storybook)MkSignupDialog.rules.vueのストーリーがビルドされていない #11252

Open
mappi-pr opened this issue Jul 12, 2023 · 1 comment
Labels
⚠️bug? This might be a bug

Comments

@mappi-pr
Copy link
Contributor

mappi-pr commented Jul 12, 2023

💡 Summary

  • feat: Server rules #10660
    のコミット(e1f9ab7)の際に、「MkSignupDialog.rules.stories.impl.ts」が作成されているが、
    「generate.tsx」に記載されているパスに誤り?があり、上記のテストシナリオがビルドされていない。

🥰 Expected Behavior

「MkSignupDialog-rules」のストーリーがビルドされる。

🤬 Actual Behavior

Storybookを参照した際に「MkSignupDialog」フォルダが作成されていない
image

📝 Steps to Reproduce

  1. cd packages/frontend/src/components
  2. ls | grep -E 'stories' | grep -v -E 'impl' | xargs rm -r
  3. cd ../../
  4. pnpm --filter frontend build-storybook
  5. pnpm storybook dev
  6. Storybookの画面を参照して、該当のストーリーがビルドされていない(左メニューに存在しない)ことを確認する。

あるいは

  1. https://6428f7d7b962f0b79f97d6e4-fnpndzauov.chromatic.com/?path=/docs/index--docs にアクセスする。
  2. Storybookの画面を参照して、該当のストーリーがビルドされていない(左メニューに存在しない)ことを確認する。

特記事項

「generate.tsx」にglob('src/components/MkSignupDialog.rules.vue'),を追加したら、
「MkSignupDialog.rules.stories.ts」は自動生成されたのですが、
const meta = { title: 'components/MkSignupDialog/rules', component: MkSignupDialog_rules, } satisfies Meta<typeof MkSignupDialog_rules>; export default meta;
と、生成されてしまい、MkSignupDialog_rulesMkSignupServerRulesへの手修正が必要でした。
(generate.tsx の生成ロジックにも何か修正が必要??)

📌 Environment

💻 Frontend

  • Model and OS of the device(s): Windows 11
  • Browser: Chrome 114.0.5735.199
  • Server URL: none(Storybook)
  • Misskey: 13.14.0-beta.2

🛰 Backend (for server admin)

  • Installation Method or Hosting Service:
  • Misskey: 13.14.0-beta.2
  • Node: 20.4.0
  • PostgreSQL: 12.15
  • Redis: 7.0.11
  • OS and Architecture: Ubuntu 20.04.6 LTS
@mappi-pr mappi-pr added the ⚠️bug? This might be a bug label Jul 12, 2023
@zyoshoka
Copy link
Contributor

おそらく #10336 (comment) の影響で意図的に記載していなかったものと予想しますが、今普通にローカルで試した感じではモックできているように見えますね

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

No branches or pull requests

2 participants