`RuboCop::Cop::Airbnb::ModuleMethodInWrongFile` returns incorrect error message when defining a method inside singleton class
还没有人认领这个 Issue。
评估
调研方向
从 rubocop-airbnb/lib/rubocop/cop/airbnb/module_method_in_wrong_file.rb 第 88 行附近以及 rubocop-airbnb/spec/rubocop/cop/airbnb/module_method_in_wrong_file_spec.rb 第 96 行附近的 spec 开始。运行该 spec，并检查 normalize_module_name(node.parent_module_name)。当 singleton-class 情况报告 “Method baz should be defined in foo.rb”，且 spec 检查 offense 文本时，即表示完成。
由索引模型根据 Issue 内容生成。
描述
When developing PR #189, tests were changed to use new rspec-rubocop syntax which also checks that the message generated is correct. After doing this, the spec that checks that an error is returned when using "<<" static methods and a non-matching name failed with an incorrect message:
Method baz should be defined in foo/foo.rb
When the expected message was:
Method baz should be defined in foo.rb
After digging a little about it to understand what was happening, we realized that this incorrect message is also present on master – but there are no assertions respecting the offense's text, thus the test succeeds.
The issue seems to be caused by the method normalize_module_name(node.parent_module_name) (on module_method_in_wrong_file.rb#L88) which returns Foo::Foo instead of just Foo.
- 主要语言
- Ruby
- 星标
- 3.9k
- 派生
- 757
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
airbnb/ruby 的其他 Issue
-
Warnings about rule namespaces for Capybara and FactoryBot可能已有人在做 @mitorez 于 138 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 58/100
-
难度 5/5 一周以上 新手友好度 10/100
-
难度 2/5 1-3 小时 新手友好度 48/100
-
难度 3/5 1-2 天 新手友好度 35/100
-
难度 2/5 1-3 小时 新手友好度 45/100
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 75/100
GantMan/jail-monkey#380 ·
-
难度 2/5 1-3 小时 新手友好度 62/100
voxpupuli/puppet-wireguard#151 ·
-
bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
bug f:search
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复