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,20 +5,20 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::TrailingWhitespace do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'registers an
|
8
|
+
it 'registers an offense for a line ending with space' do
|
9
9
|
source = ['x = 0 ']
|
10
10
|
inspect_source(cop, source)
|
11
|
-
expect(cop.
|
11
|
+
expect(cop.offenses.size).to eq(1)
|
12
12
|
end
|
13
13
|
|
14
|
-
it 'registers an
|
14
|
+
it 'registers an offense for a line ending with tab' do
|
15
15
|
inspect_source(cop, ["x = 0\t"])
|
16
|
-
expect(cop.
|
16
|
+
expect(cop.offenses.size).to eq(1)
|
17
17
|
end
|
18
18
|
|
19
19
|
it 'accepts a line without trailing whitespace' do
|
20
20
|
inspect_source(cop, ["x = 0\n"])
|
21
|
-
expect(cop.
|
21
|
+
expect(cop.offenses).to be_empty
|
22
22
|
end
|
23
23
|
|
24
24
|
it 'auto-corrects unwanted space' do
|
@@ -15,8 +15,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
15
15
|
'def Foo',
|
16
16
|
' @Foo',
|
17
17
|
'end'])
|
18
|
-
expect(cop.
|
19
|
-
expect(cop.
|
18
|
+
expect(cop.offenses.size).to eq(2)
|
19
|
+
expect(cop.offenses
|
20
20
|
.map(&:line).sort).to eq([1, 5])
|
21
21
|
end
|
22
22
|
|
@@ -30,8 +30,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
30
30
|
' !foo',
|
31
31
|
' end',
|
32
32
|
'end'])
|
33
|
-
expect(cop.
|
34
|
-
expect(cop.
|
33
|
+
expect(cop.offenses.size).to eq(1)
|
34
|
+
expect(cop.offenses
|
35
35
|
.map(&:line).sort).to eq([2])
|
36
36
|
end
|
37
37
|
|
@@ -45,8 +45,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
45
45
|
' !foo',
|
46
46
|
' end',
|
47
47
|
'end'])
|
48
|
-
expect(cop.
|
49
|
-
expect(cop.
|
48
|
+
expect(cop.offenses.size).to eq(1)
|
49
|
+
expect(cop.offenses
|
50
50
|
.map(&:line).sort).to eq([2])
|
51
51
|
end
|
52
52
|
|
@@ -59,8 +59,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
59
59
|
' end',
|
60
60
|
' end',
|
61
61
|
'end'])
|
62
|
-
expect(cop.
|
63
|
-
expect(cop.
|
62
|
+
expect(cop.offenses.size).to eq(1)
|
63
|
+
expect(cop.offenses
|
64
64
|
.map(&:line).sort).to eq([3])
|
65
65
|
end
|
66
66
|
|
@@ -119,8 +119,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
119
119
|
' @val',
|
120
120
|
' end',
|
121
121
|
'end'])
|
122
|
-
expect(cop.
|
123
|
-
expect(cop.
|
122
|
+
expect(cop.offenses.size).to eq(2)
|
123
|
+
expect(cop.offenses
|
124
124
|
.map(&:line).sort).to eq([2, 8])
|
125
125
|
end
|
126
126
|
|
@@ -132,8 +132,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
132
132
|
' @name',
|
133
133
|
' end',
|
134
134
|
'end'])
|
135
|
-
expect(cop.
|
136
|
-
expect(cop.
|
135
|
+
expect(cop.offenses.size).to eq(1)
|
136
|
+
expect(cop.offenses
|
137
137
|
.map(&:line).sort).to eq([3])
|
138
138
|
end
|
139
139
|
|
@@ -145,8 +145,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
145
145
|
' @name = name',
|
146
146
|
' end',
|
147
147
|
'end'])
|
148
|
-
expect(cop.
|
149
|
-
expect(cop.
|
148
|
+
expect(cop.offenses.size).to eq(1)
|
149
|
+
expect(cop.offenses
|
150
150
|
.map(&:line).sort).to eq([3])
|
151
151
|
end
|
152
152
|
|
@@ -159,7 +159,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
159
159
|
' or_more_of_them',
|
160
160
|
' end',
|
161
161
|
'end'])
|
162
|
-
expect(cop.
|
162
|
+
expect(cop.offenses).to be_empty
|
163
163
|
end
|
164
164
|
|
165
165
|
it 'finds trivials with less peculiar methods' do
|
@@ -198,7 +198,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
198
198
|
' #do stuff',
|
199
199
|
'end',
|
200
200
|
'end'])
|
201
|
-
expect(cop.
|
201
|
+
expect(cop.offenses).to be_empty
|
202
202
|
end
|
203
203
|
|
204
204
|
it 'treats splats as non-trivial' do
|
@@ -206,7 +206,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
206
206
|
[' def splatomatic(*values)',
|
207
207
|
' @splatomatic = values',
|
208
208
|
' end'])
|
209
|
-
expect(cop.
|
209
|
+
expect(cop.offenses).to be_empty
|
210
210
|
end
|
211
211
|
|
212
212
|
it 'finds oneliner trivials' do
|
@@ -215,8 +215,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
215
215
|
' def foo; @foo; end',
|
216
216
|
' def foo= foo; @foo = foo; end',
|
217
217
|
'end'])
|
218
|
-
expect(cop.
|
219
|
-
expect(cop.
|
218
|
+
expect(cop.offenses.size).to eq(2)
|
219
|
+
expect(cop.offenses
|
220
220
|
.map(&:line).sort).to eq([2, 3])
|
221
221
|
end
|
222
222
|
|
@@ -225,7 +225,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
225
225
|
['def bar',
|
226
226
|
' @bar + foo',
|
227
227
|
'end'])
|
228
|
-
expect(cop.
|
228
|
+
expect(cop.offenses).to be_empty
|
229
229
|
end
|
230
230
|
|
231
231
|
it 'finds trivial writer' do
|
@@ -233,8 +233,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
233
233
|
['def foo=(val)',
|
234
234
|
' @foo = val',
|
235
235
|
'end'])
|
236
|
-
expect(cop.
|
237
|
-
expect(cop.
|
236
|
+
expect(cop.offenses.size).to eq(1)
|
237
|
+
expect(cop.offenses
|
238
238
|
.map(&:line).sort).to eq([1])
|
239
239
|
end
|
240
240
|
|
@@ -259,8 +259,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
259
259
|
' end',
|
260
260
|
' end',
|
261
261
|
'end'])
|
262
|
-
expect(cop.
|
263
|
-
expect(cop.
|
262
|
+
expect(cop.offenses.size).to eq(1)
|
263
|
+
expect(cop.offenses
|
264
264
|
.map(&:line).sort).to eq([2])
|
265
265
|
end
|
266
266
|
|
@@ -280,8 +280,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
280
280
|
' @bar = bar_value',
|
281
281
|
' end',
|
282
282
|
'end'])
|
283
|
-
expect(cop.
|
284
|
-
expect(cop.
|
283
|
+
expect(cop.offenses.size).to eq(3)
|
284
|
+
expect(cop.offenses
|
285
285
|
.map(&:line).sort).to eq([2, 8, 11])
|
286
286
|
end
|
287
287
|
|
@@ -290,7 +290,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
290
290
|
['def bar=(value)',
|
291
291
|
' @bar = value + 42',
|
292
292
|
'end'])
|
293
|
-
expect(cop.
|
293
|
+
expect(cop.offenses).to be_empty
|
294
294
|
end
|
295
295
|
|
296
296
|
it 'finds trivial writers in a little less trivial class' do
|
@@ -306,8 +306,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
306
306
|
' @bar = bar_value',
|
307
307
|
' end',
|
308
308
|
'end'])
|
309
|
-
expect(cop.
|
310
|
-
expect(cop.
|
309
|
+
expect(cop.offenses.size).to eq(2)
|
310
|
+
expect(cop.offenses
|
311
311
|
.map(&:line).sort).to eq([5, 8])
|
312
312
|
end
|
313
313
|
|
@@ -326,7 +326,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
326
326
|
' foo_method',
|
327
327
|
' end',
|
328
328
|
'end'])
|
329
|
-
expect(cop.
|
329
|
+
expect(cop.offenses).to be_empty
|
330
330
|
end
|
331
331
|
|
332
332
|
it 'does not find trivial writer with exceptions' do
|
@@ -339,7 +339,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
339
339
|
' end',
|
340
340
|
' self[:expiration] = @expiration',
|
341
341
|
' end'])
|
342
|
-
expect(cop.
|
342
|
+
expect(cop.offenses).to be_empty
|
343
343
|
end
|
344
344
|
|
345
345
|
it 'accepts an initialize method looking like a writer' do
|
@@ -347,7 +347,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
347
347
|
[' def initialize(value)',
|
348
348
|
' @top = value',
|
349
349
|
' end'])
|
350
|
-
expect(cop.
|
350
|
+
expect(cop.offenses).to be_empty
|
351
351
|
end
|
352
352
|
|
353
353
|
context 'exact name match required' do
|
@@ -362,8 +362,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
362
362
|
'def bar',
|
363
363
|
' @barr',
|
364
364
|
'end'])
|
365
|
-
expect(cop.
|
366
|
-
expect(cop.
|
365
|
+
expect(cop.offenses.size).to eq(1)
|
366
|
+
expect(cop.offenses
|
367
367
|
.map(&:line).sort).to eq([1])
|
368
368
|
end
|
369
369
|
|
@@ -376,8 +376,8 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
376
376
|
'def bar=(bar)',
|
377
377
|
' @barr = bar',
|
378
378
|
'end'])
|
379
|
-
expect(cop.
|
380
|
-
expect(cop.
|
379
|
+
expect(cop.offenses.size).to eq(1)
|
380
|
+
expect(cop.offenses
|
381
381
|
.map(&:line).sort).to eq([1])
|
382
382
|
end
|
383
383
|
end
|
@@ -390,7 +390,7 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
390
390
|
[' def foo?',
|
391
391
|
' @foo',
|
392
392
|
' end'])
|
393
|
-
expect(cop.
|
393
|
+
expect(cop.offenses).to be_empty
|
394
394
|
end
|
395
395
|
end
|
396
396
|
|
@@ -402,14 +402,14 @@ describe Rubocop::Cop::Style::TrivialAccessors, :config do
|
|
402
402
|
[' def to_foo',
|
403
403
|
' @foo',
|
404
404
|
' end'])
|
405
|
-
expect(cop.
|
405
|
+
expect(cop.offenses).to be_empty
|
406
406
|
end
|
407
407
|
it 'ignores writers in the whitelist' do
|
408
408
|
inspect_source(cop,
|
409
409
|
[' def bar=(bar)',
|
410
410
|
' @bar = bar',
|
411
411
|
' end'])
|
412
|
-
expect(cop.
|
412
|
+
expect(cop.offenses).to be_empty
|
413
413
|
end
|
414
414
|
end
|
415
415
|
end
|
@@ -5,7 +5,7 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::UnlessElse do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'registers an
|
8
|
+
it 'registers an offense for an unless with else' do
|
9
9
|
inspect_source(cop, ['unless x',
|
10
10
|
' a = 1',
|
11
11
|
'else',
|
@@ -5,43 +5,43 @@ require 'spec_helper'
|
|
5
5
|
describe Rubocop::Cop::Style::VariableInterpolation do
|
6
6
|
subject(:cop) { described_class.new }
|
7
7
|
|
8
|
-
it 'registers an
|
8
|
+
it 'registers an offense for interpolated global variables' do
|
9
9
|
inspect_source(cop,
|
10
10
|
['puts "this is a #$test"'])
|
11
|
-
expect(cop.
|
11
|
+
expect(cop.offenses.size).to eq(1)
|
12
12
|
expect(cop.messages)
|
13
13
|
.to eq(['Replace interpolated var $test' \
|
14
|
-
|
14
|
+
' with expression #{$test}.'])
|
15
15
|
end
|
16
16
|
|
17
|
-
it 'registers an
|
17
|
+
it 'registers an offense for interpolated regexp back references' do
|
18
18
|
inspect_source(cop,
|
19
19
|
['puts "this is a #$1"'])
|
20
|
-
expect(cop.
|
20
|
+
expect(cop.offenses.size).to eq(1)
|
21
21
|
expect(cop.messages)
|
22
22
|
.to eq(['Replace interpolated var $1 with expression #{$1}.'])
|
23
23
|
end
|
24
24
|
|
25
|
-
it 'registers an
|
25
|
+
it 'registers an offense for interpolated instance variables' do
|
26
26
|
inspect_source(cop,
|
27
27
|
['puts "this is a #@test"'])
|
28
|
-
expect(cop.
|
28
|
+
expect(cop.offenses.size).to eq(1)
|
29
29
|
expect(cop.messages)
|
30
30
|
.to eq(['Replace interpolated var @test' \
|
31
|
-
|
31
|
+
' with expression #{@test}.'])
|
32
32
|
end
|
33
33
|
|
34
|
-
it 'registers an
|
34
|
+
it 'registers an offense for interpolated class variables' do
|
35
35
|
inspect_source(cop,
|
36
36
|
['puts "this is a #@@t"'])
|
37
|
-
expect(cop.
|
37
|
+
expect(cop.offenses.size).to eq(1)
|
38
38
|
expect(cop.messages)
|
39
39
|
.to eq(['Replace interpolated var @@t with expression #{@@t}.'])
|
40
40
|
end
|
41
41
|
|
42
|
-
it 'does not register an
|
42
|
+
it 'does not register an offense for variables in expressions' do
|
43
43
|
inspect_source(cop,
|
44
44
|
['puts "this is a #{@test} #{@@t} #{$t} #{$1}"'])
|
45
|
-
expect(cop.
|
45
|
+
expect(cop.offenses).to be_empty
|
46
46
|
end
|
47
47
|
end
|
@@ -8,52 +8,52 @@ describe Rubocop::Cop::Style::VariableName, :config do
|
|
8
8
|
shared_examples 'always accepted' do
|
9
9
|
it 'accepts screaming snake case globals' do
|
10
10
|
inspect_source(cop, '$MY_GLOBAL = 0')
|
11
|
-
expect(cop.
|
11
|
+
expect(cop.offenses).to be_empty
|
12
12
|
end
|
13
13
|
|
14
14
|
it 'accepts screaming snake case constants' do
|
15
15
|
inspect_source(cop, 'MY_CONSTANT = 0')
|
16
|
-
expect(cop.
|
16
|
+
expect(cop.offenses).to be_empty
|
17
17
|
end
|
18
18
|
|
19
19
|
it 'accepts assigning to camel case constant' do
|
20
20
|
inspect_source(cop, 'Paren = Struct.new :left, :right, :kind')
|
21
|
-
expect(cop.
|
21
|
+
expect(cop.offenses).to be_empty
|
22
22
|
end
|
23
23
|
|
24
24
|
it 'accepts assignment with indexing of self' do
|
25
25
|
inspect_source(cop, 'self[:a] = b')
|
26
|
-
expect(cop.
|
26
|
+
expect(cop.offenses).to be_empty
|
27
27
|
end
|
28
28
|
end
|
29
29
|
|
30
30
|
context 'when configured for snake_case' do
|
31
31
|
let(:cop_config) { { 'EnforcedStyle' => 'snake_case' } }
|
32
32
|
|
33
|
-
it 'registers an
|
33
|
+
it 'registers an offense for camel case in local variable name' do
|
34
34
|
inspect_source(cop, 'myLocal = 1')
|
35
|
-
expect(cop.
|
35
|
+
expect(cop.offenses.size).to eq(1)
|
36
36
|
expect(cop.highlights).to eq(['myLocal'])
|
37
|
-
expect(cop.
|
37
|
+
expect(cop.config_to_allow_offenses).to eq('EnforcedStyle' =>
|
38
38
|
'camelCase')
|
39
39
|
end
|
40
40
|
|
41
|
-
it 'registers an
|
41
|
+
it 'registers an offense for correct + opposite' do
|
42
42
|
inspect_source(cop, ['my_local = 1',
|
43
43
|
'myLocal = 1'])
|
44
44
|
expect(cop.highlights).to eq(['myLocal'])
|
45
|
-
expect(cop.
|
45
|
+
expect(cop.config_to_allow_offenses).to eq('Enabled' => false)
|
46
46
|
end
|
47
47
|
|
48
|
-
it 'registers an
|
48
|
+
it 'registers an offense for camel case in instance variable name' do
|
49
49
|
inspect_source(cop, '@myAttribute = 3')
|
50
|
-
expect(cop.
|
50
|
+
expect(cop.offenses.size).to eq(1)
|
51
51
|
expect(cop.highlights).to eq(['@myAttribute'])
|
52
52
|
end
|
53
53
|
|
54
|
-
it 'registers an
|
54
|
+
it 'registers an offense for camel case in setter name' do
|
55
55
|
inspect_source(cop, 'self.mySetter = 2')
|
56
|
-
expect(cop.
|
56
|
+
expect(cop.offenses.size).to eq(1)
|
57
57
|
expect(cop.highlights).to eq(['mySetter'])
|
58
58
|
end
|
59
59
|
|
@@ -63,34 +63,34 @@ describe Rubocop::Cop::Style::VariableName, :config do
|
|
63
63
|
context 'when configured for camelCase' do
|
64
64
|
let(:cop_config) { { 'EnforcedStyle' => 'camelCase' } }
|
65
65
|
|
66
|
-
it 'registers an
|
66
|
+
it 'registers an offense for snake case in local variable name' do
|
67
67
|
inspect_source(cop, 'my_local = 1')
|
68
|
-
expect(cop.
|
68
|
+
expect(cop.offenses.size).to eq(1)
|
69
69
|
expect(cop.highlights).to eq(['my_local'])
|
70
|
-
expect(cop.
|
70
|
+
expect(cop.config_to_allow_offenses).to eq('EnforcedStyle' =>
|
71
71
|
'snake_case')
|
72
72
|
end
|
73
73
|
|
74
|
-
it 'registers an
|
74
|
+
it 'registers an offense for opposite + correct' do
|
75
75
|
inspect_source(cop, ['my_local = 1',
|
76
76
|
'myLocal = 1'])
|
77
77
|
expect(cop.highlights).to eq(['my_local'])
|
78
|
-
expect(cop.
|
78
|
+
expect(cop.config_to_allow_offenses).to eq('Enabled' => false)
|
79
79
|
end
|
80
80
|
|
81
81
|
it 'accepts camel case in local variable name' do
|
82
82
|
inspect_source(cop, 'myLocal = 1')
|
83
|
-
expect(cop.
|
83
|
+
expect(cop.offenses).to be_empty
|
84
84
|
end
|
85
85
|
|
86
86
|
it 'accepts camel case in instance variable name' do
|
87
87
|
inspect_source(cop, '@myAttribute = 3')
|
88
|
-
expect(cop.
|
88
|
+
expect(cop.offenses).to be_empty
|
89
89
|
end
|
90
90
|
|
91
91
|
it 'accepts camel case in setter name' do
|
92
92
|
inspect_source(cop, 'self.mySetter = 2')
|
93
|
-
expect(cop.
|
93
|
+
expect(cop.offenses).to be_empty
|
94
94
|
end
|
95
95
|
|
96
96
|
include_examples 'always accepted'
|