`RuboCop::Cop::Airbnb::ModuleMethodInWrongFile` returns incorrect error message when defining a method inside singleton class
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia da rubocop-airbnb/lib/rubocop/cop/airbnb/module_method_in_wrong_file.rb intorno alla riga 88 e dalla spec in rubocop-airbnb/spec/rubocop/cop/airbnb/module_method_in_wrong_file_spec.rb intorno alla riga 96. Esegui questa spec e ispeziona normalize_module_name(node.parent_module_name). È completato quando il caso singleton-class segnala “Method baz should be defined in foo.rb” e la spec verifica il testo dell’offense.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Ruby
- Stelle
- 3.9k
- Fork
- 757
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né 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 airbnb/ruby
-
Warnings about rule namespaces for Capybara and FactoryBotForse già presa @mitorez l’ha presa 140 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
TheOdinProject/curriculum#31467 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
message-patterns
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
sisimai/rb-sisimai#468 ·
-
202610-volunteers my-first-pr PROJECT: website-redesign
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
thewca/worldcubeassociation.org#16072 ·
I maintainer di solito rispondono entro 1 giorno