rubocop 0.51.0 → 0.52.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (360) hide show
  1. checksums.yaml +4 -4
  2. data/README.md +1 -2
  3. data/config/default.yml +86 -10
  4. data/config/disabled.yml +9 -4
  5. data/config/enabled.yml +1105 -1006
  6. data/lib/rubocop.rb +30 -3
  7. data/lib/rubocop/ast/builder.rb +2 -0
  8. data/lib/rubocop/ast/node.rb +24 -2
  9. data/lib/rubocop/ast/node/args_node.rb +17 -0
  10. data/lib/rubocop/ast/node/array_node.rb +1 -1
  11. data/lib/rubocop/ast/node/mixin/basic_literal_node.rb +16 -0
  12. data/lib/rubocop/ast/node/send_node.rb +20 -0
  13. data/lib/rubocop/ast/node/str_node.rb +14 -0
  14. data/lib/rubocop/ast/node/symbol_node.rb +20 -0
  15. data/lib/rubocop/ast/traversal.rb +4 -4
  16. data/lib/rubocop/cli.rb +45 -4
  17. data/lib/rubocop/comment_config.rb +2 -5
  18. data/lib/rubocop/config.rb +42 -8
  19. data/lib/rubocop/config_loader.rb +39 -100
  20. data/lib/rubocop/config_loader_resolver.rb +99 -2
  21. data/lib/rubocop/cop/autocorrect_logic.rb +1 -1
  22. data/lib/rubocop/cop/bundler/duplicated_gem.rb +7 -7
  23. data/lib/rubocop/cop/bundler/insecure_protocol_source.rb +4 -3
  24. data/lib/rubocop/cop/bundler/ordered_gems.rb +2 -2
  25. data/lib/rubocop/cop/cop.rb +16 -46
  26. data/lib/rubocop/cop/gemspec/duplicated_assignment.rb +102 -0
  27. data/lib/rubocop/cop/gemspec/ordered_dependencies.rb +3 -2
  28. data/lib/rubocop/cop/gemspec/required_ruby_version.rb +87 -0
  29. data/lib/rubocop/cop/generator.rb +23 -4
  30. data/lib/rubocop/cop/internal_affairs.rb +1 -1
  31. data/lib/rubocop/cop/internal_affairs/node_destructuring.rb +46 -0
  32. data/lib/rubocop/cop/internal_affairs/node_type_predicate.rb +2 -2
  33. data/lib/rubocop/cop/internal_affairs/offense_location_keyword.rb +9 -8
  34. data/lib/rubocop/cop/internal_affairs/redundant_location_argument.rb +1 -1
  35. data/lib/rubocop/cop/internal_affairs/redundant_message_argument.rb +1 -1
  36. data/lib/rubocop/cop/layout/access_modifier_indentation.rb +4 -7
  37. data/lib/rubocop/cop/layout/align_hash.rb +16 -16
  38. data/lib/rubocop/cop/layout/align_parameters.rb +6 -12
  39. data/lib/rubocop/cop/layout/block_end_newline.rb +19 -5
  40. data/lib/rubocop/cop/layout/case_indentation.rb +14 -17
  41. data/lib/rubocop/cop/layout/class_structure.rb +306 -0
  42. data/lib/rubocop/cop/layout/closing_parenthesis_indentation.rb +1 -0
  43. data/lib/rubocop/cop/layout/comment_indentation.rb +39 -2
  44. data/lib/rubocop/cop/layout/dot_position.rb +22 -13
  45. data/lib/rubocop/cop/layout/else_alignment.rb +6 -6
  46. data/lib/rubocop/cop/layout/empty_line_between_defs.rb +17 -17
  47. data/lib/rubocop/cop/layout/empty_lines.rb +15 -1
  48. data/lib/rubocop/cop/layout/empty_lines_around_access_modifier.rb +3 -2
  49. data/lib/rubocop/cop/layout/empty_lines_around_arguments.rb +89 -0
  50. data/lib/rubocop/cop/layout/empty_lines_around_begin_body.rb +2 -2
  51. data/lib/rubocop/cop/layout/empty_lines_around_block_body.rb +4 -8
  52. data/lib/rubocop/cop/layout/empty_lines_around_class_body.rb +30 -5
  53. data/lib/rubocop/cop/layout/empty_lines_around_exception_handling_keywords.rb +7 -3
  54. data/lib/rubocop/cop/layout/empty_lines_around_method_body.rb +2 -2
  55. data/lib/rubocop/cop/layout/empty_lines_around_module_body.rb +22 -7
  56. data/lib/rubocop/cop/layout/end_of_line.rb +1 -1
  57. data/lib/rubocop/cop/layout/extra_spacing.rb +21 -24
  58. data/lib/rubocop/cop/layout/first_parameter_indentation.rb +4 -1
  59. data/lib/rubocop/cop/layout/indent_array.rb +67 -20
  60. data/lib/rubocop/cop/layout/indent_hash.rb +70 -25
  61. data/lib/rubocop/cop/layout/indent_heredoc.rb +51 -11
  62. data/lib/rubocop/cop/layout/indentation_width.rb +24 -17
  63. data/lib/rubocop/cop/layout/initial_indentation.rb +7 -5
  64. data/lib/rubocop/cop/layout/leading_comment_space.rb +2 -2
  65. data/lib/rubocop/cop/layout/multiline_array_brace_layout.rb +47 -14
  66. data/lib/rubocop/cop/layout/multiline_assignment_layout.rb +10 -9
  67. data/lib/rubocop/cop/layout/multiline_block_layout.rb +19 -16
  68. data/lib/rubocop/cop/layout/multiline_hash_brace_layout.rb +46 -13
  69. data/lib/rubocop/cop/layout/multiline_method_call_indentation.rb +9 -10
  70. data/lib/rubocop/cop/layout/multiline_method_definition_brace_layout.rb +4 -0
  71. data/lib/rubocop/cop/layout/rescue_ensure_alignment.rb +9 -3
  72. data/lib/rubocop/cop/layout/space_after_colon.rb +4 -4
  73. data/lib/rubocop/cop/layout/space_after_comma.rb +3 -3
  74. data/lib/rubocop/cop/layout/space_after_method_name.rb +4 -4
  75. data/lib/rubocop/cop/layout/space_after_semicolon.rb +8 -1
  76. data/lib/rubocop/cop/layout/space_around_block_parameters.rb +16 -19
  77. data/lib/rubocop/cop/layout/space_around_equals_in_parameter_default.rb +27 -16
  78. data/lib/rubocop/cop/layout/space_around_keyword.rb +13 -11
  79. data/lib/rubocop/cop/layout/space_around_operators.rb +26 -13
  80. data/lib/rubocop/cop/layout/space_before_block_braces.rb +10 -10
  81. data/lib/rubocop/cop/layout/space_before_comma.rb +12 -1
  82. data/lib/rubocop/cop/layout/space_before_comment.rb +2 -4
  83. data/lib/rubocop/cop/layout/space_before_first_arg.rb +4 -3
  84. data/lib/rubocop/cop/layout/space_before_semicolon.rb +1 -1
  85. data/lib/rubocop/cop/layout/space_in_lambda_literal.rb +6 -12
  86. data/lib/rubocop/cop/layout/space_inside_array_literal_brackets.rb +235 -0
  87. data/lib/rubocop/cop/layout/space_inside_array_percent_literal.rb +3 -3
  88. data/lib/rubocop/cop/layout/space_inside_block_braces.rb +87 -16
  89. data/lib/rubocop/cop/layout/space_inside_hash_literal_braces.rb +57 -34
  90. data/lib/rubocop/cop/layout/space_inside_parens.rb +31 -3
  91. data/lib/rubocop/cop/layout/space_inside_range_literal.rb +15 -15
  92. data/lib/rubocop/cop/layout/space_inside_reference_brackets.rb +81 -0
  93. data/lib/rubocop/cop/layout/space_inside_string_interpolation.rb +24 -13
  94. data/lib/rubocop/cop/layout/tab.rb +2 -2
  95. data/lib/rubocop/cop/layout/trailing_blank_lines.rb +7 -7
  96. data/lib/rubocop/cop/lint/ambiguous_block_association.rb +11 -20
  97. data/lib/rubocop/cop/lint/block_alignment.rb +26 -23
  98. data/lib/rubocop/cop/lint/boolean_symbol.rb +1 -1
  99. data/lib/rubocop/cop/lint/def_end_alignment.rb +5 -10
  100. data/lib/rubocop/cop/lint/else_layout.rb +2 -2
  101. data/lib/rubocop/cop/lint/end_alignment.rb +13 -14
  102. data/lib/rubocop/cop/lint/ineffective_access_modifier.rb +3 -4
  103. data/lib/rubocop/cop/lint/inherit_exception.rb +10 -14
  104. data/lib/rubocop/cop/lint/missing_cop_enable_directive.rb +81 -0
  105. data/lib/rubocop/cop/lint/nested_percent_literal.rb +58 -0
  106. data/lib/rubocop/cop/lint/non_local_exit_from_iterator.rb +2 -2
  107. data/lib/rubocop/cop/lint/parentheses_as_grouped_expression.rb +1 -1
  108. data/lib/rubocop/cop/lint/percent_string_array.rb +14 -14
  109. data/lib/rubocop/cop/lint/percent_symbol_array.rb +11 -11
  110. data/lib/rubocop/cop/lint/redundant_with_index.rb +6 -3
  111. data/lib/rubocop/cop/lint/redundant_with_object.rb +1 -1
  112. data/lib/rubocop/cop/lint/require_parentheses.rb +3 -1
  113. data/lib/rubocop/cop/lint/rescue_type.rb +3 -3
  114. data/lib/rubocop/cop/lint/script_permission.rb +1 -0
  115. data/lib/rubocop/cop/lint/shadowed_argument.rb +146 -0
  116. data/lib/rubocop/cop/lint/string_conversion_in_interpolation.rb +6 -6
  117. data/lib/rubocop/cop/lint/unneeded_disable.rb +29 -11
  118. data/lib/rubocop/cop/lint/unneeded_require_statement.rb +2 -1
  119. data/lib/rubocop/cop/lint/unneeded_splat_expansion.rb +43 -17
  120. data/lib/rubocop/cop/message_annotator.rb +4 -2
  121. data/lib/rubocop/cop/metrics/abc_size.rb +2 -2
  122. data/lib/rubocop/cop/metrics/class_length.rb +3 -1
  123. data/lib/rubocop/cop/metrics/cyclomatic_complexity.rb +2 -1
  124. data/lib/rubocop/cop/metrics/line_length.rb +2 -2
  125. data/lib/rubocop/cop/metrics/module_length.rb +3 -1
  126. data/lib/rubocop/cop/metrics/parameter_lists.rb +9 -2
  127. data/lib/rubocop/cop/metrics/perceived_complexity.rb +2 -1
  128. data/lib/rubocop/cop/mixin/documentation_comment.rb +1 -1
  129. data/lib/rubocop/cop/mixin/empty_lines_around_body.rb +7 -5
  130. data/lib/rubocop/cop/mixin/empty_parameter.rb +23 -0
  131. data/lib/rubocop/cop/mixin/end_keyword_alignment.rb +7 -3
  132. data/lib/rubocop/cop/mixin/heredoc.rb +19 -19
  133. data/lib/rubocop/cop/mixin/method_complexity.rb +32 -8
  134. data/lib/rubocop/cop/mixin/multiline_expression_indentation.rb +7 -3
  135. data/lib/rubocop/cop/mixin/multiline_literal_brace_layout.rb +6 -4
  136. data/lib/rubocop/cop/mixin/negative_conditional.rb +3 -0
  137. data/lib/rubocop/cop/mixin/ordered_gem_node.rb +16 -12
  138. data/lib/rubocop/cop/mixin/parentheses.rb +12 -0
  139. data/lib/rubocop/cop/mixin/space_after_punctuation.rb +8 -8
  140. data/lib/rubocop/cop/mixin/space_before_punctuation.rb +10 -10
  141. data/lib/rubocop/cop/mixin/statement_modifier.rb +7 -17
  142. data/lib/rubocop/cop/mixin/surrounding_space.rb +95 -8
  143. data/lib/rubocop/cop/mixin/too_many_lines.rb +2 -2
  144. data/lib/rubocop/cop/mixin/trailing_comma.rb +22 -14
  145. data/lib/rubocop/cop/mixin/unused_argument.rb +3 -40
  146. data/lib/rubocop/cop/naming/accessor_method_name.rb +20 -10
  147. data/lib/rubocop/cop/naming/ascii_identifiers.rb +33 -0
  148. data/lib/rubocop/cop/naming/binary_operator_parameter_name.rb +5 -4
  149. data/lib/rubocop/cop/naming/class_and_module_camel_case.rb +16 -12
  150. data/lib/rubocop/cop/naming/constant_name.rb +32 -3
  151. data/lib/rubocop/cop/naming/file_name.rb +19 -8
  152. data/lib/rubocop/cop/naming/heredoc_delimiter_case.rb +10 -16
  153. data/lib/rubocop/cop/naming/method_name.rb +20 -14
  154. data/lib/rubocop/cop/naming/predicate_name.rb +7 -3
  155. data/lib/rubocop/cop/naming/variable_name.rb +17 -2
  156. data/lib/rubocop/cop/naming/variable_number.rb +10 -27
  157. data/lib/rubocop/cop/offense.rb +6 -1
  158. data/lib/rubocop/cop/performance/case_when_splat.rb +2 -6
  159. data/lib/rubocop/cop/performance/casecmp.rb +5 -4
  160. data/lib/rubocop/cop/performance/compare_with_block.rb +13 -7
  161. data/lib/rubocop/cop/performance/count.rb +5 -4
  162. data/lib/rubocop/cop/performance/detect.rb +7 -4
  163. data/lib/rubocop/cop/performance/double_start_end_with.rb +2 -7
  164. data/lib/rubocop/cop/performance/end_with.rb +2 -2
  165. data/lib/rubocop/cop/performance/flat_map.rb +3 -2
  166. data/lib/rubocop/cop/performance/hash_each_methods.rb +41 -40
  167. data/lib/rubocop/cop/performance/lstrip_rstrip.rb +5 -4
  168. data/lib/rubocop/cop/performance/redundant_block_call.rb +22 -22
  169. data/lib/rubocop/cop/performance/redundant_match.rb +2 -2
  170. data/lib/rubocop/cop/performance/redundant_merge.rb +6 -4
  171. data/lib/rubocop/cop/performance/redundant_sort_by.rb +4 -4
  172. data/lib/rubocop/cop/performance/regexp_match.rb +3 -2
  173. data/lib/rubocop/cop/performance/size.rb +2 -2
  174. data/lib/rubocop/cop/performance/start_with.rb +2 -2
  175. data/lib/rubocop/cop/performance/string_replacement.rb +4 -4
  176. data/lib/rubocop/cop/performance/times_map.rb +14 -15
  177. data/lib/rubocop/cop/performance/uri_default_parser.rb +3 -3
  178. data/lib/rubocop/cop/rails/action_filter.rb +22 -0
  179. data/lib/rubocop/cop/rails/active_support_aliases.rb +2 -2
  180. data/lib/rubocop/cop/rails/application_job.rb +4 -2
  181. data/lib/rubocop/cop/rails/application_record.rb +4 -2
  182. data/lib/rubocop/cop/rails/create_table_with_timestamps.rb +82 -0
  183. data/lib/rubocop/cop/rails/delegate.rb +20 -25
  184. data/lib/rubocop/cop/rails/delegate_allow_blank.rb +2 -2
  185. data/lib/rubocop/cop/rails/environment_comparison.rb +66 -0
  186. data/lib/rubocop/cop/rails/exit.rb +7 -0
  187. data/lib/rubocop/cop/rails/has_and_belongs_to_many.rb +7 -0
  188. data/lib/rubocop/cop/rails/has_many_or_has_one_dependent.rb +6 -2
  189. data/lib/rubocop/cop/rails/http_positional_arguments.rb +36 -41
  190. data/lib/rubocop/cop/rails/inverse_of.rb +96 -0
  191. data/lib/rubocop/cop/rails/lexically_scoped_action_filter.rb +112 -0
  192. data/lib/rubocop/cop/rails/output.rb +9 -0
  193. data/lib/rubocop/cop/rails/output_safety.rb +14 -20
  194. data/lib/rubocop/cop/rails/pluralization_grammar.rb +8 -8
  195. data/lib/rubocop/cop/rails/presence.rb +105 -0
  196. data/lib/rubocop/cop/rails/read_write_attribute.rb +10 -10
  197. data/lib/rubocop/cop/rails/redundant_receiver_in_with_options.rb +91 -0
  198. data/lib/rubocop/cop/rails/relative_date_constant.rb +9 -9
  199. data/lib/rubocop/cop/rails/reversible_migration.rb +3 -4
  200. data/lib/rubocop/cop/rails/safe_navigation.rb +5 -6
  201. data/lib/rubocop/cop/rails/save_bang.rb +2 -2
  202. data/lib/rubocop/cop/rails/uniq_before_pluck.rb +1 -1
  203. data/lib/rubocop/cop/rails/validation.rb +7 -7
  204. data/lib/rubocop/cop/security/json_load.rb +4 -2
  205. data/lib/rubocop/cop/security/marshal_load.rb +4 -2
  206. data/lib/rubocop/cop/style/alias.rb +24 -26
  207. data/lib/rubocop/cop/style/and_or.rb +34 -28
  208. data/lib/rubocop/cop/style/array_join.rb +9 -0
  209. data/lib/rubocop/cop/style/ascii_comments.rb +24 -4
  210. data/lib/rubocop/cop/style/attr.rb +14 -4
  211. data/lib/rubocop/cop/style/auto_resource_cleanup.rb +5 -3
  212. data/lib/rubocop/cop/style/bare_percent_literals.rb +31 -10
  213. data/lib/rubocop/cop/style/block_comments.rb +14 -2
  214. data/lib/rubocop/cop/style/block_delimiters.rb +80 -15
  215. data/lib/rubocop/cop/style/braces_around_hash_parameters.rb +81 -48
  216. data/lib/rubocop/cop/style/case_equality.rb +12 -0
  217. data/lib/rubocop/cop/style/character_literal.rb +10 -0
  218. data/lib/rubocop/cop/style/class_and_module_children.rb +6 -2
  219. data/lib/rubocop/cop/style/class_check.rb +28 -9
  220. data/lib/rubocop/cop/style/class_methods.rb +8 -8
  221. data/lib/rubocop/cop/style/class_vars.rb +4 -3
  222. data/lib/rubocop/cop/style/collection_methods.rb +4 -2
  223. data/lib/rubocop/cop/style/colon_method_call.rb +16 -0
  224. data/lib/rubocop/cop/style/colon_method_definition.rb +36 -0
  225. data/lib/rubocop/cop/style/command_literal.rb +70 -21
  226. data/lib/rubocop/cop/style/comment_annotation.rb +35 -10
  227. data/lib/rubocop/cop/style/commented_keyword.rb +12 -9
  228. data/lib/rubocop/cop/style/conditional_assignment.rb +10 -12
  229. data/lib/rubocop/cop/style/copyright.rb +19 -20
  230. data/lib/rubocop/cop/style/date_time.rb +2 -2
  231. data/lib/rubocop/cop/style/def_with_parentheses.rb +27 -2
  232. data/lib/rubocop/cop/style/dir.rb +2 -2
  233. data/lib/rubocop/cop/style/documentation.rb +17 -2
  234. data/lib/rubocop/cop/style/each_for_simple_loop.rb +7 -7
  235. data/lib/rubocop/cop/style/each_with_object.rb +5 -5
  236. data/lib/rubocop/cop/style/empty_block_parameter.rb +47 -0
  237. data/lib/rubocop/cop/style/empty_case_condition.rb +2 -2
  238. data/lib/rubocop/cop/style/empty_else.rb +52 -22
  239. data/lib/rubocop/cop/style/empty_lambda_parameter.rb +43 -0
  240. data/lib/rubocop/cop/style/empty_literal.rb +17 -5
  241. data/lib/rubocop/cop/style/empty_method.rb +4 -10
  242. data/lib/rubocop/cop/style/encoding.rb +2 -1
  243. data/lib/rubocop/cop/style/eval_with_location.rb +146 -0
  244. data/lib/rubocop/cop/style/even_odd.rb +4 -2
  245. data/lib/rubocop/cop/style/extend_self.rb +92 -0
  246. data/lib/rubocop/cop/style/flip_flop.rb +11 -0
  247. data/lib/rubocop/cop/style/format_string.rb +29 -2
  248. data/lib/rubocop/cop/style/format_string_token.rb +15 -14
  249. data/lib/rubocop/cop/style/frozen_string_literal_comment.rb +4 -3
  250. data/lib/rubocop/cop/style/global_vars.rb +11 -1
  251. data/lib/rubocop/cop/style/hash_syntax.rb +44 -43
  252. data/lib/rubocop/cop/style/identical_conditional_branches.rb +8 -8
  253. data/lib/rubocop/cop/style/if_inside_else.rb +10 -10
  254. data/lib/rubocop/cop/style/if_unless_modifier.rb +7 -7
  255. data/lib/rubocop/cop/style/if_unless_modifier_of_if_unless.rb +2 -2
  256. data/lib/rubocop/cop/style/if_with_semicolon.rb +9 -0
  257. data/lib/rubocop/cop/style/implicit_runtime_error.rb +5 -5
  258. data/lib/rubocop/cop/style/infinite_loop.rb +2 -2
  259. data/lib/rubocop/cop/style/lambda.rb +23 -31
  260. data/lib/rubocop/cop/style/lambda_call.rb +15 -9
  261. data/lib/rubocop/cop/style/line_end_concatenation.rb +4 -4
  262. data/lib/rubocop/cop/style/method_call_with_args_parentheses.rb +2 -2
  263. data/lib/rubocop/cop/style/method_call_without_args_parentheses.rb +2 -2
  264. data/lib/rubocop/cop/style/method_def_parentheses.rb +4 -3
  265. data/lib/rubocop/cop/style/method_missing.rb +8 -8
  266. data/lib/rubocop/cop/style/min_max.rb +12 -12
  267. data/lib/rubocop/cop/style/missing_else.rb +5 -4
  268. data/lib/rubocop/cop/style/mixin_grouping.rb +29 -19
  269. data/lib/rubocop/cop/style/mixin_usage.rb +8 -10
  270. data/lib/rubocop/cop/style/module_function.rb +17 -5
  271. data/lib/rubocop/cop/style/multiline_if_modifier.rb +4 -4
  272. data/lib/rubocop/cop/style/multiline_if_then.rb +14 -13
  273. data/lib/rubocop/cop/style/multiline_memoization.rb +8 -14
  274. data/lib/rubocop/cop/style/mutable_constant.rb +12 -12
  275. data/lib/rubocop/cop/style/negated_if.rb +22 -30
  276. data/lib/rubocop/cop/style/negated_while.rb +1 -5
  277. data/lib/rubocop/cop/style/nested_parenthesized_calls.rb +18 -17
  278. data/lib/rubocop/cop/style/next.rb +39 -11
  279. data/lib/rubocop/cop/style/nil_comparison.rb +6 -6
  280. data/lib/rubocop/cop/style/non_nil_check.rb +25 -22
  281. data/lib/rubocop/cop/style/not.rb +14 -4
  282. data/lib/rubocop/cop/style/numeric_literal_prefix.rb +6 -6
  283. data/lib/rubocop/cop/style/numeric_literals.rb +7 -7
  284. data/lib/rubocop/cop/style/numeric_predicate.rb +14 -18
  285. data/lib/rubocop/cop/style/one_line_conditional.rb +4 -4
  286. data/lib/rubocop/cop/style/option_hash.rb +11 -14
  287. data/lib/rubocop/cop/style/or_assignment.rb +2 -2
  288. data/lib/rubocop/cop/style/parallel_assignment.rb +13 -13
  289. data/lib/rubocop/cop/style/parentheses_around_condition.rb +19 -3
  290. data/lib/rubocop/cop/style/percent_literal_delimiters.rb +24 -3
  291. data/lib/rubocop/cop/style/percent_q_literals.rb +27 -6
  292. data/lib/rubocop/cop/style/perl_backrefs.rb +7 -0
  293. data/lib/rubocop/cop/style/preferred_hash_methods.rb +6 -10
  294. data/lib/rubocop/cop/style/proc.rb +8 -0
  295. data/lib/rubocop/cop/style/raise_args.rb +8 -14
  296. data/lib/rubocop/cop/style/random_with_offset.rb +160 -0
  297. data/lib/rubocop/cop/style/redundant_begin.rb +14 -4
  298. data/lib/rubocop/cop/style/redundant_conditional.rb +9 -8
  299. data/lib/rubocop/cop/style/redundant_return.rb +5 -4
  300. data/lib/rubocop/cop/style/redundant_self.rb +25 -26
  301. data/lib/rubocop/cop/style/regexp_literal.rb +74 -21
  302. data/lib/rubocop/cop/style/rescue_modifier.rb +11 -0
  303. data/lib/rubocop/cop/style/rescue_standard_error.rb +122 -0
  304. data/lib/rubocop/cop/style/return_nil.rb +4 -8
  305. data/lib/rubocop/cop/style/safe_navigation.rb +20 -2
  306. data/lib/rubocop/cop/style/self_assignment.rb +13 -13
  307. data/lib/rubocop/cop/style/semicolon.rb +18 -8
  308. data/lib/rubocop/cop/style/send.rb +9 -0
  309. data/lib/rubocop/cop/style/signal_exception.rb +100 -0
  310. data/lib/rubocop/cop/style/single_line_block_params.rb +2 -2
  311. data/lib/rubocop/cop/style/single_line_methods.rb +20 -8
  312. data/lib/rubocop/cop/style/special_global_vars.rb +16 -11
  313. data/lib/rubocop/cop/style/stabby_lambda_parentheses.rb +7 -12
  314. data/lib/rubocop/cop/style/stderr_puts.rb +5 -3
  315. data/lib/rubocop/cop/style/string_hash_keys.rb +36 -0
  316. data/lib/rubocop/cop/style/string_literals.rb +22 -0
  317. data/lib/rubocop/cop/style/string_literals_in_interpolation.rb +2 -8
  318. data/lib/rubocop/cop/style/string_methods.rb +18 -7
  319. data/lib/rubocop/cop/style/symbol_array.rb +5 -9
  320. data/lib/rubocop/cop/style/symbol_proc.rb +5 -4
  321. data/lib/rubocop/cop/style/ternary_parentheses.rb +31 -40
  322. data/lib/rubocop/cop/style/trailing_body_on_method_definition.rb +101 -0
  323. data/lib/rubocop/cop/style/trailing_comma_in_arguments.rb +20 -6
  324. data/lib/rubocop/cop/style/trailing_comma_in_literal.rb +22 -7
  325. data/lib/rubocop/cop/style/trailing_method_end_statement.rb +95 -0
  326. data/lib/rubocop/cop/style/trailing_underscore_variable.rb +8 -4
  327. data/lib/rubocop/cop/style/trivial_accessors.rb +32 -10
  328. data/lib/rubocop/cop/style/unless_else.rb +15 -0
  329. data/lib/rubocop/cop/style/unneeded_capital_w.rb +17 -7
  330. data/lib/rubocop/cop/style/unneeded_interpolation.rb +14 -18
  331. data/lib/rubocop/cop/style/unneeded_percent_q.rb +13 -12
  332. data/lib/rubocop/cop/style/variable_interpolation.rb +20 -9
  333. data/lib/rubocop/cop/style/when_then.rb +13 -0
  334. data/lib/rubocop/cop/style/while_until_do.rb +26 -4
  335. data/lib/rubocop/cop/style/while_until_modifier.rb +25 -6
  336. data/lib/rubocop/cop/style/word_array.rb +4 -8
  337. data/lib/rubocop/cop/style/yoda_condition.rb +10 -16
  338. data/lib/rubocop/cop/style/zero_length_predicate.rb +23 -15
  339. data/lib/rubocop/cop/util.rb +42 -33
  340. data/lib/rubocop/cop/variable_force.rb +1 -1
  341. data/lib/rubocop/cop/variable_force/assignment.rb +4 -2
  342. data/lib/rubocop/cop/variable_force/scope.rb +1 -5
  343. data/lib/rubocop/cop/variable_force/variable.rb +1 -1
  344. data/lib/rubocop/formatter/disabled_config_formatter.rb +2 -3
  345. data/lib/rubocop/formatter/formatter_set.rb +2 -1
  346. data/lib/rubocop/formatter/json_formatter.rb +9 -3
  347. data/lib/rubocop/formatter/quiet_formatter.rb +13 -0
  348. data/lib/rubocop/node_pattern.rb +2 -3
  349. data/lib/rubocop/options.rb +8 -8
  350. data/lib/rubocop/path_util.rb +15 -3
  351. data/lib/rubocop/processed_source.rb +5 -2
  352. data/lib/rubocop/rspec/shared_contexts.rb +4 -0
  353. data/lib/rubocop/runner.rb +1 -1
  354. data/lib/rubocop/token.rb +74 -0
  355. data/lib/rubocop/version.rb +1 -1
  356. metadata +37 -10
  357. data/lib/rubocop/cop/internal_affairs/deprecated_positional_arguments.rb +0 -81
  358. data/lib/rubocop/cop/layout/space_inside_brackets.rb +0 -20
  359. data/lib/rubocop/cop/lint/rescue_without_error_class.rb +0 -41
  360. data/lib/rubocop/cop/mixin/space_inside.rb +0 -76
@@ -32,14 +32,14 @@ module RuboCop
32
32
  add_offense(offending_node)
33
33
  end
34
34
 
35
- private
36
-
37
35
  def autocorrect(pair_node)
38
36
  lambda do |corrector|
39
37
  corrector.replace(pair_node.key.source_range, 'allow_nil')
40
38
  end
41
39
  end
42
40
 
41
+ private
42
+
43
43
  def allow_blank_option(node)
44
44
  delegate_options(node) do |hash|
45
45
  hash.pairs.find { |opt| allow_blank_option?(opt) }
@@ -0,0 +1,66 @@
1
+ # frozen_string_literal: true
2
+
3
+ module RuboCop
4
+ module Cop
5
+ module Rails
6
+ # This cop checks that Rails.env is compared using `.production?`-like
7
+ # methods instead of equality against a string or symbol.
8
+ #
9
+ # @example
10
+ # # bad
11
+ # Rails.env == 'production'
12
+ #
13
+ # # bad, always returns false
14
+ # Rails.env == :test
15
+ #
16
+ # # good
17
+ # Rails.env.production?
18
+ class EnvironmentComparison < Cop
19
+ MSG = 'Favor `Rails.env.%s?` over `Rails.env == %s`.'.freeze
20
+
21
+ SYM_MSG = 'Do not compare `Rails.env` with a symbol, it will always ' \
22
+ 'evaluate to `false`.'.freeze
23
+
24
+ def_node_matcher :environment_str_comparison?, <<-PATTERN
25
+ (send
26
+ (send (const {nil? cbase} :Rails) :env)
27
+ :==
28
+ $str
29
+ )
30
+ PATTERN
31
+
32
+ def_node_matcher :environment_sym_comparison?, <<-PATTERN
33
+ (send
34
+ (send (const {nil? cbase} :Rails) :env)
35
+ :==
36
+ $sym
37
+ )
38
+ PATTERN
39
+
40
+ def on_send(node)
41
+ environment_str_comparison?(node) do |env_node|
42
+ env, = *env_node
43
+ add_offense(node, message: format(MSG, env, env_node.source))
44
+ end
45
+ environment_sym_comparison?(node) do |_|
46
+ add_offense(node, message: SYM_MSG)
47
+ end
48
+ end
49
+
50
+ def autocorrect(node)
51
+ lambda do |corrector|
52
+ corrector.replace(node.source_range, replacement(node))
53
+ end
54
+ end
55
+
56
+ private
57
+
58
+ def replacement(node)
59
+ receiver, _, compared = *node
60
+ environment, = *compared
61
+ "#{receiver.source}.#{environment}?"
62
+ end
63
+ end
64
+ end
65
+ end
66
+ end
@@ -16,6 +16,13 @@ module RuboCop
16
16
  # - Usage in application code outside of the web process could result in
17
17
  # the program exiting, which could result in the code failing to run and
18
18
  # do its job.
19
+ #
20
+ # @example
21
+ # # bad
22
+ # exit(0)
23
+ #
24
+ # # good
25
+ # raise 'a bad error has happened'
19
26
  class Exit < Cop
20
27
  include ConfigurableEnforcedStyle
21
28
 
@@ -4,6 +4,13 @@ module RuboCop
4
4
  module Cop
5
5
  module Rails
6
6
  # This cop checks for the use of the has_and_belongs_to_many macro.
7
+ #
8
+ # @example
9
+ # # bad
10
+ # # has_and_belongs_to_many :ingredients
11
+ #
12
+ # # good
13
+ # # has_many :ingredients, through: :recipe_ingredients
7
14
  class HasAndBelongsToMany < Cop
8
15
  MSG = 'Prefer `has_many :through` to `has_and_belongs_to_many`.'.freeze
9
16
 
@@ -49,8 +49,12 @@ module RuboCop
49
49
  def on_send(node)
50
50
  if !association_without_options?(node)
51
51
  return if valid_options?(association_with_options?(node))
52
- elsif with_options_block(node.parent)
53
- return if valid_options?(with_options_block(node.parent))
52
+ else
53
+ n = node.parent.begin_type? ? node.parent.parent : node.parent
54
+
55
+ if with_options_block(n)
56
+ return if valid_options?(with_options_block(n))
57
+ end
54
58
  end
55
59
 
56
60
  add_offense(node, location: :selector)
@@ -29,52 +29,16 @@ module RuboCop
29
29
  minimum_target_rails_version 5.0
30
30
 
31
31
  def_node_matcher :http_request?, <<-PATTERN
32
- (send nil? {#{HTTP_METHODS.map(&:inspect).join(' ')}} !nil? $_data ...)
32
+ (send nil? {#{HTTP_METHODS.map(&:inspect).join(' ')}} !nil? $_ ...)
33
33
  PATTERN
34
34
 
35
35
  def on_send(node)
36
- data = http_request?(node)
37
- # if the data is nil then we don't need to add keyword arguments
38
- # because there is no data to put in params or headers, so skip
39
- return if data.nil?
40
- return unless needs_conversion?(data)
36
+ http_request?(node) do |data|
37
+ return unless needs_conversion?(data)
41
38
 
42
- add_offense(node, location: :selector,
43
- message: format(MSG, node.method_name))
44
- end
45
-
46
- # @return [Boolean] true if the line needs to be converted
47
- def needs_conversion?(data)
48
- return true unless data.hash_type?
49
- children = data.child_nodes
50
-
51
- value = children.find do |d|
52
- special_keyword_arg?(d.children.first) ||
53
- (format_arg?(d.children.first) && children.size == 1)
39
+ add_offense(node, location: :selector,
40
+ message: format(MSG, node.method_name))
54
41
  end
55
-
56
- value.nil?
57
- end
58
-
59
- def special_keyword_arg?(node)
60
- KEYWORD_ARGS.include?(node.children.first) if node.sym_type?
61
- end
62
-
63
- def format_arg?(node)
64
- node.children.first == :format if node.sym_type?
65
- end
66
-
67
- def convert_hash_data(data, type)
68
- # empty hash or no hash return empty string
69
- return '' if data.nil? || data.children.empty?
70
- hash_data = if data.hash_type?
71
- format('{ %s }', data.pairs.map(&:source).join(', '))
72
- else
73
- # user supplies an object,
74
- # no need to surround with braces
75
- data.source
76
- end
77
- format(', %s: %s', type, hash_data)
78
42
  end
79
43
 
80
44
  # given a pre Rails 5 method: get :new, user_id: @user.id, {}
@@ -101,6 +65,37 @@ module RuboCop
101
65
  params, headers)
102
66
  ->(corrector) { corrector.replace(code_to_replace, new_code) }
103
67
  end
68
+
69
+ def needs_conversion?(data)
70
+ return true unless data.hash_type?
71
+
72
+ data.each_pair.none? do |pair|
73
+ special_keyword_arg?(pair.key) ||
74
+ format_arg?(pair.key) && data.pairs.one?
75
+ end
76
+ end
77
+
78
+ def special_keyword_arg?(node)
79
+ node.sym_type? && KEYWORD_ARGS.include?(node.value)
80
+ end
81
+
82
+ def format_arg?(node)
83
+ node.sym_type? && node.value == :format
84
+ end
85
+
86
+ def convert_hash_data(data, type)
87
+ return '' if data.hash_type? && data.empty?
88
+
89
+ hash_data = if data.hash_type?
90
+ format('{ %s }', data.pairs.map(&:source).join(', '))
91
+ else
92
+ # user supplies an object,
93
+ # no need to surround with braces
94
+ data.source
95
+ end
96
+
97
+ format(', %s: %s', type, hash_data)
98
+ end
104
99
  end
105
100
  end
106
101
  end
@@ -0,0 +1,96 @@
1
+ # frozen_string_literal: true
2
+
3
+ module RuboCop
4
+ module Cop
5
+ module Rails
6
+ # This cop looks for has_(one|many) and belongs_to associations where
7
+ # ActiveRecord can't automatically determine the inverse association
8
+ # because of a scope or the options used. This can result in unnecessary
9
+ # queries in some circumstances. `:inverse_of` must be manually specified
10
+ # for associations to work in both ways, or set to `false` to opt-out.
11
+ #
12
+ # @example
13
+ # # bad
14
+ # class Blog < ApplicationRecord
15
+ # has_many :recent_posts, -> { order(published_at: :desc) }
16
+ # end
17
+ #
18
+ # # good
19
+ # class Blog < ApplicationRecord
20
+ # has_many(:recent_posts,
21
+ # -> { order(published_at: :desc) },
22
+ # inverse_of: :blog
23
+ # )
24
+ # end
25
+ #
26
+ # @see http://guides.rubyonrails.org/association_basics.html#bi-directional-associations
27
+ # @see http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#module-ActiveRecord::Associations::ClassMethods-label-Setting+Inverses
28
+ class InverseOf < Cop
29
+ extend TargetRailsVersion
30
+
31
+ minimum_target_rails_version 4.1
32
+
33
+ MSG = 'Specify an `:inverse_of` option.'.freeze
34
+
35
+ def_node_matcher :association_arguments, <<-PATTERN
36
+ (send nil? {:has_many :has_one :belongs_to} _ $...)
37
+ PATTERN
38
+
39
+ def_node_matcher :options_from_argument, <<-PATTERN
40
+ (hash $...)
41
+ PATTERN
42
+
43
+ def_node_matcher :conditions_option?, <<-PATTERN
44
+ (pair (sym :conditions) !nil)
45
+ PATTERN
46
+
47
+ def_node_matcher :through_option?, <<-PATTERN
48
+ (pair (sym :through) !nil)
49
+ PATTERN
50
+
51
+ def_node_matcher :polymorphic_option?, <<-PATTERN
52
+ (pair (sym :polymorphic) !nil)
53
+ PATTERN
54
+
55
+ def_node_matcher :foreign_key_option?, <<-PATTERN
56
+ (pair (sym :foreign_key) !nil)
57
+ PATTERN
58
+
59
+ def_node_matcher :inverse_of_option?, <<-PATTERN
60
+ (pair (sym :inverse_of) !nil)
61
+ PATTERN
62
+
63
+ def on_send(node)
64
+ arguments = association_arguments(node)
65
+ return unless arguments
66
+
67
+ options = arguments.flat_map { |arg| options_from_argument(arg) }
68
+ return unless scope?(arguments) ||
69
+ options_requiring_inverse_of?(options)
70
+
71
+ return if options_contain_inverse_of?(options)
72
+ add_offense(node, location: :selector)
73
+ end
74
+
75
+ def scope?(arguments)
76
+ arguments.any?(&:block_type?)
77
+ end
78
+
79
+ def options_requiring_inverse_of?(options)
80
+ required = options.any? do |opt|
81
+ conditions_option?(opt) ||
82
+ through_option?(opt) ||
83
+ foreign_key_option?(opt)
84
+ end
85
+
86
+ return required if target_rails_version >= 5.2
87
+ required || options.any? { |opt| polymorphic_option?(opt) }
88
+ end
89
+
90
+ def options_contain_inverse_of?(options)
91
+ options.any? { |opt| inverse_of_option?(opt) }
92
+ end
93
+ end
94
+ end
95
+ end
96
+ end
@@ -0,0 +1,112 @@
1
+ # frozen_string_literal: true
2
+
3
+ module RuboCop
4
+ module Cop
5
+ module Rails
6
+ # This cop checks that methods specified in the filter's `only`
7
+ # or `except` options are explicitly defined in the controller.
8
+ #
9
+ # You can specify methods of superclass or methods added by mixins
10
+ # on the filter, but these confuse developers. If you specify methods
11
+ # where are defined on another controller, you should define the filter
12
+ # in that controller.
13
+ #
14
+ # @example
15
+ # # bad
16
+ # class LoginController < ApplicationController
17
+ # before_action :require_login, only: %i[index settings logout]
18
+ #
19
+ # def index
20
+ # end
21
+ # end
22
+ #
23
+ # # good
24
+ # class LoginController < ApplicationController
25
+ # before_action :require_login, only: %i[index settings logout]
26
+ #
27
+ # def index
28
+ # end
29
+ #
30
+ # def settings
31
+ # end
32
+ #
33
+ # def logout
34
+ # end
35
+ # end
36
+ class LexicallyScopedActionFilter < Cop
37
+ MSG = '%s not explicitly defined on the controller.'.freeze
38
+
39
+ FILTERS = %w[
40
+ :after_action
41
+ :append_after_action
42
+ :append_around_action
43
+ :append_before_action
44
+ :around_action
45
+ :before_action
46
+ :prepend_after_action
47
+ :prepend_around_action
48
+ :prepend_before_action
49
+ :skip_after_action
50
+ :skip_around_action
51
+ :skip_before_action
52
+ :skip_action_callback
53
+ ].freeze
54
+
55
+ def_node_matcher :only_or_except_filter_methods, <<-PATTERN
56
+ (send
57
+ nil?
58
+ {#{FILTERS.join(' ')}}
59
+ _
60
+ (hash
61
+ (pair
62
+ (sym {:only :except})
63
+ $_)))
64
+ PATTERN
65
+
66
+ def on_send(node)
67
+ methods_node = only_or_except_filter_methods(node)
68
+ return unless methods_node
69
+
70
+ defined_methods = node.parent.each_child_node(:def).map(&:method_name)
71
+ methods = array_values(methods_node).reject do |method|
72
+ defined_methods.include?(method)
73
+ end
74
+
75
+ add_offense(node, message: message(methods)) unless methods.empty?
76
+ end
77
+
78
+ private
79
+
80
+ # @param node [RuboCop::AST::Node]
81
+ # @return [Array<Symbol>]
82
+ def array_values(node) # rubocop:disable Metrics/MethodLength
83
+ case node.type
84
+ when :str
85
+ [node.str_content.to_sym]
86
+ when :sym
87
+ [node.value]
88
+ when :array
89
+ node.values.map do |v|
90
+ case v.type
91
+ when :str
92
+ v.str_content.to_sym
93
+ when :sym
94
+ v.value
95
+ end
96
+ end.compact
97
+ else
98
+ []
99
+ end
100
+ end
101
+
102
+ def message(methods)
103
+ if methods.size == 1
104
+ format(MSG, "`#{methods[0]}` is")
105
+ else
106
+ format(MSG, "`#{methods.join('`, `')}` are")
107
+ end
108
+ end
109
+ end
110
+ end
111
+ end
112
+ end
@@ -4,6 +4,15 @@ module RuboCop
4
4
  module Cop
5
5
  module Rails
6
6
  # This cop checks for the use of output calls like puts and print
7
+ #
8
+ # @example
9
+ # # bad
10
+ # puts 'A debug message'
11
+ # pp 'A debug message'
12
+ # print 'A debug message'
13
+ #
14
+ # # good
15
+ # Rails.logger.debug 'A debug message'
7
16
  class Output < Cop
8
17
  MSG = 'Do not write to stdout. ' \
9
18
  "Use Rails's logger if you want to log.".freeze