脱レガシープロダクトに向けて開発環境にCIを整備し単体テストを布教した話/developers boost2021

Slide 1

Slide 1 text

Slide 2

Slide 2 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential ● DX Engineerをやっています。 ● 2019/04に新卒入社 ● プロダクト開発部署に配属後１年間、開発環境の改善に取り組む ● 2020/12より半年間の育休を取得 ● 復帰後、開発部門全体のDX改善チーム立ち上げに参画 ○ Developer Experience ○ Digital Transformation 自己紹介 2

Slide 3

Slide 3 text

Slide 4

Slide 4 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 大規模ソフトウェア開発って？ 4 チームA チームB チームC オフショアA 大規模ソースコード ● 20年間保守 ● 数百万行規模 ● 計40-50人（＋オフショア） ● 全チームが協調して共通のソースを編集 ● 5-10 (人) x 6 (チーム) ● 一部オフショアあり

Slide 5

Slide 5 text

Slide 6

Slide 6 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 大規模プロダクト開発のつらいところ 6 ● ソースコードが大規模かつ依存関係が複雑 ○ 「機能追加したけどどこから手をつければいいんだっけ？」 ○ 「このAPI変更したら他チームも使用しているので修正依頼してね」 ● ビルドやサーバ起動等に時間がかかる ○ 「ビルド走らせている間にコーヒーでも淹れるか・・・」 ○ 「サーバ再起動している間に(ry」 ● テスト自動化が難しく手動テストに頼らざるを得ない ○ 「リファクタリングしたい？テスト工数が確保できないから無理」

Slide 7

Slide 7 text

Slide 8

Slide 8 text

Slide 9

Slide 9 text

Slide 10

Slide 10 text

Slide 11

Slide 11 text

Slide 12

Slide 12 text

Slide 13

Slide 13 text

Slide 14

Slide 14 text

Slide 15

Slide 15 text

Slide 16

Slide 16 text

Slide 17

Slide 17 text

Slide 18

Slide 18 text

Slide 19

Slide 19 text

Slide 20

Slide 20 text

Slide 21

Slide 21 text

Slide 22

Slide 22 text

Slide 23

Slide 23 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential モダンな開発環境とは 1.モダンで快適な開発環境の整備 23 ● 開発者全員が共通の設定のIDEを利用 ● IDEのファイルを編集のたびにフォーマッタやLinterを実行 ● コードレビュー時に各種チェック（CI）を実行 ○ ビルド ○ フォーマッタ ○ Linter ○ 単体テスト

Slide 24

Slide 24 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 快適な開発環境とは（Developer Experience） 1.モダンで快適な開発環境の整備 24 ● IDEのセットアップを自動化されている ● 自動チェックがすぐに終わる ○ プレマージCIの実行時間は平均５分以内を目標に ○ 時間のかかるチェックはStaging branchで定期実行 ● 自動チェックの誤検知が少ない ○ 静的解析等の既存のエラーは抑制 ○ 誤検知率をモニタリングし対策する ● 自動チェックが失敗した時に原因がすぐにわかる ○ ビルドログから原因を抽出しコメントで指摘

Slide 25

Slide 25 text

Slide 26

Slide 26 text

Slide 27

Slide 27 text

Slide 28

Slide 28 text

Slide 29

Slide 29 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential JUnit dashboard なぜテストが増えないのか？ 29 ● JUnitの実行結果をデータベースに入力してMetabaseで可視化 ● 「最近追加されたテスト」 ○ 最近テストが追加されたパッケージを表示 ○ どの部署がテストを書いているか概ね把握できるように ● 結果 ○ テストを書いていたのは一部の部署のみ

Slide 30

Slide 30 text

Slide 31

Slide 31 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 単体テストに対するチームの状態なぜテストが増えないのか？ 31 チーム１チーム２チーム3 レガシーコードに単体テストが書ける新規コードに単体テストが書ける単体テストが書けない、メリットを感じていない「単体テスト環境整備されたらしいけど、どう書いたらいいか分からない。」「習得する時間もないから今回は手動テストでいいかな」

Slide 32

Slide 32 text

Slide 33

Slide 33 text

Slide 34

Slide 34 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential JUnit dojoマインド編 2. 新規コードに単体テストを書く 34 ● t_wadaさんの「TDD Boot Camp 2020 Online #1 基調講演/ライブコーディング」の鑑賞会 ● 目的 ○ 単体テストの書き方やメリットを知る ○ 単体テストを書いていこうという意識づけ https://www.youtube.com/watch?v=Q-FJ3XmFlT8

Slide 35

Slide 35 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential JUnit dojo基礎編 2. 新規コードに単体テストを書く 35 ● 単体テストの基本的な書き方をハンズオン形式で習得 ○ 単体テストとは ○ テストのライフサイクル ○ アサーションの書き方 ○ テストダブルについて ● 目的 ○ 単体テストのコーディングスタイルを揃える ○ 新規コードに対して単体テストが書けるように

Slide 36

Slide 36 text

Slide 37

Slide 37 text

Slide 38

Slide 38 text

Slide 39

Slide 39 text

Slide 40

Slide 40 text

Slide 41

Slide 41 text

Slide 42

Slide 42 text

Slide 43

Slide 43 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential JUnit dojo実践編 3. レガシーコードに単体テストを書く 43 ● 内容 1. 実プロダクトでテストを書くライブコーディング 2. 実プロダクトを模した研修用レガシーコードで単体テストを書くテクニックを実践 i. リファクタリングツールを使い依存関係を整理 ii. 仕様化テストを書いてカバレッジを担保 iii. 単体テストで保護された部分をリファクタリング https://github.com/autotaker/LegacyEx

Slide 44

Slide 44 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 実践編後 3. レガシーコードに単体テストを書く 44 「レガシーコードにも単体テストが書ける（らしい）！」「今度のチケットで実践してみよう」「困ったら〇〇さんに聞いてみよう」チーム１チーム２チーム3 レガシーコードに単体テストが書ける新規コードに単体テストが書ける単体テストが書けない、メリットを感じていない

Slide 45

Slide 45 text

Slide 46

Slide 46 text

Slide 47

Slide 47 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential JUnit dojoの後に起こった開発組織の変化脱レガシーに向けて 47 ● dojo参加者が講師役となってJUnit dojo再放送が行われた ● 単体テストの勉強会が各チームで開催されるようになった ● 単体テストを書きやすくするために共通モックが整備された ● 他プロダクトまで噂が広がり、単体テストの環境が各プロダクトに整備された ● 実務で単体テストを書いていく定期的な活動（JUnit部）が発足

Slide 48

Slide 48 text

Slide 49

Slide 49 text

Slide 50

Slide 50 text

Slide 51

Slide 51 text

Slide 52

Slide 52 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 研修１年後脱レガシーに向けて 52 「マージリクエスト作りました。単体テストのカバレッジ通っています」「単体テスト書いたし、このロジックリファクタリングしよう」チーム１チーム２チーム3 レガシーコードに単体テストが書ける新規コードに単体テストが書ける単体テストが書けない、メリットを感じていない

Slide 53

Slide 53 text

Slide 54

Slide 54 text

© 2021 Works Human Intelligence Co., Ltd. Strictly Confidential 脱レガシーのために単体テストを書く文化を醸成３つのポイント 1.モダンで快適な開発環境を整備する ○ テストをかくハードルを下げる 2.開発者のテストスキルを高める ○ 足りていないのは工数ではなく技術 3.単体テストを実際に書いてみる ○ テストのある世界を体感することで意識が変化する ○ 各チームのエバンジェリストを通じて徐々に広がるまとめ 54

Slide 55

Slide 55 text