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

[WIP] エンジンのモック作成+それを使ったコンポーネントテスト #2152

Draft
wants to merge 87 commits into
base: main
Choose a base branch
from

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Jul 2, 2024

内容

の解決を目指したプルリクエストです。
ついでにストーリーブック上でコンポーネントテストする方法を色々試そうとしてます。

TalkEditorの表示と、モックエンジンを使ったピッチ推論までできたのですが、なぜかscssが読み込まれずにスプリッターの色指定がうまくいってないです。
Viteとかの設定な気がしないでもないので、詳しい方いらっしゃったらヘルプいただけると助かります 🙇

追記:わかりました!!!たぶん色の初期化をしてないからでした!!

関連 Issue

fix #2144

スクリーンショット・動画など

こんな感じで境界線がない。多分正確には透明になってる。
image

その他

@Hiroshiba
Copy link
Member Author

テーマ周りで気になった挙動まとめ

  • ipcのset/getが同じ関数になってる
  • availableThemesだけセットしたいのにcurrentThemeもセットするmutationしかない
  • currentThemeのセッターに副作用がある

@Hiroshiba Hiroshiba force-pushed the エンジンのmockを作る branch from ddc7f1b to b1b10e4 Compare July 12, 2024 14:51
@Hiroshiba
Copy link
Member Author

とりあえずスナップショットテストができた!
それとは関係ないところでeslintエラーが出たのでissue立ててみました

@Hiroshiba Hiroshiba force-pushed the エンジンのmockを作る branch from e9a6b07 to 1240838 Compare August 10, 2024 15:24
@Hiroshiba
Copy link
Member Author

📝
モック部分は完成。
あとは切り分けてプルリクエストにしていく。

とりあえず第1弾として、engineMockディレクトリと、そのモックを使ったスナップショットテストだけプルリクエストを送信する。
これのレビューは不可能なので、えいやで実装しちゃう。

その後モックを使ったいろんなコード変更をプルリクエストしていく。
こっちは結構分かりやすいはず!

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Nov 23, 2024

📝

  • 本家kuromoji.jsはpathが無いので動かない
  • クローンの@sglkc/kuromojiはフルパス URL に対応してない
  • フォークするか、パッチするか。

@Hiroshiba
Copy link
Member Author

とりあえずできるかわかりませんが、VOICEVOX orgにforkして、それを取り込む方針で行こうと思います!
うまくいかなかったらフォークしたものを消そうと思います。

@Hiroshiba
Copy link
Member Author

ブラウザでkuromojiが正しく動くように、VOICEVOX/kuromoji.jsを作りました!!

これでプルリクを作っていこうと思います。

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.

Vuexのレイヤーでエンジンのmockを作る
2 participants