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
40
[2019/03/23]プルリクとの上手な付き合い方
tosite
0
22
Other Decks in Programming
See All in Programming
データベースのオペレーターであるCloudNativePGがStatefulSetを使わない理由に迫る
nnaka2992
0
130
SwiftUIで単方向アーキテクチャを導入して得られた成果
takuyaosawa
0
270
Ruby on cygwin 2025-02
fd0
0
140
データの整合性を保つ非同期処理アーキテクチャパターン / Async Architecture Patterns
mokuo
46
17k
Pythonでもちょっとリッチな見た目のアプリを設計してみる
ueponx
1
550
Writing documentation can be fun with plugin system
okuramasafumi
0
120
バックエンドのためのアプリ内課金入門 (サブスク編)
qnighy
8
1.8k
CNCF Project の作者が考えている OSS の運営
utam0k
6
710
JavaScriptツール群「UnJS」を5分で一気に駆け巡る!
k1tikurisu
9
1.8k
AIの力でお手軽Chrome拡張機能作り
taiseiue
0
170
2024年のWebフロントエンドのふりかえりと2025年
sakito
2
240
個人アプリを2年ぶりにアプデしたから褒めて / I just updated my personal app, praise me!
lovee
0
340
Featured
See All Featured
Evolution of real-time – Irina Nazarova, EuRuKo, 2024
irinanazarova
6
550
Faster Mobile Websites
deanohume
306
31k
Optimizing for Happiness
mojombo
376
70k
The Cost Of JavaScript in 2023
addyosmani
47
7.3k
The Art of Programming - Codeland 2020
erikaheidi
53
13k
Making the Leap to Tech Lead
cromwellryan
133
9.1k
Designing for humans not robots
tammielis
250
25k
Helping Users Find Their Own Way: Creating Modern Search Experiences
danielanewman
29
2.4k
Site-Speed That Sticks
csswizardry
4
380
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
33
2.1k
How GitHub (no longer) Works
holman
314
140k
Facilitating Awesome Meetings
lara
51
6.2k
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 関数群を切り出すことによって スコープが明確になった 関数がある 副次的にその関数の リファクタを推し進められた
まとめ \もうこんな時間/
レガシーな肥大化した アプリケーションを リファクタするのは 難しい
特にテストコードが ない場合はなおさら
でもアプローチを 工夫すれば必ず安全に やり遂げられる
皆さんもテストを書いて 安全にリファクタして いきましょう!
ご清聴ありがとう ございました