Upgrade to Pro — share decks privately, control downloads, hide ads and more …

0 -> 1でフロントエンドのテストを 書く文化を作っている話

gtnao
October 22, 2023

0 -> 1でフロントエンドのテストを 書く文化を作っている話

gtnao

October 22, 2023
Tweet

More Decks by gtnao

Other Decks in Programming

Transcript

  1. ©primeNumber Inc. 2 中根 直孝 WHO AM I? 株式会社primeNumber イネーブリングチーム(来月から)

    経歴 2017:新卒でWebエンジニアのキャリアをスタート 2018/11:primeNumberにジョイン(trocco®のローン チとほぼ同時期) 趣味/好きなもの 音楽/ギター、サウナ、飲酒、 辛いもの(最近はCoCo壱20辛) SNSなど X: @gtnao, GitHub: @gtnao0219 1

  2. 会社概要 会社名 代表 創業 Office COMPANY 株式会社primeNumber 代表取締役CEO 田邊 雄樹

    2015年11月 東京都品川区上大崎3丁目1番1号 JR東急目黒ビル5F 3
  3. ©primeNumber Inc. 8 trocco®とは フルマネージド ETL/ELT 日本特有のサービスを含めた、約100種のコネクタに対応 データ分析基盤のための技術スタックを提供するサービスです。 GUI ワークフロー

    複雑なデータ処理フローを GUI 上で設定・運用 データマネジメント機能 データリネージ / Git 連携・コード管理 / スキーマ追従 / データカタログ etc.
  4. ©primeNumber Inc. 9 • trocco®はローンチから5年のSaaS • ありがたいことのお客様は増加の一途 • 当たり前のように負債も蓄積 •

    当時、品質改善PJT(仮)が発足 • 改善したい点を上げ出したら、朝まで飲めるが… 背景 何からやっていけばいいんや…
  5. ©primeNumber Inc. 13 • PJTメンバーだけでテストを拡充していってもいいが… • 至極当たり前のこのサイクルを皆がフロントエンドでもできるように • 「0 ->

    1でフロントエンドのテストを書く文化を作った話」 • 最近流行ってるから「FrontendTestEnabling」PJTに改名 ◦ 乗るしかない 背景 既存のrspecがコケる 新機能実装 テスト/実装を修正
  6. ©primeNumber Inc. 15 • ところで、フロントエンドのテストって何を書けばいいの? ◦ 条件に応じてコンポーネントが表示/非表示されるか? ◦ フォーム周りの挙動? ◦

    APIリクエスト? ◦ VisualRegression? ◦ etc…(巷には綺麗なサンプルが溢れているが…) • 原点に立ち戻る 方針 Q: 手っ取り早く皆がテストに触れるには?
  7. ©primeNumber Inc. 17 • 何か新しいフィールドを追加すると基本落ちる • 落ちたままだとよからぬバグに繋がる可能性が高い ◦ バリデーション表示のテストなどは見送った (後述)。最悪サーバーで弾かれるため

    • このテストを各ページに60点レベルでひたすら追加 していく ◦ 「0 -> 1でReact書くのは厳しいけど、ちょっ と修正するだけなら」といった人も多いので テストファイルがあることが重要 • まずはPJTメンバーで書いたテストのPRを、PJT外 のメンバーにレビューアサインし、「こんな感じ ね」と知ってもらう 方針 フォームに値を入力 保存ボタンを押下 想定通りの HTTPリクエストボディ が送信されることを確認 ✅
  8. ©primeNumber Inc. 23 • 外部パッケージにESModuleが使われている場合の対策をあれこれしたら激遅になって しまった ◦ ESModuleFirstなvitestへの乗り換え ◦ 幸い、既存テストが少なかったので楽だった

    • MonacoEditorなど特定のコンポーネントを使ってるページで激遅に ◦ 最終的にテストでは生のtextareaなどにモックするように ◦ 特定に時間がかかった(操作関係なく存在するだけで影響したため) • それでもまだ厳しい ◦ PJTメンバーのCI職人にチューニングしてもらう ◦ (あんまりやりたくなかったが)余分なケースを減らす ▪ 前述のバリデーションなど 問題② CI速度低下問題
  9. ©primeNumber Inc. 24 • RTLは要素の取得にはa11yを考慮した方法を使いましょうという思想だが… ◦ 現実はそんなに甘くない(海外のデータ系のSaaSを見てもdata-testid振ってい た) ◦ react-hook-formを使っており、各種フィールドにはname属性が付与されている

    ことが多かったのでそれを利用 ◦ 使ってない箇所は今回テストのためだけに属性を追加 • z-index対策からか、モーダルのコンポーネントをReact.createPortalで階層外にレン ダーしていた ◦ モーダル内にある要素の操作方法が他と異なる • etc…(上げるとキリがないのでまたの機会に) 問題③ その他の諸問題
  10. ©primeNumber Inc. 25 • 1週間のPRのうち、フロントエン ドのファイルに変更があるPRに テストが書かれているか • trocco®(転送元GitHub)で取得 ◦

    一部温もりのある手作業で フィルタリング • 週によって母数がまちまちに… ◦ 指標としては微妙だった ふりかえり
  11. ©primeNumber Inc. 27 • jsパッケージのDependabotのマージが気軽にできるようになった • テストを書いてたら既にあったバグを発見 • テスト書いてくれる人が増えた ◦

    何かしらテストを追加してくれた方が0ベースから10人超え ◦ 業務委託など社外の方々も書いてくれている ◦ 「来月ここら辺いじる予定だから、先にテストのPR出しておくわ〜」 とかも • 来月からフロントエンドに限らないイネーブリングチームが正式に誕生 ふりかえり 良かったこと
  12. ©primeNumber Inc. 28 • 成果を定量化するのが難しい • まだまだ浸透しているとは言い難い ◦ 実装自体が結構バラバラで抽象化しづらい ▪

    このページはこの書き方をしないと〜、とかがある ◦ 落ちたときのエラーメッセージが非自明で分かりにくい ▪ デバッグTipsドキュメント整備中 • フォームの入力→APIリクエスト以外のテストケースの拡充もしたい • そもそもプロダクトコードをリファクタリングしていきたい ◦ テストがほとんどなかった時代に比べてハードルが下がった ふりかえり 反省点とこれから
  13. ©primeNumber Inc. 29 • trocco®を一緒に創ってくれる方を探しています ◦ エンジニアのドメイン知識が活かせるSaaS ▪ 社内利用のドッグフーディングも活発 ◦

    海外展開も進行中 ◦ Hotなデータエンジニアリングの知識も身につくかも • 色々なポジションで募集中です ◦ ソフトウェアエンジニア ◦ SRE ◦ QAエンジニア ◦ ソフトウェアエンジニア(イネーブリングチーム) ◦ 詳しくはこちらから ▪ https://herp.careers/v1/primenumber/requisition-groups/b258370f- bf7b-4358-9631-d53f51decef6 We Are Hiring!