rubocop 0.18.1 → 0.19.0
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of rubocop might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/.travis.yml +1 -1
- data/CHANGELOG.md +65 -1
- data/Gemfile +0 -3
- data/LICENSE.txt +1 -1
- data/README.md +179 -77
- data/Rakefile +8 -0
- data/config/default.yml +151 -14
- data/config/enabled.yml +65 -8
- data/lib/rubocop.rb +19 -4
- data/lib/rubocop/cli.rb +2 -1
- data/lib/rubocop/comment_config.rb +97 -0
- data/lib/rubocop/config.rb +1 -1
- data/lib/rubocop/config_loader.rb +3 -2
- data/lib/rubocop/config_store.rb +6 -2
- data/lib/rubocop/cop/commissioner.rb +2 -2
- data/lib/rubocop/cop/cop.rb +22 -26
- data/lib/rubocop/cop/lint/ambiguous_operator.rb +2 -2
- data/lib/rubocop/cop/lint/ambiguous_regexp_literal.rb +1 -1
- data/lib/rubocop/cop/lint/assignment_in_condition.rb +1 -1
- data/lib/rubocop/cop/lint/block_alignment.rb +6 -6
- data/lib/rubocop/cop/lint/condition_position.rb +1 -1
- data/lib/rubocop/cop/lint/debugger.rb +1 -1
- data/lib/rubocop/cop/lint/deprecated_class_methods.rb +60 -0
- data/lib/rubocop/cop/lint/else_layout.rb +1 -1
- data/lib/rubocop/cop/lint/empty_ensure.rb +1 -1
- data/lib/rubocop/cop/lint/end_alignment.rb +14 -12
- data/lib/rubocop/cop/lint/end_in_method.rb +1 -1
- data/lib/rubocop/cop/lint/ensure_return.rb +1 -1
- data/lib/rubocop/cop/lint/eval.rb +1 -1
- data/lib/rubocop/cop/lint/handle_exceptions.rb +1 -1
- data/lib/rubocop/cop/lint/literal_in_condition.rb +13 -2
- data/lib/rubocop/cop/lint/literal_in_interpolation.rb +28 -0
- data/lib/rubocop/cop/lint/loop.rb +4 -4
- data/lib/rubocop/cop/lint/parentheses_as_grouped_expression.rb +1 -1
- data/lib/rubocop/cop/lint/require_parentheses.rb +6 -6
- data/lib/rubocop/cop/lint/rescue_exception.rb +1 -1
- data/lib/rubocop/cop/lint/shadowing_outer_local_variable.rb +2 -2
- data/lib/rubocop/cop/lint/string_conversion_in_interpolation.rb +30 -0
- data/lib/rubocop/cop/lint/syntax.rb +5 -5
- data/lib/rubocop/cop/lint/unreachable_code.rb +1 -1
- data/lib/rubocop/cop/lint/useless_assignment.rb +5 -5
- data/lib/rubocop/cop/lint/useless_comparison.rb +1 -1
- data/lib/rubocop/cop/lint/useless_setter_call.rb +1 -1
- data/lib/rubocop/cop/lint/void.rb +5 -5
- data/lib/rubocop/cop/mixin/autocorrect_alignment.rb +8 -5
- data/lib/rubocop/cop/mixin/code_length.rb +2 -2
- data/lib/rubocop/cop/mixin/configurable_enforced_style.rb +7 -7
- data/lib/rubocop/cop/mixin/configurable_max.rb +1 -1
- data/lib/rubocop/cop/mixin/configurable_naming.rb +1 -1
- data/lib/rubocop/cop/mixin/if_then_else.rb +1 -1
- data/lib/rubocop/cop/mixin/negative_conditional.rb +1 -1
- data/lib/rubocop/cop/mixin/parser_diagnostic.rb +2 -2
- data/lib/rubocop/cop/mixin/space_after_punctuation.rb +3 -2
- data/lib/rubocop/cop/mixin/space_inside.rb +1 -1
- data/lib/rubocop/cop/mixin/statement_modifier.rb +1 -0
- data/lib/rubocop/cop/mixin/string_help.rb +4 -4
- data/lib/rubocop/cop/{offence.rb → offense.rb} +7 -7
- data/lib/rubocop/cop/rails/action_filter.rb +73 -0
- data/lib/rubocop/cop/rails/default_scope.rb +1 -1
- data/lib/rubocop/cop/rails/has_and_belongs_to_many.rb +1 -1
- data/lib/rubocop/cop/rails/output.rb +1 -10
- data/lib/rubocop/cop/rails/read_attribute.rb +1 -1
- data/lib/rubocop/cop/rails/scope_args.rb +33 -0
- data/lib/rubocop/cop/rails/validation.rb +1 -1
- data/lib/rubocop/cop/style/access_modifier_indentation.rb +1 -1
- data/lib/rubocop/cop/style/accessor_method_name.rb +2 -2
- data/lib/rubocop/cop/style/alias.rb +2 -2
- data/lib/rubocop/cop/style/align_array.rb +1 -1
- data/lib/rubocop/cop/style/align_hash.rb +29 -2
- data/lib/rubocop/cop/style/align_parameters.rb +16 -2
- data/lib/rubocop/cop/style/and_or.rb +2 -2
- data/lib/rubocop/cop/style/ascii_comments.rb +1 -1
- data/lib/rubocop/cop/style/ascii_identifiers.rb +1 -1
- data/lib/rubocop/cop/style/attr.rb +1 -1
- data/lib/rubocop/cop/style/begin_block.rb +1 -1
- data/lib/rubocop/cop/style/block_comments.rb +1 -1
- data/lib/rubocop/cop/style/block_nesting.rb +1 -1
- data/lib/rubocop/cop/style/blocks.rb +2 -2
- data/lib/rubocop/cop/style/braces_around_hash_parameters.rb +2 -2
- data/lib/rubocop/cop/style/case_equality.rb +1 -1
- data/lib/rubocop/cop/style/case_indentation.rb +2 -2
- data/lib/rubocop/cop/style/character_literal.rb +2 -2
- data/lib/rubocop/cop/style/class_and_module_camel_case.rb +1 -1
- data/lib/rubocop/cop/style/class_and_module_children.rb +69 -0
- data/lib/rubocop/cop/style/class_methods.rb +1 -1
- data/lib/rubocop/cop/style/class_vars.rb +3 -3
- data/lib/rubocop/cop/style/collection_methods.rb +4 -4
- data/lib/rubocop/cop/style/colon_method_call.rb +1 -1
- data/lib/rubocop/cop/style/comment_annotation.rb +1 -1
- data/lib/rubocop/cop/style/constant_name.rb +1 -1
- data/lib/rubocop/cop/style/cyclomatic_complexity.rb +2 -2
- data/lib/rubocop/cop/style/def_parentheses.rb +2 -2
- data/lib/rubocop/cop/style/documentation.rb +1 -1
- data/lib/rubocop/cop/style/dot_position.rb +10 -11
- data/lib/rubocop/cop/style/double_negation.rb +40 -0
- data/lib/rubocop/cop/style/empty_line_between_defs.rb +8 -1
- data/lib/rubocop/cop/style/empty_lines.rb +3 -2
- data/lib/rubocop/cop/style/empty_lines_around_access_modifier.rb +1 -1
- data/lib/rubocop/cop/style/empty_lines_around_body.rb +1 -1
- data/lib/rubocop/cop/style/empty_literal.rb +3 -3
- data/lib/rubocop/cop/style/encoding.rb +1 -1
- data/lib/rubocop/cop/style/end_block.rb +1 -1
- data/lib/rubocop/cop/style/end_of_line.rb +2 -2
- data/lib/rubocop/cop/style/even_odd.rb +2 -2
- data/lib/rubocop/cop/style/favor_join.rb +1 -1
- data/lib/rubocop/cop/style/file_name.rb +29 -0
- data/lib/rubocop/cop/style/final_newline.rb +1 -1
- data/lib/rubocop/cop/style/flip_flop.rb +2 -2
- data/lib/rubocop/cop/style/for.rb +2 -2
- data/lib/rubocop/cop/style/format_string.rb +66 -0
- data/lib/rubocop/cop/style/global_vars.rb +2 -2
- data/lib/rubocop/cop/style/hash_methods.rb +1 -1
- data/lib/rubocop/cop/style/hash_syntax.rb +3 -3
- data/lib/rubocop/cop/style/if_unless_modifier.rb +2 -1
- data/lib/rubocop/cop/style/indent_array.rb +41 -0
- data/lib/rubocop/cop/style/indent_hash.rb +119 -0
- data/lib/rubocop/cop/style/indentation_consistency.rb +3 -11
- data/lib/rubocop/cop/style/indentation_width.rb +44 -23
- data/lib/rubocop/cop/style/lambda.rb +2 -2
- data/lib/rubocop/cop/style/lambda_call.rb +3 -3
- data/lib/rubocop/cop/style/leading_comment_space.rb +1 -1
- data/lib/rubocop/cop/style/line_end_concatenation.rb +7 -3
- data/lib/rubocop/cop/style/line_length.rb +2 -2
- data/lib/rubocop/cop/style/method_call_parentheses.rb +2 -2
- data/lib/rubocop/cop/style/method_called_on_do_end_block.rb +2 -2
- data/lib/rubocop/cop/style/method_def_parentheses.rb +9 -7
- data/lib/rubocop/cop/style/module_function.rb +1 -1
- data/lib/rubocop/cop/style/multiline_block_chain.rb +1 -1
- data/lib/rubocop/cop/style/multiline_ternary_operator.rb +1 -1
- data/lib/rubocop/cop/style/nested_ternary_operator.rb +2 -2
- data/lib/rubocop/cop/style/nil_comparison.rb +1 -1
- data/lib/rubocop/cop/style/not.rb +1 -1
- data/lib/rubocop/cop/style/numeric_literals.rb +5 -5
- data/lib/rubocop/cop/style/op_method.rb +2 -2
- data/lib/rubocop/cop/style/parameter_lists.rb +1 -1
- data/lib/rubocop/cop/style/parentheses_around_condition.rb +5 -4
- data/lib/rubocop/cop/style/percent_literal_delimiters.rb +143 -0
- data/lib/rubocop/cop/style/perl_backrefs.rb +1 -1
- data/lib/rubocop/cop/style/predicate_name.rb +1 -1
- data/lib/rubocop/cop/style/proc.rb +1 -1
- data/lib/rubocop/cop/style/raise_args.rb +2 -2
- data/lib/rubocop/cop/style/redundant_begin.rb +1 -1
- data/lib/rubocop/cop/style/redundant_exception.rb +1 -1
- data/lib/rubocop/cop/style/redundant_return.rb +1 -1
- data/lib/rubocop/cop/style/redundant_self.rb +2 -2
- data/lib/rubocop/cop/style/regexp_literal.rb +45 -19
- data/lib/rubocop/cop/style/rescue_modifier.rb +1 -1
- data/lib/rubocop/cop/style/self_assignment.rb +73 -0
- data/lib/rubocop/cop/style/semicolon.rb +12 -8
- data/lib/rubocop/cop/style/signal_exception.rb +2 -2
- data/lib/rubocop/cop/style/single_line_block_params.rb +1 -1
- data/lib/rubocop/cop/style/single_line_methods.rb +38 -1
- data/lib/rubocop/cop/style/space_after_colon.rb +3 -3
- data/lib/rubocop/cop/style/space_after_comma.rb +1 -1
- data/lib/rubocop/cop/style/space_after_control_keyword.rb +1 -1
- data/lib/rubocop/cop/style/space_after_method_name.rb +11 -3
- data/lib/rubocop/cop/style/space_after_not.rb +1 -1
- data/lib/rubocop/cop/style/space_after_semicolon.rb +1 -1
- data/lib/rubocop/cop/style/space_around_equals_in_parameter_default.rb +41 -6
- data/lib/rubocop/cop/style/space_around_operators.rb +4 -4
- data/lib/rubocop/cop/style/space_before_block_braces.rb +63 -0
- data/lib/rubocop/cop/style/space_before_modifier_keyword.rb +1 -1
- data/lib/rubocop/cop/style/{space_around_block_braces.rb → space_inside_block_braces.rb} +15 -24
- data/lib/rubocop/cop/style/space_inside_hash_literal_braces.rb +14 -7
- data/lib/rubocop/cop/style/special_global_vars.rb +1 -1
- data/lib/rubocop/cop/style/string_literals.rb +1 -1
- data/lib/rubocop/cop/style/symbol_array.rb +1 -1
- data/lib/rubocop/cop/style/tab.rb +1 -1
- data/lib/rubocop/cop/style/trailing_blank_lines.rb +2 -2
- data/lib/rubocop/cop/style/trailing_comma.rb +4 -4
- data/lib/rubocop/cop/style/trailing_whitespace.rb +1 -1
- data/lib/rubocop/cop/style/trivial_accessors.rb +2 -2
- data/lib/rubocop/cop/style/unless_else.rb +1 -1
- data/lib/rubocop/cop/style/variable_interpolation.rb +1 -1
- data/lib/rubocop/cop/style/when_then.rb +1 -1
- data/lib/rubocop/cop/style/while_until_do.rb +1 -1
- data/lib/rubocop/cop/style/while_until_modifier.rb +2 -1
- data/lib/rubocop/cop/style/word_array.rb +2 -2
- data/lib/rubocop/cop/team.rb +24 -7
- data/lib/rubocop/cop/util.rb +3 -3
- data/lib/rubocop/cop/variable_inspector.rb +25 -13
- data/lib/rubocop/cop/variable_inspector/assignment.rb +1 -1
- data/lib/rubocop/cop/variable_inspector/reference.rb +1 -1
- data/lib/rubocop/cop/variable_inspector/scope.rb +1 -1
- data/lib/rubocop/cop/variable_inspector/variable.rb +2 -2
- data/lib/rubocop/cop/variable_inspector/variable_table.rb +1 -1
- data/lib/rubocop/file_inspector.rb +12 -13
- data/lib/rubocop/formatter/base_formatter.rb +4 -4
- data/lib/rubocop/formatter/clang_style_formatter.rb +2 -2
- data/lib/rubocop/formatter/disabled_config_formatter.rb +12 -12
- data/lib/rubocop/formatter/emacs_style_formatter.rb +3 -2
- data/lib/rubocop/formatter/file_list_formatter.rb +4 -4
- data/lib/rubocop/formatter/formatter_set.rb +2 -1
- data/lib/rubocop/formatter/fuubar_style_formatter.rb +76 -0
- data/lib/rubocop/formatter/json_formatter.rb +17 -16
- data/lib/rubocop/formatter/offense_count_formatter.rb +54 -0
- data/lib/rubocop/formatter/progress_formatter.rb +16 -16
- data/lib/rubocop/formatter/simple_text_formatter.rb +25 -25
- data/lib/rubocop/options.rb +8 -7
- data/lib/rubocop/processed_source.rb +3 -2
- data/lib/rubocop/source_parser.rb +1 -59
- data/lib/rubocop/version.rb +3 -3
- data/relnotes/0.19.0.md +70 -0
- data/rubocop-todo.yml +6 -6
- data/rubocop.gemspec +2 -1
- data/spec/rubocop/cli_spec.rb +431 -268
- data/spec/rubocop/comment_config_spec.rb +103 -0
- data/spec/rubocop/config_loader_spec.rb +4 -4
- data/spec/rubocop/cop/commissioner_spec.rb +7 -7
- data/spec/rubocop/cop/cop_spec.rb +17 -17
- data/spec/rubocop/cop/lint/ambiguous_operator_spec.rb +16 -16
- data/spec/rubocop/cop/lint/ambiguous_regexp_literal_spec.rb +5 -5
- data/spec/rubocop/cop/lint/assignment_in_condition_spec.rb +19 -19
- data/spec/rubocop/cop/lint/block_alignment_spec.rb +52 -52
- data/spec/rubocop/cop/lint/condition_position_spec.rb +7 -7
- data/spec/rubocop/cop/lint/debugger_spec.rb +10 -10
- data/spec/rubocop/cop/lint/deprecated_class_methods_spec.rb +33 -0
- data/spec/rubocop/cop/lint/else_layout_spec.rb +7 -7
- data/spec/rubocop/cop/lint/empty_ensure_spec.rb +4 -4
- data/spec/rubocop/cop/lint/end_alignment_spec.rb +33 -34
- data/spec/rubocop/cop/lint/end_in_method_spec.rb +5 -5
- data/spec/rubocop/cop/lint/ensure_return_spec.rb +4 -4
- data/spec/rubocop/cop/lint/eval_spec.rb +8 -8
- data/spec/rubocop/cop/lint/handle_exceptions_spec.rb +4 -4
- data/spec/rubocop/cop/lint/invalid_character_literal_spec.rb +3 -3
- data/spec/rubocop/cop/lint/literal_in_condition_spec.rb +65 -10
- data/spec/rubocop/cop/lint/literal_in_interpolation_spec.rb +21 -0
- data/spec/rubocop/cop/lint/loop_spec.rb +6 -6
- data/spec/rubocop/cop/lint/parentheses_as_grouped_expression_spec.rb +14 -14
- data/spec/rubocop/cop/lint/require_parentheses_spec.rb +15 -15
- data/spec/rubocop/cop/lint/rescue_exception_spec.rb +21 -21
- data/spec/rubocop/cop/lint/shadowing_outer_local_variable_spec.rb +16 -16
- data/spec/rubocop/cop/lint/string_conversion_in_interpolation_spec.rb +27 -0
- data/spec/rubocop/cop/lint/syntax_spec.rb +11 -11
- data/spec/rubocop/cop/lint/unreachable_code_spec.rb +8 -8
- data/spec/rubocop/cop/lint/useless_assignment_spec.rb +207 -169
- data/spec/rubocop/cop/lint/useless_comparison_spec.rb +5 -5
- data/spec/rubocop/cop/lint/useless_else_without_rescue_spec.rb +4 -4
- data/spec/rubocop/cop/lint/useless_setter_call_spec.rb +17 -17
- data/spec/rubocop/cop/lint/void_spec.rb +8 -8
- data/spec/rubocop/cop/{offence_spec.rb → offense_spec.rb} +17 -17
- data/spec/rubocop/cop/rails/action_filter_spec.rb +69 -0
- data/spec/rubocop/cop/rails/default_scope_spec.rb +9 -9
- data/spec/rubocop/cop/rails/has_and_belongs_to_many_spec.rb +2 -2
- data/spec/rubocop/cop/rails/output_spec.rb +8 -18
- data/spec/rubocop/cop/rails/read_attribute_spec.rb +2 -2
- data/spec/rubocop/cop/rails/scope_args_spec.rb +25 -0
- data/spec/rubocop/cop/rails/validation_spec.rb +3 -3
- data/spec/rubocop/cop/style/access_modifier_indentation_spec.rb +38 -38
- data/spec/rubocop/cop/style/accessor_method_name_spec.rb +14 -14
- data/spec/rubocop/cop/style/alias_spec.rb +11 -11
- data/spec/rubocop/cop/style/align_array_spec.rb +23 -7
- data/spec/rubocop/cop/style/align_hash_spec.rb +111 -30
- data/spec/rubocop/cop/style/align_parameters_spec.rb +260 -187
- data/spec/rubocop/cop/style/and_or_spec.rb +6 -6
- data/spec/rubocop/cop/style/ascii_comments_spec.rb +3 -3
- data/spec/rubocop/cop/style/ascii_identifiers_spec.rb +5 -5
- data/spec/rubocop/cop/style/attr_spec.rb +2 -2
- data/spec/rubocop/cop/style/begin_block_spec.rb +2 -2
- data/spec/rubocop/cop/style/block_comments_spec.rb +3 -3
- data/spec/rubocop/cop/style/block_nesting_spec.rb +26 -26
- data/spec/rubocop/cop/style/blocks_spec.rb +8 -8
- data/spec/rubocop/cop/style/braces_around_hash_parameters_spec.rb +8 -8
- data/spec/rubocop/cop/style/case_equality_spec.rb +2 -2
- data/spec/rubocop/cop/style/case_indentation_spec.rb +21 -21
- data/spec/rubocop/cop/style/character_literal_spec.rb +6 -6
- data/spec/rubocop/cop/style/class_and_module_camel_case_spec.rb +4 -4
- data/spec/rubocop/cop/style/class_and_module_children_spec.rb +129 -0
- data/spec/rubocop/cop/style/class_length_spec.rb +9 -9
- data/spec/rubocop/cop/style/class_methods_spec.rb +8 -8
- data/spec/rubocop/cop/style/class_vars_spec.rb +4 -4
- data/spec/rubocop/cop/style/collection_methods_spec.rb +6 -6
- data/spec/rubocop/cop/style/colon_method_call_spec.rb +16 -16
- data/spec/rubocop/cop/style/comment_annotation_spec.rb +16 -16
- data/spec/rubocop/cop/style/constant_name_spec.rb +12 -12
- data/spec/rubocop/cop/style/cyclomatic_complexity_spec.rb +16 -16
- data/spec/rubocop/cop/style/def_with_parentheses_spec.rb +6 -6
- data/spec/rubocop/cop/style/documentation_spec.rb +14 -14
- data/spec/rubocop/cop/style/dot_position_spec.rb +23 -26
- data/spec/rubocop/cop/style/double_negation_spec.rb +22 -0
- data/spec/rubocop/cop/style/empty_line_between_defs_spec.rb +22 -14
- data/spec/rubocop/cop/style/empty_lines_around_access_modifier_spec.rb +4 -4
- data/spec/rubocop/cop/style/empty_lines_around_body_spec.rb +20 -20
- data/spec/rubocop/cop/style/empty_lines_spec.rb +17 -6
- data/spec/rubocop/cop/style/empty_literal_spec.rb +20 -20
- data/spec/rubocop/cop/style/encoding_spec.rb +5 -5
- data/spec/rubocop/cop/style/end_block_spec.rb +2 -2
- data/spec/rubocop/cop/style/end_of_line_spec.rb +4 -4
- data/spec/rubocop/cop/style/even_odd_spec.rb +18 -18
- data/spec/rubocop/cop/style/favor_join_spec.rb +7 -7
- data/spec/rubocop/cop/style/favor_unless_over_negated_if_spec.rb +7 -7
- data/spec/rubocop/cop/style/favor_until_over_negated_while_spec.rb +3 -3
- data/spec/rubocop/cop/style/file_name_spec.rb +71 -0
- data/spec/rubocop/cop/style/final_newline_spec.rb +4 -4
- data/spec/rubocop/cop/style/flip_flop_spec.rb +4 -4
- data/spec/rubocop/cop/style/for_spec.rb +13 -13
- data/spec/rubocop/cop/style/format_string_spec.rb +136 -0
- data/spec/rubocop/cop/style/global_vars_spec.rb +7 -7
- data/spec/rubocop/cop/style/hash_methods_spec.rb +6 -6
- data/spec/rubocop/cop/style/hash_syntax_spec.rb +11 -11
- data/spec/rubocop/cop/style/if_unless_modifier_spec.rb +29 -11
- data/spec/rubocop/cop/style/if_with_semicolon_spec.rb +1 -1
- data/spec/rubocop/cop/style/indent_array_spec.rb +136 -0
- data/spec/rubocop/cop/style/indent_hash_spec.rb +238 -0
- data/spec/rubocop/cop/style/indentation_consistency_spec.rb +69 -49
- data/spec/rubocop/cop/style/indentation_width_spec.rb +264 -128
- data/spec/rubocop/cop/style/lambda_call_spec.rb +14 -14
- data/spec/rubocop/cop/style/lambda_spec.rb +7 -7
- data/spec/rubocop/cop/style/leading_comment_space_spec.rb +15 -15
- data/spec/rubocop/cop/style/line_end_concatenation_spec.rb +11 -4
- data/spec/rubocop/cop/style/line_length_spec.rb +5 -5
- data/spec/rubocop/cop/style/method_call_parentheses_spec.rb +5 -5
- data/spec/rubocop/cop/style/method_called_on_do_end_block_spec.rb +9 -9
- data/spec/rubocop/cop/style/method_def_parentheses_spec.rb +19 -19
- data/spec/rubocop/cop/style/method_length_spec.rb +17 -17
- data/spec/rubocop/cop/style/method_name_spec.rb +21 -21
- data/spec/rubocop/cop/style/module_function_spec.rb +3 -3
- data/spec/rubocop/cop/style/multiline_block_chain_spec.rb +13 -13
- data/spec/rubocop/cop/style/multiline_if_then_spec.rb +10 -10
- data/spec/rubocop/cop/style/multiline_ternary_operator_spec.rb +3 -3
- data/spec/rubocop/cop/style/nested_ternary_operator_spec.rb +3 -3
- data/spec/rubocop/cop/style/nil_comparison_spec.rb +7 -7
- data/spec/rubocop/cop/style/not_spec.rb +6 -11
- data/spec/rubocop/cop/style/numeric_literals_spec.rb +8 -8
- data/spec/rubocop/cop/style/one_line_conditional_spec.rb +1 -1
- data/spec/rubocop/cop/style/op_method_spec.rb +13 -13
- data/spec/rubocop/cop/style/parameter_lists_spec.rb +6 -6
- data/spec/rubocop/cop/style/parentheses_around_condition_spec.rb +11 -10
- data/spec/rubocop/cop/style/percent_literal_delimiters_spec.rb +250 -0
- data/spec/rubocop/cop/style/perl_backrefs_spec.rb +2 -2
- data/spec/rubocop/cop/style/predicate_name_spec.rb +3 -3
- data/spec/rubocop/cop/style/proc_spec.rb +4 -4
- data/spec/rubocop/cop/style/raise_args_spec.rb +20 -20
- data/spec/rubocop/cop/style/redundant_begin_spec.rb +6 -6
- data/spec/rubocop/cop/style/redundant_exception_spec.rb +6 -6
- data/spec/rubocop/cop/style/redundant_return_spec.rb +22 -22
- data/spec/rubocop/cop/style/redundant_self_spec.rb +14 -14
- data/spec/rubocop/cop/style/regexp_literal_spec.rb +88 -67
- data/spec/rubocop/cop/style/rescue_modifier_spec.rb +17 -17
- data/spec/rubocop/cop/style/self_assignment_spec.rb +43 -0
- data/spec/rubocop/cop/style/semicolon_spec.rb +31 -17
- data/spec/rubocop/cop/style/signal_exception_spec.rb +29 -29
- data/spec/rubocop/cop/style/single_line_block_params_spec.rb +6 -6
- data/spec/rubocop/cop/style/single_line_methods_spec.rb +44 -6
- data/spec/rubocop/cop/style/space_after_colon_spec.rb +1 -1
- data/spec/rubocop/cop/style/space_after_comma_spec.rb +3 -3
- data/spec/rubocop/cop/style/space_after_control_keyword_spec.rb +12 -12
- data/spec/rubocop/cop/style/space_after_method_name_spec.rb +8 -8
- data/spec/rubocop/cop/style/space_after_not_spec.rb +3 -3
- data/spec/rubocop/cop/style/space_after_semicolon_spec.rb +1 -1
- data/spec/rubocop/cop/style/space_around_equals_in_parameter_default_spec.rb +63 -21
- data/spec/rubocop/cop/style/space_around_operators_spec.rb +24 -24
- data/spec/rubocop/cop/style/space_before_block_braces_spec.rb +72 -0
- data/spec/rubocop/cop/style/space_before_modifier_keyword_spec.rb +4 -4
- data/spec/rubocop/cop/style/{space_around_block_braces_spec.rb → space_inside_block_braces_spec.rb} +39 -41
- data/spec/rubocop/cop/style/space_inside_brackets_spec.rb +1 -1
- data/spec/rubocop/cop/style/space_inside_hash_literal_braces_spec.rb +15 -15
- data/spec/rubocop/cop/style/space_inside_parens_spec.rb +1 -1
- data/spec/rubocop/cop/style/special_global_vars_spec.rb +10 -10
- data/spec/rubocop/cop/style/string_literals_spec.rb +31 -31
- data/spec/rubocop/cop/style/symbol_array_spec.rb +9 -9
- data/spec/rubocop/cop/style/tab_spec.rb +3 -3
- data/spec/rubocop/cop/style/trailing_blank_lines_spec.rb +3 -3
- data/spec/rubocop/cop/style/trailing_comma_spec.rb +32 -32
- data/spec/rubocop/cop/style/trailing_whitespace_spec.rb +5 -5
- data/spec/rubocop/cop/style/trivial_accessors_spec.rb +39 -39
- data/spec/rubocop/cop/style/unless_else_spec.rb +1 -1
- data/spec/rubocop/cop/style/variable_interpolation_spec.rb +12 -12
- data/spec/rubocop/cop/style/variable_name_spec.rb +21 -21
- data/spec/rubocop/cop/style/when_then_spec.rb +1 -1
- data/spec/rubocop/cop/style/while_until_do_spec.rb +8 -8
- data/spec/rubocop/cop/style/while_until_modifier_spec.rb +25 -7
- data/spec/rubocop/cop/style/word_array_spec.rb +23 -23
- data/spec/rubocop/cop/team_spec.rb +14 -14
- data/spec/rubocop/file_inspector_spec.rb +14 -12
- data/spec/rubocop/formatter/base_formatter_spec.rb +19 -19
- data/spec/rubocop/formatter/clang_style_formatter_spec.rb +14 -14
- data/spec/rubocop/formatter/disabled_config_formatter_spec.rb +7 -7
- data/spec/rubocop/formatter/emacs_style_formatter_spec.rb +7 -7
- data/spec/rubocop/formatter/file_list_formatter_spec.rb +4 -4
- data/spec/rubocop/formatter/fuubar_style_formatter_spec.rb +129 -0
- data/spec/rubocop/formatter/json_formatter_spec.rb +23 -19
- data/spec/rubocop/formatter/offense_count_formatter_spec.rb +77 -0
- data/spec/rubocop/formatter/progress_formatter_spec.rb +27 -27
- data/spec/rubocop/formatter/simple_text_formatter_spec.rb +16 -16
- data/spec/rubocop/options_spec.rb +7 -8
- data/spec/rubocop/source_parser_spec.rb +0 -54
- data/spec/spec_helper.rb +11 -7
- data/spec/support/file_helper.rb +1 -1
- data/spec/support/mri_syntax_checker.rb +4 -4
- data/spec/support/shared_examples.rb +6 -6
- data/spec/support/statement_modifier_helper.rb +3 -3
- metadata +76 -16
- data/lib/rubocop/cop/style/favor_sprintf.rb +0 -26
- data/lib/rubocop/formatter/offence_count_formatter.rb +0 -49
- data/spec/rubocop/cop/style/favor_sprintf_spec.rb +0 -47
- data/spec/rubocop/formatter/offence_count_formatter_spec.rb +0 -52
@@ -5,9 +5,9 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::PerlBackrefs do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'registers an
|
8
|
+
it 'registers an offense for $1' do
|
9
9
|
inspect_source(cop, ['puts $1'])
|
10
|
-
expect(cop.
|
10
|
+
expect(cop.offenses.size).to eq(1)
|
11
11
|
end
|
12
12
|
|
13
13
|
it 'auto-corrects $1 to Regexp.last_match[1]' do
|
@@ -7,11 +7,11 @@ describe Rubocop::Cop::Style::PredicateName, :config do
|
|
7
7
|
let(:cop_config) { { 'NamePrefixBlacklist' => %w(has_ is_) } }
|
8
8
|
|
9
9
|
%w(has_ is_).each do |prefix|
|
10
|
-
it 'registers an
|
10
|
+
it 'registers an offense for blacklisted method_name' do
|
11
11
|
inspect_source(cop, ["def #{prefix}_attr",
|
12
12
|
' # ...',
|
13
13
|
'end'])
|
14
|
-
expect(cop.
|
14
|
+
expect(cop.offenses.size).to eq(1)
|
15
15
|
expect(cop.highlights).to eq(["#{prefix}_attr"])
|
16
16
|
end
|
17
17
|
end
|
@@ -20,6 +20,6 @@ describe Rubocop::Cop::Style::PredicateName, :config do
|
|
20
20
|
inspect_source(cop, ['def have_attr',
|
21
21
|
' # ...',
|
22
22
|
'end'])
|
23
|
-
expect(cop.
|
23
|
+
expect(cop.offenses).to be_empty
|
24
24
|
end
|
25
25
|
end
|
@@ -5,19 +5,19 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::Proc do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'registers an
|
8
|
+
it 'registers an offense for a Proc.new call' do
|
9
9
|
inspect_source(cop, ['f = Proc.new { |x| puts x }'])
|
10
|
-
expect(cop.
|
10
|
+
expect(cop.offenses.size).to eq(1)
|
11
11
|
end
|
12
12
|
|
13
13
|
it 'accepts the proc method' do
|
14
14
|
inspect_source(cop, ['f = proc { |x| puts x }'])
|
15
|
-
expect(cop.
|
15
|
+
expect(cop.offenses).to be_empty
|
16
16
|
end
|
17
17
|
|
18
18
|
it 'accepts the Proc.new call outside of block' do
|
19
19
|
inspect_source(cop, ['p = Proc.new'])
|
20
|
-
expect(cop.
|
20
|
+
expect(cop.offenses).to be_empty
|
21
21
|
end
|
22
22
|
|
23
23
|
it 'auto-corrects Proc.new to proc' do
|
@@ -8,75 +8,75 @@ describe Rubocop::Cop::Style::RaiseArgs, :config do
|
|
8
8
|
context 'when enforced style is compact' do
|
9
9
|
let(:cop_config) { { 'EnforcedStyle' => 'compact' } }
|
10
10
|
|
11
|
-
it 'reports an
|
11
|
+
it 'reports an offense for a raise with 2 args' do
|
12
12
|
inspect_source(cop, ['raise RuntimeError, msg'])
|
13
|
-
expect(cop.
|
14
|
-
expect(cop.
|
13
|
+
expect(cop.offenses.size).to eq(1)
|
14
|
+
expect(cop.config_to_allow_offenses).to eq('EnforcedStyle' => 'exploded')
|
15
15
|
end
|
16
16
|
|
17
|
-
it 'reports an
|
17
|
+
it 'reports an offense for correct + opposite' do
|
18
18
|
inspect_source(cop, ['if a',
|
19
19
|
' raise RuntimeError, msg',
|
20
20
|
'else',
|
21
21
|
' raise Ex.new(msg)',
|
22
22
|
'end'])
|
23
|
-
expect(cop.
|
24
|
-
expect(cop.
|
23
|
+
expect(cop.offenses.size).to eq(1)
|
24
|
+
expect(cop.config_to_allow_offenses).to eq('Enabled' => false)
|
25
25
|
end
|
26
26
|
|
27
|
-
it 'reports an
|
27
|
+
it 'reports an offense for a raise with 3 args' do
|
28
28
|
inspect_source(cop, ['raise RuntimeError, msg, caller'])
|
29
|
-
expect(cop.
|
29
|
+
expect(cop.offenses.size).to eq(1)
|
30
30
|
end
|
31
31
|
|
32
32
|
it 'accepts a raise with msg argument' do
|
33
33
|
inspect_source(cop, ['raise msg'])
|
34
|
-
expect(cop.
|
34
|
+
expect(cop.offenses).to be_empty
|
35
35
|
end
|
36
36
|
|
37
37
|
it 'accepts a raise with an exception argument' do
|
38
38
|
inspect_source(cop, ['raise Ex.new(msg)'])
|
39
|
-
expect(cop.
|
39
|
+
expect(cop.offenses).to be_empty
|
40
40
|
end
|
41
41
|
end
|
42
42
|
|
43
43
|
context 'when enforced style is exploded' do
|
44
44
|
let(:cop_config) { { 'EnforcedStyle' => 'exploded' } }
|
45
45
|
|
46
|
-
it 'reports an
|
46
|
+
it 'reports an offense for a raise with exception object' do
|
47
47
|
inspect_source(cop, ['raise Ex.new(msg)'])
|
48
|
-
expect(cop.
|
49
|
-
expect(cop.
|
48
|
+
expect(cop.offenses.size).to eq(1)
|
49
|
+
expect(cop.config_to_allow_offenses).to eq('EnforcedStyle' => 'compact')
|
50
50
|
end
|
51
51
|
|
52
|
-
it 'reports an
|
52
|
+
it 'reports an offense for opposite + correct' do
|
53
53
|
inspect_source(cop, ['if a',
|
54
54
|
' raise RuntimeError, msg',
|
55
55
|
'else',
|
56
56
|
' raise Ex.new(msg)',
|
57
57
|
'end'])
|
58
|
-
expect(cop.
|
59
|
-
expect(cop.
|
58
|
+
expect(cop.offenses.size).to eq(1)
|
59
|
+
expect(cop.config_to_allow_offenses).to eq('Enabled' => false)
|
60
60
|
end
|
61
61
|
|
62
62
|
it 'accepts exception constructor with more than 1 argument' do
|
63
63
|
inspect_source(cop, ['raise RuntimeError.new(a1, a2, a3)'])
|
64
|
-
expect(cop.
|
64
|
+
expect(cop.offenses).to be_empty
|
65
65
|
end
|
66
66
|
|
67
67
|
it 'accepts a raise with 3 args' do
|
68
68
|
inspect_source(cop, ['raise RuntimeError, msg, caller'])
|
69
|
-
expect(cop.
|
69
|
+
expect(cop.offenses).to be_empty
|
70
70
|
end
|
71
71
|
|
72
72
|
it 'accepts a raise with 2 args' do
|
73
73
|
inspect_source(cop, ['raise RuntimeError, msg'])
|
74
|
-
expect(cop.
|
74
|
+
expect(cop.offenses).to be_empty
|
75
75
|
end
|
76
76
|
|
77
77
|
it 'accepts a raise with msg argument' do
|
78
78
|
inspect_source(cop, ['raise msg'])
|
79
|
-
expect(cop.
|
79
|
+
expect(cop.offenses).to be_empty
|
80
80
|
end
|
81
81
|
end
|
82
82
|
end
|
@@ -5,7 +5,7 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::RedundantBegin do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'reports an
|
8
|
+
it 'reports an offense for def with redundant begin block' do
|
9
9
|
src = ['def func',
|
10
10
|
' begin',
|
11
11
|
' ala',
|
@@ -14,10 +14,10 @@ describe Rubocop::Cop::Style::RedundantBegin do
|
|
14
14
|
' end',
|
15
15
|
'end']
|
16
16
|
inspect_source(cop, src)
|
17
|
-
expect(cop.
|
17
|
+
expect(cop.offenses.size).to eq(1)
|
18
18
|
end
|
19
19
|
|
20
|
-
it 'reports an
|
20
|
+
it 'reports an offense for defs with redundant begin block' do
|
21
21
|
src = ['def Test.func',
|
22
22
|
' begin',
|
23
23
|
' ala',
|
@@ -26,7 +26,7 @@ describe Rubocop::Cop::Style::RedundantBegin do
|
|
26
26
|
' end',
|
27
27
|
'end']
|
28
28
|
inspect_source(cop, src)
|
29
|
-
expect(cop.
|
29
|
+
expect(cop.offenses.size).to eq(1)
|
30
30
|
end
|
31
31
|
|
32
32
|
it 'accepts a def with required begin block' do
|
@@ -39,7 +39,7 @@ describe Rubocop::Cop::Style::RedundantBegin do
|
|
39
39
|
' something',
|
40
40
|
'end']
|
41
41
|
inspect_source(cop, src)
|
42
|
-
expect(cop.
|
42
|
+
expect(cop.offenses).to be_empty
|
43
43
|
end
|
44
44
|
|
45
45
|
it 'accepts a defs with required begin block' do
|
@@ -52,6 +52,6 @@ describe Rubocop::Cop::Style::RedundantBegin do
|
|
52
52
|
' something',
|
53
53
|
'end']
|
54
54
|
inspect_source(cop, src)
|
55
|
-
expect(cop.
|
55
|
+
expect(cop.offenses).to be_empty
|
56
56
|
end
|
57
57
|
end
|
@@ -5,23 +5,23 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::RedundantException do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'reports an
|
8
|
+
it 'reports an offense for a raise with RuntimeError' do
|
9
9
|
inspect_source(cop, ['raise RuntimeError, msg'])
|
10
|
-
expect(cop.
|
10
|
+
expect(cop.offenses.size).to eq(1)
|
11
11
|
end
|
12
12
|
|
13
|
-
it 'reports an
|
13
|
+
it 'reports an offense for a fail with RuntimeError' do
|
14
14
|
inspect_source(cop, ['fail RuntimeError, msg'])
|
15
|
-
expect(cop.
|
15
|
+
expect(cop.offenses.size).to eq(1)
|
16
16
|
end
|
17
17
|
|
18
18
|
it 'accepts a raise with RuntimeError if it does not have 2 args' do
|
19
19
|
inspect_source(cop, ['raise RuntimeError, msg, caller'])
|
20
|
-
expect(cop.
|
20
|
+
expect(cop.offenses).to be_empty
|
21
21
|
end
|
22
22
|
|
23
23
|
it 'accepts a fail with RuntimeError if it does not have 2 args' do
|
24
24
|
inspect_source(cop, ['fail RuntimeError, msg, caller'])
|
25
|
-
expect(cop.
|
25
|
+
expect(cop.offenses).to be_empty
|
26
26
|
end
|
27
27
|
end
|
@@ -6,40 +6,40 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
6
6
|
subject(:cop) { described_class.new(config) }
|
7
7
|
let(:cop_config) { { 'AllowMultipleReturnValues' => false } }
|
8
8
|
|
9
|
-
it 'reports an
|
9
|
+
it 'reports an offense for def with only a return' do
|
10
10
|
src = ['def func',
|
11
11
|
' return something',
|
12
12
|
'end']
|
13
13
|
inspect_source(cop, src)
|
14
|
-
expect(cop.
|
14
|
+
expect(cop.offenses.size).to eq(1)
|
15
15
|
end
|
16
16
|
|
17
|
-
it 'reports an
|
17
|
+
it 'reports an offense for defs with only a return' do
|
18
18
|
src = ['def Test.func',
|
19
19
|
' return something',
|
20
20
|
'end']
|
21
21
|
inspect_source(cop, src)
|
22
|
-
expect(cop.
|
22
|
+
expect(cop.offenses.size).to eq(1)
|
23
23
|
end
|
24
24
|
|
25
|
-
it 'reports an
|
25
|
+
it 'reports an offense for def ending with return' do
|
26
26
|
src = ['def func',
|
27
27
|
' one',
|
28
28
|
' two',
|
29
29
|
' return something',
|
30
30
|
'end']
|
31
31
|
inspect_source(cop, src)
|
32
|
-
expect(cop.
|
32
|
+
expect(cop.offenses.size).to eq(1)
|
33
33
|
end
|
34
34
|
|
35
|
-
it 'reports an
|
35
|
+
it 'reports an offense for defs ending with return' do
|
36
36
|
src = ['def func',
|
37
37
|
' one',
|
38
38
|
' two',
|
39
39
|
' return something',
|
40
40
|
'end']
|
41
41
|
inspect_source(cop, src)
|
42
|
-
expect(cop.
|
42
|
+
expect(cop.offenses.size).to eq(1)
|
43
43
|
end
|
44
44
|
|
45
45
|
it 'accepts return in a non-final position' do
|
@@ -47,14 +47,14 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
47
47
|
' return something if something_else',
|
48
48
|
'end']
|
49
49
|
inspect_source(cop, src)
|
50
|
-
expect(cop.
|
50
|
+
expect(cop.offenses).to be_empty
|
51
51
|
end
|
52
52
|
|
53
53
|
it 'does not blow up on empty method body' do
|
54
54
|
src = ['def func',
|
55
55
|
'end']
|
56
56
|
inspect_source(cop, src)
|
57
|
-
expect(cop.
|
57
|
+
expect(cop.offenses).to be_empty
|
58
58
|
end
|
59
59
|
|
60
60
|
it 'auto-corrects by removing redundant returns' do
|
@@ -73,40 +73,40 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
73
73
|
end
|
74
74
|
|
75
75
|
context 'when multi-value returns are not allowed' do
|
76
|
-
it 'reports an
|
76
|
+
it 'reports an offense for def with only a return' do
|
77
77
|
src = ['def func',
|
78
78
|
' return something, test',
|
79
79
|
'end']
|
80
80
|
inspect_source(cop, src)
|
81
|
-
expect(cop.
|
81
|
+
expect(cop.offenses.size).to eq(1)
|
82
82
|
end
|
83
83
|
|
84
|
-
it 'reports an
|
84
|
+
it 'reports an offense for defs with only a return' do
|
85
85
|
src = ['def Test.func',
|
86
86
|
' return something, test',
|
87
87
|
'end']
|
88
88
|
inspect_source(cop, src)
|
89
|
-
expect(cop.
|
89
|
+
expect(cop.offenses.size).to eq(1)
|
90
90
|
end
|
91
91
|
|
92
|
-
it 'reports an
|
92
|
+
it 'reports an offense for def ending with return' do
|
93
93
|
src = ['def func',
|
94
94
|
' one',
|
95
95
|
' two',
|
96
96
|
' return something, test',
|
97
97
|
'end']
|
98
98
|
inspect_source(cop, src)
|
99
|
-
expect(cop.
|
99
|
+
expect(cop.offenses.size).to eq(1)
|
100
100
|
end
|
101
101
|
|
102
|
-
it 'reports an
|
102
|
+
it 'reports an offense for defs ending with return' do
|
103
103
|
src = ['def func',
|
104
104
|
' one',
|
105
105
|
' two',
|
106
106
|
' return something, test',
|
107
107
|
'end']
|
108
108
|
inspect_source(cop, src)
|
109
|
-
expect(cop.
|
109
|
+
expect(cop.offenses.size).to eq(1)
|
110
110
|
end
|
111
111
|
|
112
112
|
it 'auto-corrects by making implicit arrays explicit' do
|
@@ -129,7 +129,7 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
129
129
|
' return something, test',
|
130
130
|
'end']
|
131
131
|
inspect_source(cop, src)
|
132
|
-
expect(cop.
|
132
|
+
expect(cop.offenses).to be_empty
|
133
133
|
end
|
134
134
|
|
135
135
|
it 'accepts defs with only a return' do
|
@@ -137,7 +137,7 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
137
137
|
' return something, test',
|
138
138
|
'end']
|
139
139
|
inspect_source(cop, src)
|
140
|
-
expect(cop.
|
140
|
+
expect(cop.offenses).to be_empty
|
141
141
|
end
|
142
142
|
|
143
143
|
it 'accepts def ending with return' do
|
@@ -147,7 +147,7 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
147
147
|
' return something, test',
|
148
148
|
'end']
|
149
149
|
inspect_source(cop, src)
|
150
|
-
expect(cop.
|
150
|
+
expect(cop.offenses).to be_empty
|
151
151
|
end
|
152
152
|
|
153
153
|
it 'accepts defs ending with return' do
|
@@ -157,7 +157,7 @@ describe Rubocop::Cop::Style::RedundantReturn, :config do
|
|
157
157
|
' return something, test',
|
158
158
|
'end']
|
159
159
|
inspect_source(cop, src)
|
160
|
-
expect(cop.
|
160
|
+
expect(cop.offenses).to be_empty
|
161
161
|
end
|
162
162
|
|
163
163
|
it 'does not auto-correct' do
|
@@ -5,46 +5,46 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::RedundantSelf do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'reports an
|
8
|
+
it 'reports an offense a self receiver on an rvalue' do
|
9
9
|
src = ['a = self.b']
|
10
10
|
inspect_source(cop, src)
|
11
|
-
expect(cop.
|
11
|
+
expect(cop.offenses.size).to eq(1)
|
12
12
|
end
|
13
13
|
|
14
14
|
it 'accepts a self receiver on an lvalue of an assignment' do
|
15
15
|
src = ['self.a = b']
|
16
16
|
inspect_source(cop, src)
|
17
|
-
expect(cop.
|
17
|
+
expect(cop.offenses).to be_empty
|
18
18
|
end
|
19
19
|
|
20
20
|
it 'accepts a self receiver on an lvalue of an or-assignment' do
|
21
21
|
src = ['self.logger ||= Rails.logger']
|
22
22
|
inspect_source(cop, src)
|
23
|
-
expect(cop.
|
23
|
+
expect(cop.offenses).to be_empty
|
24
24
|
end
|
25
25
|
|
26
26
|
it 'accepts a self receiver on an lvalue of an and-assignment' do
|
27
27
|
src = ['self.flag &&= value']
|
28
28
|
inspect_source(cop, src)
|
29
|
-
expect(cop.
|
29
|
+
expect(cop.offenses).to be_empty
|
30
30
|
end
|
31
31
|
|
32
32
|
it 'accepts a self receiver on an lvalue of a plus-assignment' do
|
33
33
|
src = ['self.sum += 10']
|
34
34
|
inspect_source(cop, src)
|
35
|
-
expect(cop.
|
35
|
+
expect(cop.offenses).to be_empty
|
36
36
|
end
|
37
37
|
|
38
38
|
it 'accepts a self receiver with the square bracket operator' do
|
39
39
|
src = ['self[a]']
|
40
40
|
inspect_source(cop, src)
|
41
|
-
expect(cop.
|
41
|
+
expect(cop.offenses).to be_empty
|
42
42
|
end
|
43
43
|
|
44
44
|
it 'accepts a self receiver with the double less-than operator' do
|
45
45
|
src = ['self << a']
|
46
46
|
inspect_source(cop, src)
|
47
|
-
expect(cop.
|
47
|
+
expect(cop.offenses).to be_empty
|
48
48
|
end
|
49
49
|
|
50
50
|
it 'accepts a self receiver for methods named like ruby keywords' do
|
@@ -86,7 +86,7 @@ describe Rubocop::Cop::Style::RedundantSelf do
|
|
86
86
|
'self.yield'
|
87
87
|
]
|
88
88
|
inspect_source(cop, src)
|
89
|
-
expect(cop.
|
89
|
+
expect(cop.offenses).to be_empty
|
90
90
|
end
|
91
91
|
|
92
92
|
it 'accepts a self receiver used to distinguish from blockarg' do
|
@@ -95,7 +95,7 @@ describe Rubocop::Cop::Style::RedundantSelf do
|
|
95
95
|
'end'
|
96
96
|
]
|
97
97
|
inspect_source(cop, src)
|
98
|
-
expect(cop.
|
98
|
+
expect(cop.offenses).to be_empty
|
99
99
|
end
|
100
100
|
|
101
101
|
it 'accepts a self receiver used to distinguish from argument' do
|
@@ -104,7 +104,7 @@ describe Rubocop::Cop::Style::RedundantSelf do
|
|
104
104
|
'end'
|
105
105
|
]
|
106
106
|
inspect_source(cop, src)
|
107
|
-
expect(cop.
|
107
|
+
expect(cop.offenses).to be_empty
|
108
108
|
end
|
109
109
|
|
110
110
|
it 'accepts a self receiver used to distinguish from argument' do
|
@@ -113,7 +113,7 @@ describe Rubocop::Cop::Style::RedundantSelf do
|
|
113
113
|
'end'
|
114
114
|
]
|
115
115
|
inspect_source(cop, src)
|
116
|
-
expect(cop.
|
116
|
+
expect(cop.offenses).to be_empty
|
117
117
|
end
|
118
118
|
|
119
119
|
it 'accepts a self receiver used to distinguish from local variable' do
|
@@ -126,13 +126,13 @@ describe Rubocop::Cop::Style::RedundantSelf do
|
|
126
126
|
'end'
|
127
127
|
]
|
128
128
|
inspect_source(cop, src)
|
129
|
-
expect(cop.
|
129
|
+
expect(cop.offenses).to be_empty
|
130
130
|
end
|
131
131
|
|
132
132
|
it 'accepts a self receiver used to distinguish from constant' do
|
133
133
|
src = ['self.Foo']
|
134
134
|
inspect_source(cop, src)
|
135
|
-
expect(cop.
|
135
|
+
expect(cop.offenses).to be_empty
|
136
136
|
end
|
137
137
|
|
138
138
|
it 'auto-corrects by removing redundant self' do
|