Submit Search
Upload
2021-12-16 テストコードのないレガシーアプリケーションとの向き合い方
•
0 likes
•
513 views
N
naoto teshima
Follow
JaSST'21Kyushu LT
Read less
Read more
Career
Report
Share
Report
Share
1 of 66
Download now
Download to read offline
Recommended
「のどが渇いた」というユーザーに何を出す? ユーザーの「欲しい」に惑わされない、本当のインサイトを見つけるUXデザイン・UXリサーチ
「のどが渇いた」というユーザーに何を出す? ユーザーの「欲しい」に惑わされない、本当のインサイトを見つけるUXデザイン・UXリサーチ
Yoshiki Hayama
「UXデザインとは」からはじめる「本流」のUXデザインはじめの一歩 | UXデザイン基礎セミナー 第1回
「UXデザインとは」からはじめる「本流」のUXデザインはじめの一歩 | UXデザイン基礎セミナー 第1回
Yoshiki Hayama
フロー効率性とリソース効率性について #xpjug
フロー効率性とリソース効率性について #xpjug
Itsuki Kuroda
組織にテストを書く文化を根付かせる戦略と戦術
組織にテストを書く文化を根付かせる戦略と戦術
Takuto Wada
インフラエンジニアの綺麗で優しい手順書の書き方
インフラエンジニアの綺麗で優しい手順書の書き方
Shohei Koyama
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
Takao Oyobe
新たなgitのブランチモデル「Git Feature Flow」!Git Flow,Git Hub Flow,Git Lab Flowを超えれるか?
新たなgitのブランチモデル「Git Feature Flow」!Git Flow,Git Hub Flow,Git Lab Flowを超えれるか?
naoki koyama
Discordから バーチャルオフィス「Teamflow」 に乗り換えてみた 雑談を生む工夫
Discordから バーチャルオフィス「Teamflow」 に乗り換えてみた 雑談を生む工夫
Koichiro Matsuoka
Recommended
「のどが渇いた」というユーザーに何を出す? ユーザーの「欲しい」に惑わされない、本当のインサイトを見つけるUXデザイン・UXリサーチ
「のどが渇いた」というユーザーに何を出す? ユーザーの「欲しい」に惑わされない、本当のインサイトを見つけるUXデザイン・UXリサーチ
Yoshiki Hayama
「UXデザインとは」からはじめる「本流」のUXデザインはじめの一歩 | UXデザイン基礎セミナー 第1回
「UXデザインとは」からはじめる「本流」のUXデザインはじめの一歩 | UXデザイン基礎セミナー 第1回
Yoshiki Hayama
フロー効率性とリソース効率性について #xpjug
フロー効率性とリソース効率性について #xpjug
Itsuki Kuroda
組織にテストを書く文化を根付かせる戦略と戦術
組織にテストを書く文化を根付かせる戦略と戦術
Takuto Wada
インフラエンジニアの綺麗で優しい手順書の書き方
インフラエンジニアの綺麗で優しい手順書の書き方
Shohei Koyama
5分でわかった気になるインセプションデッキ
5分でわかった気になるインセプションデッキ
Takao Oyobe
新たなgitのブランチモデル「Git Feature Flow」!Git Flow,Git Hub Flow,Git Lab Flowを超えれるか?
新たなgitのブランチモデル「Git Feature Flow」!Git Flow,Git Hub Flow,Git Lab Flowを超えれるか?
naoki koyama
Discordから バーチャルオフィス「Teamflow」 に乗り換えてみた 雑談を生む工夫
Discordから バーチャルオフィス「Teamflow」 に乗り換えてみた 雑談を生む工夫
Koichiro Matsuoka
TDD のこころ
TDD のこころ
Takuto Wada
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
Nobuhiro Yoshitake
開発速度が速い #とは(LayerX社内資料)
開発速度が速い #とは(LayerX社内資料)
mosa siru
ユーザーストーリー駆動開発で行こう。
ユーザーストーリー駆動開発で行こう。
toshihiro ichitani
テストとリファクタリングに関する深い方法論 #wewlc_jp
テストとリファクタリングに関する深い方法論 #wewlc_jp
kyon mm
振り返り(アジャイルレトロスペクティブズ)
振り返り(アジャイルレトロスペクティブズ)
Keisuke Tameyasu
シリコンバレーの「何が」凄いのか
シリコンバレーの「何が」凄いのか
Atsushi Nakada
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
Yasuharu Nishi
メトリクスによるソフトウェア品質評価・改善および製品品質実態
メトリクスによるソフトウェア品質評価・改善および製品品質実態
Hironori Washizaki
世界でいちばんわかりやすいドメイン駆動設計
世界でいちばんわかりやすいドメイン駆動設計
増田 亨
マイクロにしすぎた結果がこれだよ!
マイクロにしすぎた結果がこれだよ!
mosa siru
IT系エンジニアのためのプレゼンテーション入門
IT系エンジニアのためのプレゼンテーション入門
Masahito Zembutsu
テストエンジニアの品格 #automatornight
テストエンジニアの品格 #automatornight
kyon mm
僕がつくった 70個のうちの48個のWebサービス達
僕がつくった 70個のうちの48個のWebサービス達
Yusuke Wada
ユーザーインタビューするときは、どうやらゾンビのおでましさ
ユーザーインタビューするときは、どうやらゾンビのおでましさ
Yoshiki Hayama
LINE Developer Meetup in Tokyo #39 Presentation
LINE Developer Meetup in Tokyo #39 Presentation
Yasuharu Nishi
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
Yoshiki Hayama
プレゼン初心者にありがちなアンチパターン
プレゼン初心者にありがちなアンチパターン
真俊 横田
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
森 哲也
例外設計における大罪
例外設計における大罪
Takuto Wada
DevIoSapporo.pdf
DevIoSapporo.pdf
HisakoShimizuHisakoS
Scratchを使った CaT的脳トレのすすめ
Scratchを使った CaT的脳トレのすすめ
Masao Niizuma
More Related Content
What's hot
TDD のこころ
TDD のこころ
Takuto Wada
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
Nobuhiro Yoshitake
開発速度が速い #とは(LayerX社内資料)
開発速度が速い #とは(LayerX社内資料)
mosa siru
ユーザーストーリー駆動開発で行こう。
ユーザーストーリー駆動開発で行こう。
toshihiro ichitani
テストとリファクタリングに関する深い方法論 #wewlc_jp
テストとリファクタリングに関する深い方法論 #wewlc_jp
kyon mm
振り返り(アジャイルレトロスペクティブズ)
振り返り(アジャイルレトロスペクティブズ)
Keisuke Tameyasu
シリコンバレーの「何が」凄いのか
シリコンバレーの「何が」凄いのか
Atsushi Nakada
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
Yasuharu Nishi
メトリクスによるソフトウェア品質評価・改善および製品品質実態
メトリクスによるソフトウェア品質評価・改善および製品品質実態
Hironori Washizaki
世界でいちばんわかりやすいドメイン駆動設計
世界でいちばんわかりやすいドメイン駆動設計
増田 亨
マイクロにしすぎた結果がこれだよ!
マイクロにしすぎた結果がこれだよ!
mosa siru
IT系エンジニアのためのプレゼンテーション入門
IT系エンジニアのためのプレゼンテーション入門
Masahito Zembutsu
テストエンジニアの品格 #automatornight
テストエンジニアの品格 #automatornight
kyon mm
僕がつくった 70個のうちの48個のWebサービス達
僕がつくった 70個のうちの48個のWebサービス達
Yusuke Wada
ユーザーインタビューするときは、どうやらゾンビのおでましさ
ユーザーインタビューするときは、どうやらゾンビのおでましさ
Yoshiki Hayama
LINE Developer Meetup in Tokyo #39 Presentation
LINE Developer Meetup in Tokyo #39 Presentation
Yasuharu Nishi
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
Yoshiki Hayama
プレゼン初心者にありがちなアンチパターン
プレゼン初心者にありがちなアンチパターン
真俊 横田
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
森 哲也
例外設計における大罪
例外設計における大罪
Takuto Wada
What's hot
(20)
TDD のこころ
TDD のこころ
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
インセプションデッキ:やらないことリストとトレードオフスライダーをやってる話
開発速度が速い #とは(LayerX社内資料)
開発速度が速い #とは(LayerX社内資料)
ユーザーストーリー駆動開発で行こう。
ユーザーストーリー駆動開発で行こう。
テストとリファクタリングに関する深い方法論 #wewlc_jp
テストとリファクタリングに関する深い方法論 #wewlc_jp
振り返り(アジャイルレトロスペクティブズ)
振り返り(アジャイルレトロスペクティブズ)
シリコンバレーの「何が」凄いのか
シリコンバレーの「何が」凄いのか
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
品質を加速させるために、テスターを増やす前から考えるべきQMファンネルの話(3D版)
メトリクスによるソフトウェア品質評価・改善および製品品質実態
メトリクスによるソフトウェア品質評価・改善および製品品質実態
世界でいちばんわかりやすいドメイン駆動設計
世界でいちばんわかりやすいドメイン駆動設計
マイクロにしすぎた結果がこれだよ!
マイクロにしすぎた結果がこれだよ!
IT系エンジニアのためのプレゼンテーション入門
IT系エンジニアのためのプレゼンテーション入門
テストエンジニアの品格 #automatornight
テストエンジニアの品格 #automatornight
僕がつくった 70個のうちの48個のWebサービス達
僕がつくった 70個のうちの48個のWebサービス達
ユーザーインタビューするときは、どうやらゾンビのおでましさ
ユーザーインタビューするときは、どうやらゾンビのおでましさ
LINE Developer Meetup in Tokyo #39 Presentation
LINE Developer Meetup in Tokyo #39 Presentation
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
45分間で「ユーザー中心のものづくり」ができるまで詰め込む
プレゼン初心者にありがちなアンチパターン
プレゼン初心者にありがちなアンチパターン
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
パワポ版の落合先生流論文要旨のテンプレートを作ったので配布する
例外設計における大罪
例外設計における大罪
Similar to 2021-12-16 テストコードのないレガシーアプリケーションとの向き合い方
DevIoSapporo.pdf
DevIoSapporo.pdf
HisakoShimizuHisakoS
Scratchを使った CaT的脳トレのすすめ
Scratchを使った CaT的脳トレのすすめ
Masao Niizuma
Can We Change The World?
Can We Change The World?
toshihiro ichitani
2022-12-07 この素晴らしいアプリケーションにテストコードを
2022-12-07 この素晴らしいアプリケーションにテストコードを
naoto teshima
塹壕戦から揚陸艇強襲上陸まで (2012/03/17 pyfes)
塹壕戦から揚陸艇強襲上陸まで (2012/03/17 pyfes)
natsu_bm
独学プログラマーのその後
独学プログラマーのその後
Takayuki Shimizukawa
特にタイトルはない
特にタイトルはない
Aya Komuro
千年繁栄する法
千年繁栄する法
Ashitaba YOSHIOKA
江戸川大学講義 "How to Connect the Dots"
江戸川大学講義 "How to Connect the Dots"
Osamu Ise
20230314_JBUG_miyagi_LT_tm_v4.pptx
20230314_JBUG_miyagi_LT_tm_v4.pptx
Toshiki Miyaji
Experience Visionのはじめかた に見るDevLOVE勉強会のススメ
Experience Visionのはじめかた に見るDevLOVE勉強会のススメ
陽一 滝川
Rでを作る
Rでを作る
Nagi Teramo
Jaws niigata#8
Jaws niigata#8
Kahori Takeda
Similar to 2021-12-16 テストコードのないレガシーアプリケーションとの向き合い方
(13)
DevIoSapporo.pdf
DevIoSapporo.pdf
Scratchを使った CaT的脳トレのすすめ
Scratchを使った CaT的脳トレのすすめ
Can We Change The World?
Can We Change The World?
2022-12-07 この素晴らしいアプリケーションにテストコードを
2022-12-07 この素晴らしいアプリケーションにテストコードを
塹壕戦から揚陸艇強襲上陸まで (2012/03/17 pyfes)
塹壕戦から揚陸艇強襲上陸まで (2012/03/17 pyfes)
独学プログラマーのその後
独学プログラマーのその後
特にタイトルはない
特にタイトルはない
千年繁栄する法
千年繁栄する法
江戸川大学講義 "How to Connect the Dots"
江戸川大学講義 "How to Connect the Dots"
20230314_JBUG_miyagi_LT_tm_v4.pptx
20230314_JBUG_miyagi_LT_tm_v4.pptx
Experience Visionのはじめかた に見るDevLOVE勉強会のススメ
Experience Visionのはじめかた に見るDevLOVE勉強会のススメ
Rでを作る
Rでを作る
Jaws niigata#8
Jaws niigata#8
More from naoto teshima
2022-03-25 コミュニティから学ぶエンジニアリング
2022-03-25 コミュニティから学ぶエンジニアリング
naoto teshima
2019-07-27 はじめよう、ニコカレ!
2019-07-27 はじめよう、ニコカレ!
naoto teshima
2019-03-23 プルリクとの上手な付き合い方
2019-03-23 プルリクとの上手な付き合い方
naoto teshima
2018-12-12 ティファニーで転職を〜夏の日の2018〜
2018-12-12 ティファニーで転職を〜夏の日の2018〜
naoto teshima
ニコカレという名のアジャイルプラクティス
ニコカレという名のアジャイルプラクティス
naoto teshima
コミュニケーションから始まるアジャイル - Agile Japan 2018 サテライト長崎
コミュニケーションから始まるアジャイル - Agile Japan 2018 サテライト長崎
naoto teshima
アジャイルジャパン2018 参加レポート
アジャイルジャパン2018 参加レポート
naoto teshima
社内SEはVBAを投げ捨ててPHPの夢を見るか?-おかわり
社内SEはVBAを投げ捨ててPHPの夢を見るか?-おかわり
naoto teshima
社内SEはVBAを投げ捨ててPHPの夢を見るか?
社内SEはVBAを投げ捨ててPHPの夢を見るか?
naoto teshima
いっぽんのサービスの向こうに_公開用
いっぽんのサービスの向こうに_公開用
naoto teshima
現場からは以上です。
現場からは以上です。
naoto teshima
Relation with Nagasaki And Agile
Relation with Nagasaki And Agile
naoto teshima
More from naoto teshima
(12)
2022-03-25 コミュニティから学ぶエンジニアリング
2022-03-25 コミュニティから学ぶエンジニアリング
2019-07-27 はじめよう、ニコカレ!
2019-07-27 はじめよう、ニコカレ!
2019-03-23 プルリクとの上手な付き合い方
2019-03-23 プルリクとの上手な付き合い方
2018-12-12 ティファニーで転職を〜夏の日の2018〜
2018-12-12 ティファニーで転職を〜夏の日の2018〜
ニコカレという名のアジャイルプラクティス
ニコカレという名のアジャイルプラクティス
コミュニケーションから始まるアジャイル - Agile Japan 2018 サテライト長崎
コミュニケーションから始まるアジャイル - Agile Japan 2018 サテライト長崎
アジャイルジャパン2018 参加レポート
アジャイルジャパン2018 参加レポート
社内SEはVBAを投げ捨ててPHPの夢を見るか?-おかわり
社内SEはVBAを投げ捨ててPHPの夢を見るか?-おかわり
社内SEはVBAを投げ捨ててPHPの夢を見るか?
社内SEはVBAを投げ捨ててPHPの夢を見るか?
いっぽんのサービスの向こうに_公開用
いっぽんのサービスの向こうに_公開用
現場からは以上です。
現場からは以上です。
Relation with Nagasaki And Agile
Relation with Nagasaki And Agile
2021-12-16 テストコードのないレガシーアプリケーションとの向き合い方
1.
テストコードのない レガシーアプリケーション との向き合い方 in JaSST'21 Kyushu 2021-12-18
Sat
2.
はじめまして \の方もそうでない方も/
3.
はじめまして 3
4.
はじめまして 4 GMOペパボ株式会社 ホスティング事業部 バックエンドエンジニア 趣味: インターネットJK・ キャンプ @tosite / まおちゃ
/ 手島 尚人
5.
はい \会社紹介/
6.
はじめまして 6
7.
はじめまして 7 ドメイン・レンタルサーバー ご用命の際はぜひ弊社サービスを よろしくお願いいたします!
8.
はじめまして 8
9.
はじめまして 9 コミュニティでステッカーやファングッズ・ ノベルティなどを作成したい場合は ご利用ください!
10.
ここから本題 \いい仕事した/
11.
突然ですが皆さん テストは 書いていますか? \かの有名な方もこうおっしゃっていますよね/
12.
突然ですが皆さん テストは 書いていますか? 著者: @mty_mno 「テスターちゃん番外編『ヤツが来る!!』の巻&メイキングオブジョジョ風の絵
」
13.
と言っても テストが不十分な アプリケーションが あるのも現実… \理想と現実ってやつ/
14.
前提 14 今年のテーマは「歴史から学ぶ」
15.
前提 15 歴史の話をしよう
16.
前提 16
17.
前提 17 ホスティング事業部のサービスは 20年もの長い歴史があります 今も一部リポジトリでは 次のようなレガシーな構成を 目にすることがあります \今回担当したのはムームードメインのPHPで書かれたアプリケーションコードでした/
18.
前提 18 レガシーなアプリケーションではまれに 次のような構成を目にすることがあります \ちなみに案件はPHPでした/
19.
前提 19 コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行)
20.
前提 20 コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行)
1ファイルで全ての処理を実行している コントローラー・ビュー・ ビジネスロジックが密結合している ためテストが難しい 当然、既存のテストコードは ほとんどない状態
21.
前提 21 1ファイル(数千行) コントローラーロジック(数百行) 関数群(数百〜数千行) ビューファイル(数百〜数千行)
・1ファイルで全ての処理を実行 ・コントローラー・ビュー・ ビジネスロジックが密結合 しているためテストが難しい ・当然、既存のテストコードは ほとんどない状態
22.
前提 22 リファクタリングしたいが テストコードも仕様書もなく 「動いているコードが正しい」 という状態
23.
前提 23 要件は 振る舞いを変えつつ 変更後の挙動を担保した状態で かつバグが混入しないこと というもの
24.
前提 24 つまり リアーキテクティング した上で振る舞いを変える 必要があるということ
25.
前提 25 リアーキテクティングするための テストがない コードが密結合しているため ユニットテストが書けない \鶏卵問題ですね/
26.
前提 26 その状況でいきなり 振る舞いを変えるのは 自殺行為
27.
前提 27 リファクタリングするためのテストがなく、 テストを書こうにも密結合しているため そのままではテストできない とは言えいきなりリファクタリングするのも エンバグする危険性が極めて高い \鶏卵問題ですね/
28.
皆さんなら どうしますか? \一緒に考えてみましょう/
29.
私は次のような アプローチを取りました \一例ですが/
30.
フェーズ1. ロジックの分離 \ローマの道も一歩から/
31.
フェーズ1. ロジックの分離 31
32.
フェーズ1. ロジックの分離 32 これを
33.
フェーズ1. ロジックの分離 33 index.php functions.php
34.
フェーズ1. ロジックの分離 34 こう
35.
フェーズ1. ロジックの分離 35 まずはファイルから ビジネスロジックを 切り出すだけの作業 を行った 😅
36.
フェーズ2. 既存の関数に対して テストを記述 \リファクタリングらしくなってきました/
37.
フェーズ2. 既存の関数に対してテストを記述 37
38.
フェーズ2. 既存の関数に対してテストを記述 38 既存の仕様を満たす テストケースを記載
39.
フェーズ2. 既存の関数に対してテストを記述 39 ファイルを分割したので テストが書ける!! 😃
40.
フェーズ2. 既存の関数に対してテストを記述 40 とは言え全ての関数にテストを 追加する工数はなかったので
リアーキテクティング対象 のみに留めた
41.
フェーズ3. リアーキテクティング \テストがあるので安全だね/
42.
フェーズ3. リアーキテクティング 42
43.
フェーズ3. リアーキテクティング 43 気になる部分を リファクタ
44.
フェーズ4. 落ちるテストを記述 \赤より始めよ/
45.
フェーズ3. 落ちるテストを記述 45 修正する関数のテストケース 既存の仕様を満たすテスト郡
リファクタリング後に 仕様を満たすようになるテスト郡 リファクタリング前にレッドにな ることを確認しておく リファクタリング後は グリーンになることを確認 することで安全に仕様を 変えることができる 脆弱性が混入されていないことを 確認するテスト郡
46.
フェーズ4. 落ちるテストを記述 46 振る舞い変更前 =>
レッド 振る舞い変更後 => グリーン
47.
フェーズ4. 落ちるテストを記述 47 「既存仕様が壊れていないこと」は 現在の仕様を満たすテストコードから確認できる 「新規仕様が実装されたこと」は 落ちるテストが通ることで確認できる
48.
フェーズ5. 振る舞いを変更 \さあ、コーディングの時間だ!/
49.
フェーズ5. 振る舞いを変更 49 ここまで来たら安心安全に 振る舞いを変更できる ようになっている
50.
フェーズ5. 振る舞いを変更 50
51.
フェーズ5. 振る舞いを変更 51 フェーズ3で記述した 落ちるテストが 通るように修正 したら実装完了!
52.
フェーズ5. 振る舞いを変更 52 全てグリーンで通過!
53.
メリット \こんないいことがあったよ/
54.
メリット1.
55.
メリット 55 テストがない世界からの脱却! 今後機能追加や改修をやるとなった ときの開発基盤が整った
56.
メリット2.
57.
メリット 57 コードベースだけでなく テストケースベースで レビューできる ので安心感があった
58.
メリット 58 特に一部関数には 正規表現が使用されており 修正前と後で目grepで挙動を 担保するのはつらぽよだった 修正後の挙動を テストケースという仕様書 にできた
59.
メリット3.
60.
メリット 60 関数群を切り出すことによって スコープが明確になった 関数がある 副次的にその関数の リファクタを推し進められた
61.
まとめ \もうこんな時間/
62.
レガシーな肥大化した アプリケーションを リファクタするのは 難しい
63.
特にテストコードが ない場合はなおさら
64.
でもアプローチを 工夫すれば必ず安全に やり遂げられる
65.
皆さんもテストを書いて 安全にリファクタして いきましょう!
66.
ご清聴ありがとう ございました
Download now