【 #92 で対応出来たためマージせずにclose】ログインに問題がないtestブランチからlogin_bug_from_testブランチを切って対応

[pekorinko]

やったこと

前提

mainブランチは「GitHubでログイン」ボタンを押すとログイン出来ずにトップページへリダイレクトされてしまい、「ログインしてください」のアラートが出る

• mainブランチの精査、mainブランチから切ったlogin_bugブランチ（プルリク #92）でのログインに関連していそうなログイン周りの編集等実施したが上記問題が解消されない
• そのためログインが問題なく行えるtestブランチから新たにlogin_bug_from_testを切ってこれをmainにマージしたらログインできない問題が解消するのでは無いかと考えた

  問題点

• login_bug_from_testブランチは検索結果URLが入力された場合のselenium関連の処理に対応していない（testブランチが対応していなかったため）
  • selenium_tool.rbで取得した口コミモーダルURLを place_data_scraper.rbのinitializeメソッドに渡す処理を書き加えた
  • その上で口コミがあるGoogleの検索結果URLを入力したところ「口コミの取得に失敗しました」とエラーが出てしまい、自動で取得した口コミモーダルのURLでのスクレイピングが出来なかった
  • 上記の原因はselenium_tool.rbのinitializeメソッドにて本来@wait = Selenium::WebDriver::Wait.new(timeout: 30)で有るべきはずの箇所がwait = Selenium::WebDriver::Wait.new(timeout: 30)となっていたことが原因
  • initializeメソッドだけでなく、replace_url_if_neededでもwaitを使用することから@waitとしてインスタンス変数にする必要があった

    やった理由

• ログインが問題なく行えるlogin_bug_from_testブランチにおいても検索結果URLを入力された際に自動で「Googleの口コミ」ボタンを押して口コミモーダルのURLを取得出来るようにしたかったため
• login_bug_from_testブランチがログイン可能かつ検索結果URLに対応出来るようになれば、本ブランチをmainにマージすることでmainのログインできない問題が解消出来るのではないかと考えたため

  確認項目

• [x] 検索結果URLが入力された場合は自動で「Googleの口コミ」ボタンを押して口コミモーダルのURLを取得してスクレイピングする
• [x] login_bug_from_testブランチにおいては「GitHubでログイン」ボタンを押すと問題なくログイン後の画面に遷移出来る

  Issue

  Fix https://github.com/pekorinko/review_check/issues/91#issuecomment-855478588