Remove redundant explicit RSpec `type:` metadata from specs
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
Direzione di ricerca
Inizia da spec/rails_helper.rb e spec/spec_helper.rb per verificare i metadati dedotti, quindi esamina i dodici file spec interessati elencati nell’issue. Rimuovi solo i metadati di tipo espliciti ridondanti da ogni riga RSpec.describe ed esegui la suite di test. Il lavoro è completato quando la suite passa senza dichiarazioni di tipo esplicite in quei file.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Raised in review comment on #2923: spec/services/newsletter_subscription_service_spec.rb declares RSpec.describe NewsletterSubscriptionService, type: :service, but the type: :service metadatum is unnecessary.
spec/rails_helper.rb and spec/spec_helper.rb both call config.infer_spec_type_from_file_location!, which infers the type from the spec's directory (spec/services/ → :service, spec/lib/tasks/ → :task, and so on). The explicit metadata adds nothing and invites drift if a spec moves directory.
Affected files (all carry a redundant type:):
spec/services/newsletter_subscription_service_spec.rbspec/services/signup_nudge_email_service_spec.rbspec/services/three_month_email_service_spec.rbspec/services/subscription_tombstone_backfill_spec.rbspec/services/admin/stats/range_spec.rbspec/lib/tasks/mailing_list_rake_spec.rbspec/lib/tasks/delete_member_rake_spec.rbspec/lib/tasks/reminders_workshop_rake_spec.rbspec/lib/tasks/reminders_meeting_rake_spec.rbspec/lib/tasks/feedback_rake_spec.rbspec/lib/tasks/chaser_rake_spec.rbspec/presenters/workshop_presenter_capacity_spec.rb
Suggested change: drop the , type: :service (etc.) from each RSpec.describe line and run the suite to confirm nothing changes. config.include ... type: :component in rails_helper.rb and similar conditional includes keep working, since they match on inferred metadata too.
- Lingua principale
- Ruby
- Stelle
- 104
- Fork
- 205
- Merge medio
- 1g 6h
- PR unite (30g)
- 70
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di codebar/planner
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
expenses
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
codebar/planner#2929 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di codebar/planner
Issue simili
-
security
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
solana-foundation/pay-kit#341 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
julienXX/terminal-notifier#333 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
serhii-londar/open-source-mac-os-apps#1419 ·
I maintainer di solito rispondono entro 5 giorni