mirror of
https://github.com/discourse/discourse.git
synced 2026-08-12 05:37:26 +08:00
6df9b0b357RSpec setup becomes harder to follow at either extreme: trivial fixture wrappers hide lifecycle and intent, while forcing every named operation inline repeats low-level protocol and configuration details. This change documents and applies a test-setup hierarchy: - use `fab!`, `let`, `let!`, `subject`, and inline `Fabricate` according to lifecycle and role; - use a small example-group method when parameterized behavior gives one spec useful vocabulary; - move helpers into auto-loaded `spec/support` only when they are shared across spec files; - use fabricators and page objects for the data shapes and system-test interfaces they own. Core and plugin support files are loaded centrally by `rails_helper`, so plugin-specific support loaders are unnecessary. The migration specs encountered during the sweep are removed according to repository policy; production migrations are unchanged. |
||
|---|---|---|
| .. | ||
| page_objects | ||
| admin_dashboard_manage_reports_hint_spec.rb | ||
| ai_query_generation_spec.rb | ||
| bookmark_spec.rb | ||
| core_features_spec.rb | ||
| new_query_spec.rb | ||
| param_input_spec.rb | ||
| query_runner_spec.rb | ||
| reports_spec.rb | ||
