Class: RuboCop::Cop::FormulaAudit::ComponentsRedundancy Private

Inherits:
RuboCop::Cop::FormulaCop show all
Defined in:
rubocops/components_redundancy.rb

Overview

This class is part of a private API. This class may only be used in the Homebrew/brew repository. Third parties should avoid using this class if possible, as it may be removed or changed without warning.

This cop checks if redundant components are present and for other component errors.

  • url|checksum|mirror should be inside stable block
  • head and head do should not be simultaneously present
  • bottle :unneeded/:disable and bottle do should not be simultaneously present
  • stable do should not be present without a head spec

Constant Summary collapse

HEAD_MSG =

This constant is part of a private API. This constant may only be used in the Homebrew/brew repository. Third parties should avoid using this constant if possible, as it may be removed or changed without warning. 

"`head` and `head do` should not be simultaneously present"
BOTTLE_MSG =

This constant is part of a private API. This constant may only be used in the Homebrew/brew repository. Third parties should avoid using this constant if possible, as it may be removed or changed without warning. 

"`bottle :modifier` and `bottle do` should not be simultaneously present"
STABLE_MSG =

This constant is part of a private API. This constant may only be used in the Homebrew/brew repository. Third parties should avoid using this constant if possible, as it may be removed or changed without warning. 

"`stable do` should not be present without a `head` spec"

Instance Attribute Summary

Attributes inherited from RuboCop::Cop::FormulaCop

#file_path

Instance Method Summary collapse

Methods inherited from RuboCop::Cop::FormulaCop

#audit_comments, #audit_urls, #caveats_strings, #depends_on?, #depends_on_name_type?, #formula_tap, #get_checksum_node, #on_class, #tap_style_exception?, #versioned_formula?

Methods included from HelperFunctions

#block_method_called_in_block?, #block_size, #check_precedence, #class_name, #component_precedes?, #end_column, #expression_negated?, #find_all_blocks, #find_block, #find_blocks, #find_const, #find_every_func_call_by_name, #find_every_method_call_by_name, #find_instance_call, #find_instance_method_call, #find_method_calls_by_name, #find_method_def, #find_method_with_args, #find_node_method_by_name, #find_strings, #format_component, #line_number, #line_start_column, #method_called?, #method_called_ever?, #method_name, #node_equals?, #offending_node, #parameters, #parameters_passed?, #problem, #regex_match_group, #size, #source_buffer, #start_column, #string_content

Instance Method Details

#audit_formula(_node, _class_node, _parent_class_node, body_node) ⇒ Object

This method is part of a private API. This method may only be used in the Homebrew/brew repository. Third parties should avoid using this method if possible, as it may be removed or changed without warning. 



22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
# File 'rubocops/components_redundancy.rb', line 22

def audit_formula(_node, _class_node, _parent_class_node, body_node)
  return if body_node.nil?

  urls = find_method_calls_by_name(body_node, :url)

  urls.each do |url|
    url.arguments.each do |arg|
      next if arg.class != RuboCop::AST::HashNode

      url_args = arg.keys.each.map(&:value)
      if method_called?(body_node, :sha256) && url_args.include?(:tag) && url_args.include?(:revision)
        problem "Do not use both sha256 and tag/revision."
      end
    end
  end

  stable_block = find_block(body_node, :stable)
  if stable_block
    [:url, :sha256, :mirror].each do |method_name|
      problem "`#{method_name}` should be put inside `stable` block" if method_called?(body_node, method_name)
    end
  end

  problem HEAD_MSG if method_called?(body_node, :head) &&
                      find_block(body_node, :head)

  problem BOTTLE_MSG if method_called?(body_node, :bottle) &&
                        find_block(body_node, :bottle)

  return if method_called?(body_node, :head) ||
            find_block(body_node, :head)

  problem STABLE_MSG if stable_block
end