新卒2ヶ月目で起こしたインシデントの話

Slide 1

Slide 1 text

Slide 2

Slide 2 text

自己紹介氏家虎之介 (Ujike Toranosuke) フロントエンドエンジニアとして、 Wantedly Visitを主にUI 面から改善することで、ユーザー、企業への提供価値を向上させている。 © 2023 Wantedly, Inc. 所属: Wantedly, inc. X: @tora_tora_bit ある特定の技術領域に縛られないことをモットーに、現在は優秀な同期と共にバックエンドについて学習中。

Slide 3

Slide 3 text

Slide 4

Slide 4 text

Slide 5

Slide 5 text

Slide 6

Slide 6 text

Slide 7

Slide 7 text

Slide 8

Slide 8 text

Slide 9

Slide 9 text

Slide 10

Slide 10 text

実際に何が起こっていたのか © 2023 Wantedly, Inc. user = UserHashService.compose!( # hashを作成 name: name, age: age, city: city, ) # user hashのkeyはシンボルで宣言されているため文字列で指定できない name = user["name"] age = user["age"] city = user["city"] let(:user) { # モックデータ { "name" => "Tora", "age" => 26, "city" => "Kyoto", } } … allow(UserHashService).to receive(:compose!).and_return(user) … it "user test" do name = { user["name"] } expect(name).to eq("Tora") # モックデータのキーとして扱われているのは文字列なのでテストが通る end 例) user_message_service.rb 例) user_message_service_spec.rb

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

よかったこと ● インシデントを起こした数十分後に上司が出社した ○ 上司と同期的に密なコミュニケーションがとれた ● リリース後の監視がうまくワークした ○ ユーザー問い合わせ前の内部発見に繋がった ● 毎週金曜日に行われている All Hands Meeting 前に気づけた ○ 復旧作業を他のエンジニアと協力して迅速に行えた ○ 焦ることなく、行うべき一次対応に集中できた © 2023 Wantedly, Inc.

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

都合の良いモックデータを与えたことが本当の原因なのか 🤔 © 2023 Wantedly, Inc. UserHashService は内部で UserService を使い User の情報を引っ張ってきている本来は UserHashService の返り値をモックするのではなく UserService の返り値をモックしてあげるべきなのでは？そもそもモックせずに UserService から取得した User のデータをそのまま使うことはできなかったのか？