Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Remove redundant explicit RSpec `type:` metadata from specs

Fermée Adaptée aux débutants
  • Quelque chose ne va pas ?
#2,934 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
88/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
rails, ruby
Domaine
testing

Piste de recherche

Commencez par spec/rails_helper.rb et spec/spec_helper.rb afin de vérifier les métadonnées déduites, puis examinez les douze fichiers spec concernés listés dans l’issue. Supprimez uniquement les métadonnées de type explicites redondantes de chaque ligne RSpec.describe et exécutez la suite de tests. La tâche est terminée lorsque la suite passe sans aucune déclaration de type explicite dans ces fichiers.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

good first issue tech debt

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.rb
  • spec/services/signup_nudge_email_service_spec.rb
  • spec/services/three_month_email_service_spec.rb
  • spec/services/subscription_tombstone_backfill_spec.rb
  • spec/services/admin/stats/range_spec.rb
  • spec/lib/tasks/mailing_list_rake_spec.rb
  • spec/lib/tasks/delete_member_rake_spec.rb
  • spec/lib/tasks/reminders_workshop_rake_spec.rb
  • spec/lib/tasks/reminders_meeting_rake_spec.rb
  • spec/lib/tasks/feedback_rake_spec.rb
  • spec/lib/tasks/chaser_rake_spec.rb
  • spec/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.

Langage dominant
Ruby
Étoiles
104
Forks
205
Merge moyen
1 j 4 h
PR mergées (30 j)
77

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de codebar/planner

Toutes les issues de codebar/planner

Issues similaires

Plus d'issues Ruby

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.