yochiyochirb/kajaeru

RSpec 関連のファイルなどを消す

Closed this issue · 4 comments

yucao24hours commented

#120 でテストを作り始めたので、今ある RSpec 関連のファイルなどを消したい。

今後 RSpec を使うことももしかしたらあるかもしれないけど、今すぐには必要ないため。

via #87

yucao24hours commented

こちらは今日のミートアップで azuma さんが既に対応してくださっています。（諸処の事情があってまだ push できていないとのことなので、push していただき次第 PR ができる予定です）

yucao24hours commented

あと消す必要がありそうなやつ

  • .rspec を消す
  • features/support/database_cleaner.rb に DatabaseCleaner の設定を書き直す
# features/support/env.rb に以下を書く
DatabaseCleaner.clean_with :truncation
  • spec/ ディレクトリまるごと消す
yucao24hours commented

わたしが spec ディレクトリをまるごと消そうとしたとき、「どうやって 不要かどうかを判別するのか？」というはなしが 3/7 のミートアップでありました。
rspec-rails の中で spec ディレクトリをその関連ファイルとして読みこむように指定されているから、というのが抽象的な回答になると思うのですが、rails_helper.rb にもそのようなコメントが入っています（わたしが過去に消してしまったけど）。こちらは spec/support に関する話。

https://github.com/rspec/rspec-rails/blob/078b29fefb129b7942ba512cc0173b0c6d20c582/lib/generators/rspec/install/templates/spec/rails_helper.rb#L10-L21

あとは、 https://github.com/rspec/rspec-rails/blob/98ce7670832f96fd2420bac557723543c5a9b1ba/lib/rspec/rails/configuration.rb#L27-L37 とかで指定しているのかな。こちらは RSpec.configure で使う infer_spec_type_from_file_location! というメソッドの実装で使う定数ですが。

yucao24hours commented

@murasakiToshiya キリの良いところで、先に Pull Request を作っておいていただけると確認しやすいので助かります 🙏

Pull Request の title

[WIP] RSpec 関連のファイルや設定を消す
（WIP とは Work In Progress の略で、このブランチはまだ作業中ですよということがひと目でわかるようにするためのしるしです）

Pull Request の description:

## やったこと

このブランチでやったこと（やっていること）を簡潔にして書いておいてください

## TODO
- [ ] .rspec を消す
- [ ] features/support/env.rb  に以下の DatabaseCleaner の設定を追加する
    - `DatabaseCleaner.clean_with :truncation`
- [ ] spec/ ディレクトリまるごと消す
- [ ] 他に必要そうなタスクがあったら書いておいてください

## 対応する issue

#121

みたいな感じで。
TODO のチェックボックスについては終わり次第チェックを入れていって、このブランチの作業の進捗がわかるようにしておいていただきたいです。で、全部チェックが入ったら、タイトルの [WIP] を削除する、というフローでいきたいと思っています。
よくわからないところがあったら idobata でもこの issue でもいいので、いつでもおたずねください！ 🙋