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
54
[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
[2024/10/25]CREの守護者たち 〜DevOps×シフトレフト - 俺またプロダクト救っちゃいました!?〜
tosite
0
720
[2024/07/11]Guardianとして生まれ変わった俺は攻めと守りの運用で無双する 〜守りの天才が考える、攻めの運用術〜
tosite
0
870
[2024/04/23]tbls活用事例 〜 ビューポイントから データベースを整理してみた話 〜
tosite
0
410
[2023/09/15]ER図クエスト 過ぎ去りしドキュメントを求めて 〜複雑性に眠る秘宝〜
tosite
0
620
[2022/12/07]この素晴らしいアプリケーションにテストコードを
tosite
0
40
[2022/03/25]コミュニティから学ぶエンジニアリング
tosite
0
350
[2019/07/27]はじめよう、ニコカレ!
tosite
0
38
[2019/03/23]プルリクとの上手な付き合い方
tosite
0
21
[2018-12-12]ティファニーで転職を〜夏の日の2018〜
tosite
0
38
Other Decks in Programming
See All in Programming
AWSのLambdaで PHPを動かす選択肢
rinchoku
2
390
ChatGPT とつくる PHP で OS 実装
memory1994
PRO
3
190
「とりあえず動く」コードはよい、「読みやすい」コードはもっとよい / Code that 'just works' is good, but code that is 'readable' is even better.
mkmk884
6
1.4k
盆栽転じて家具となる / Bonsai and Furnitures
aereal
0
1.9k
Запуск 1С:УХ в крупном энтерпрайзе: мечта и реальность ПМа
lamodatech
0
960
ecspresso, ecschedule, lambroll を PipeCDプラグインとして動かしてみた (プロトタイプ) / Running ecspresso, ecschedule, and lambroll as PipeCD Plugins (prototype)
tkikuc
2
2k
EC2からECSへ 念願のコンテナ移行と巨大レガシーPHPアプリケーションの再構築
sumiyae
3
600
Simple組み合わせ村から大都会Railsにやってきた俺は / Coming to Rails from the Simple
moznion
3
2.6k
asdf-ecspresso作って 友達が増えた話 / Fujiwara Tech Conference 2025
koluku
0
1.4k
生成AIでGitHubソースコード取得して仕様書を作成
shukob
0
630
自動で //nolint を挿入する取り組み / Gopher's Gathering
utgwkk
1
110
Azure AI Foundryのご紹介
qt_luigi
1
210
Featured
See All Featured
The MySQL Ecosystem @ GitHub 2015
samlambert
250
12k
No one is an island. Learnings from fostering a developers community.
thoeni
19
3.1k
Site-Speed That Sticks
csswizardry
3
280
How to Think Like a Performance Engineer
csswizardry
22
1.3k
How to Create Impact in a Changing Tech Landscape [PerfNow 2023]
tammyeverts
49
2.2k
Evolution of real-time – Irina Nazarova, EuRuKo, 2024
irinanazarova
6
500
BBQ
matthewcrist
85
9.4k
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
panda_program
98
18k
Bash Introduction
62gerente
610
210k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
127
18k
Why Our Code Smells
bkeepers
PRO
335
57k
KATA
mclloyd
29
14k
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 関数群を切り出すことによって スコープが明確になった 関数がある 副次的にその関数の リファクタを推し進められた
まとめ \もうこんな時間/
レガシーな肥大化した アプリケーションを リファクタするのは 難しい
特にテストコードが ない場合はなおさら
でもアプローチを 工夫すれば必ず安全に やり遂げられる
皆さんもテストを書いて 安全にリファクタして いきましょう!
ご清聴ありがとう ございました