Remove redundant explicit RSpec `type:` metadata from specs
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
先从 spec/rails_helper.rb 和 spec/spec_helper.rb 开始，验证推断出的元数据，然后检查 issue 中列出的十二个受影响的 spec 文件。仅从每个 RSpec.describe 行中删除冗余的显式类型元数据，并运行测试套件。当这些文件中不再有显式类型声明且测试套件通过时，即表示完成。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Ruby
- 星标
- 104
- 派生
- 205
- 平均合并
- 1 天 2 小时
- 30 天内合并 PR
- 77
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
codebar/planner 的其他 Issue
-
enhancement
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 56/100
维护者通常 1 天内回复
-
Chapter show pages allocate ~21k+ objects per render for large chapters (18% of app allocations)未关闭performance
难度 4/5 3-5 天 新手友好度 55/100
codebar/planner#2952 · 已指派 1 人 ·
维护者通常 1 天内回复
-
performance
难度 4/5 3-5 天 新手友好度 52/100
维护者通常 1 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
FreeCAD/homebrew-freecad#870 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
security
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
OSCON 2016未关闭content
难度 2/5 1-3 小时 新手友好度 68/100
rubyevents/rubyevents#2148 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
we-promise/sure#3838 · 2 条评论 ·
维护者通常 1 天内回复