Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
[2021/12/16]テストコードのないレガシーアプリケーションとの向き合い方
Search
tosite
December 16, 2021
Programming
0
65
[2021/12/16]テストコードのないレガシーアプリケーションとの向き合い方
ソフトウェアテストシンポジウム 2021 九州
https://www.jasst.jp/symposium/jasst21kyushu.html
tosite
December 16, 2021
Tweet
Share
More Decks by tosite
See All by tosite
[2025-02-07]生成AIで変える問い合わせの未来 〜チームグローバル化の香りを添えて〜
tosite
1
430
[2024/10/25]CREの守護者たち 〜DevOps×シフトレフト - 俺またプロダクト救っちゃいました!?〜
tosite
0
810
[2024/07/11]Guardianとして生まれ変わった俺は攻めと守りの運用で無双する 〜守りの天才が考える、攻めの運用術〜
tosite
0
900
[2024/04/23]tbls活用事例 〜 ビューポイントから データベースを整理してみた話 〜
tosite
0
430
[2023/09/15]ER図クエスト 過ぎ去りしドキュメントを求めて 〜複雑性に眠る秘宝〜
tosite
0
640
[2022/12/07]この素晴らしいアプリケーションにテストコードを
tosite
0
42
[2022/03/25]コミュニティから学ぶエンジニアリング
tosite
0
350
[2019/07/27]はじめよう、ニコカレ!
tosite
0
39
[2019/03/23]プルリクとの上手な付き合い方
tosite
0
21
Other Decks in Programming
See All in Programming
Conform を推す - Advocating for Conform
mizoguchicoji
3
690
Java Webフレームワークの現状 / java web framework at burikaigi
kishida
9
2.2k
Rails アプリ地図考 Flush Cut
makicamel
1
120
Amazon Bedrock Multi Agentsを試してきた
tm2
1
280
DROBEの生成AI活用事例 with AWS
ippey
0
130
パスキーのすべて ── 導入・UX設計・実装の紹介 / 20250213 パスキー開発者の集い
kuralab
3
760
Djangoアプリケーション 運用のリアル 〜問題発生から可視化、最適化への道〜 #pyconshizu
kashewnuts
1
240
Honoをフロントエンドで使う 3つのやり方
yusukebe
7
3.2k
チームリードになって変わったこと
isaka1022
0
200
SwiftUIで単方向アーキテクチャを導入して得られた成果
takuyaosawa
0
270
Unity Android XR入門
sakutama_11
0
150
Compose でデザインと実装の差異を減らすための取り組み
oidy
1
300
Featured
See All Featured
Java REST API Framework Comparison - PWX 2021
mraible
28
8.4k
Fashionably flexible responsive web design (full day workshop)
malarkey
406
66k
How GitHub (no longer) Works
holman
314
140k
Rails Girls Zürich Keynote
gr2m
94
13k
Building a Scalable Design System with Sketch
lauravandoore
461
33k
Building Applications with DynamoDB
mza
93
6.2k
Building a Modern Day E-commerce SEO Strategy
aleyda
38
7.1k
Automating Front-end Workflow
addyosmani
1368
200k
Navigating Team Friction
lara
183
15k
The Art of Delivering Value - GDevCon NA Keynote
reverentgeek
10
1.3k
Measuring & Analyzing Core Web Vitals
bluesmoon
6
240
ピンチをチャンスに:未来をつくるプロダクトロードマップ #pmconf2020
aki_iinuma
114
50k
Transcript
テストコードのない レガシーアプリケーション との向き合い方 in JaSST'21 Kyushu 2021-12-18 Sat
はじめまして \の方もそうでない方も/
はじめまして 3
はじめまして 4 GMOペパボ株式会社 ホスティング事業部 バックエンドエンジニア 趣味: インターネットJK・ キャンプ @tosite /
まおちゃ / 手島 尚人
はい \会社紹介/
はじめまして 6
はじめまして 7 ドメイン・レンタルサーバー ご用命の際はぜひ弊社サービスを よろしくお願いいたします!
はじめまして 8
はじめまして 9 コミュニティでステッカーやファングッズ・ ノベルティなどを作成したい場合は ご利用ください!
ここから本題 \いい仕事した/
突然ですが皆さん テストは 書いていますか? \かの有名な方もこうおっしゃっていますよね/
突然ですが皆さん テストは 書いていますか? 著者: @mty_mno 「テスターちゃん番外編『ヤツが来る!!』の巻&メイキングオブジョジョ風の絵 」
と言っても テストが不十分な アプリケーションが あるのも現実… \理想と現実ってやつ/
前提 14 今年のテーマは「歴史から学ぶ」
前提 15 歴史の話をしよう
前提 16
前提 17 ホスティング事業部のサービスは 20年もの長い歴史があります 今も一部リポジトリでは 次のようなレガシーな構成を 目にすることがあります \今回担当したのはムームードメインのPHPで書かれたアプリケーションコードでした/
前提 18 レガシーなアプリケーションではまれに 次のような構成を目にすることがあります \ちなみに案件はPHPでした/
前提 19 コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行)
前提 20 コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行) 1ファイルで全ての処理を実行している
コントローラー・ビュー・ ビジネスロジックが密結合している ためテストが難しい 当然、既存のテストコードは ほとんどない状態
前提 21 1ファイル(数千行) コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行) ・1ファイルで全ての処理を実行
・コントローラー・ビュー・ ビジネスロジックが密結合 しているためテストが難しい ・当然、既存のテストコードは ほとんどない状態
前提 22 リファクタリングしたいが テストコードも仕様書もなく 「動いているコードが正しい」 という状態
前提 23 要件は 振る舞いを変えつつ 変更後の挙動を担保した状態で かつバグが混入しないこと というもの
前提 24 つまり リアーキテクティング した上で振る舞いを変える 必要があるということ
前提 25 リアーキテクティングするための テストがない コードが密結合しているため ユニットテストが書けない \鶏卵問題ですね/
前提 26 その状況でいきなり 振る舞いを変えるのは 自殺行為
前提 27 リファクタリングするためのテストがなく、 テストを書こうにも密結合しているため そのままではテストできない とは言えいきなりリファクタリングするのも エンバグする危険性が極めて高い \鶏卵問題ですね/
皆さんなら どうしますか? \一緒に考えてみましょう/
私は次のような アプローチを取りました \一例ですが/
フェーズ1. ロジックの分離 \ローマの道も一歩から/
フェーズ1. ロジックの分離 31
フェーズ1. ロジックの分離 32 これを
フェーズ1. ロジックの分離 33 index.php functions.php
フェーズ1. ロジックの分離 34 こう
フェーズ1. ロジックの分離 35 まずはファイルから ビジネスロジックを 切り出すだけの作業 を行った 😅
フェーズ2. 既存の関数に対して テストを記述 \リファクタリングらしくなってきました/
フェーズ2. 既存の関数に対してテストを記述 37
フェーズ2. 既存の関数に対してテストを記述 38 既存の仕様を満たす テストケースを記載
フェーズ2. 既存の関数に対してテストを記述 39 ファイルを分割したので テストが書ける!! 😃
フェーズ2. 既存の関数に対してテストを記述 40 とは言え全ての関数にテストを 追加する工数はなかったので リアーキテクティング対象 のみに留めた
フェーズ3. リアーキテクティング \テストがあるので安全だね/
フェーズ3. リアーキテクティング 42
フェーズ3. リアーキテクティング 43 気になる部分を リファクタ
フェーズ4. 落ちるテストを記述 \赤より始めよ/
フェーズ3. 落ちるテストを記述 45 修正する関数のテストケース 既存の仕様を満たすテスト郡 リファクタリング後に 仕様を満たすようになるテスト郡
リファクタリング前にレッドにな ることを確認しておく リファクタリング後は グリーンになることを確認 することで安全に仕様を 変えることができる 脆弱性が混入されていないことを 確認するテスト郡
フェーズ4. 落ちるテストを記述 46 振る舞い変更前 => レッド 振る舞い変更後 => グリーン
フェーズ4. 落ちるテストを記述 47 「既存仕様が壊れていないこと」は 現在の仕様を満たすテストコードから確認できる 「新規仕様が実装されたこと」は 落ちるテストが通ることで確認できる
フェーズ5. 振る舞いを変更 \さあ、コーディングの時間だ!/
フェーズ5. 振る舞いを変更 49 ここまで来たら安心安全に 振る舞いを変更できる ようになっている
フェーズ5. 振る舞いを変更 50
フェーズ5. 振る舞いを変更 51 フェーズ3で記述した 落ちるテストが 通るように修正 したら実装完了!
フェーズ5. 振る舞いを変更 52 全てグリーンで通過!
メリット \こんないいことがあったよ/
メリット1.
メリット 55 テストがない世界からの脱却! 今後機能追加や改修をやるとなった ときの開発基盤が整った
メリット2.
メリット 57 コードベースだけでなく テストケースベースで レビューできる ので安心感があった
メリット 58 特に一部関数には 正規表現が使用されており 修正前と後で目grepで挙動を 担保するのはつらぽよだった 修正後の挙動を
テストケースという仕様書 にできた
メリット3.
メリット 60 関数群を切り出すことによって スコープが明確になった 関数がある 副次的にその関数の リファクタを推し進められた
まとめ \もうこんな時間/
レガシーな肥大化した アプリケーションを リファクタするのは 難しい
特にテストコードが ない場合はなおさら
でもアプローチを 工夫すれば必ず安全に やり遂げられる
皆さんもテストを書いて 安全にリファクタして いきましょう!
ご清聴ありがとう ございました