Remove redundant explicit RSpec `type:` metadata from specs
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
Línea de trabajo
Comienza con spec/rails_helper.rb y spec/spec_helper.rb para verificar los metadatos inferidos; después, inspecciona los doce archivos spec afectados que se indican en el issue. Elimina únicamente los metadatos de tipo explícitos redundantes de cada línea de RSpec.describe y ejecuta la suite de pruebas. La tarea estará terminada cuando la suite pase sin declaraciones de tipo explícitas en esos archivos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Ruby
- Estrellas
- 104
- Forks
- 205
- Merge medio
- 1 d 5 h
- PR fusionados (30 d)
- 72
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de codebar/planner
-
performance
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
good first issue tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
good first issue refactoring tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Chapter show pages allocate ~21k+ objects per render for large chapters (18% of app allocations)Abiertoperformance
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Los mantenedores suelen responder en 1 día
Todos los issues de codebar/planner
Issues similares
-
P2 testing
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
Blame view link labels are wrongAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
openSUSE/open-build-service#20338 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
DB上でコメント本文がNULLを許容しているAbiertoバグ
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100