require 'set' require File.join(File.dirname(File.dirname(File.expand_path(__FILE__))), 'smell_warning') require File.join(File.dirname(File.dirname(File.expand_path(__FILE__))), 'core', 'smell_configuration') module Reek module Smells module ExcludeInitialize def self.default_config super.adopt(EXCLUDE_KEY => ['initialize']) end def initialize(source, config = self.class.default_config) super(source, config) end end # # Shared responsibilities of all smell detectors. # class SmellDetector # The name of the config field that lists the names of code contexts # that should not be checked. Add this field to the config for each # smell that should ignore this code element. EXCLUDE_KEY = 'exclude' # The default value for the +EXCLUDE_KEY+ if it isn't specified # in any configuration file. DEFAULT_EXCLUDE_SET = [] class << self def contexts [:defn, :defs] end def default_config { Core::SmellConfiguration::ENABLED_KEY => true, EXCLUDE_KEY => DEFAULT_EXCLUDE_SET } end end attr_reader :smells_found # SMELL: only published for tests def initialize(source, config = self.class.default_config) @source = source @config = Core::SmellConfiguration.new(config) @smells_found = [] end def register(hooks) return unless @config.enabled? self.class.contexts.each { |ctx| hooks[ctx] << self } end # SMELL: Getter (only used in 1 test) def enabled? @config.enabled? end def configure_with(config) @config.adopt!(config) end def examine(context) enabled = @config.enabled? && config_for(context)[Core::SmellConfiguration::ENABLED_KEY] != false if enabled && !exception?(context) sm = examine_context(context) @smells_found += sm end end def exception?(context) context.matches?(value(EXCLUDE_KEY, context, DEFAULT_EXCLUDE_SET)) end def report_on(report) @smells_found.each { |smell| smell.report_on(report) } end def value(key, ctx, fall_back) config_for(ctx)[key] || @config.value(key, ctx, fall_back) end def config_for(ctx) ctx.config[self.class.name.split(/::/)[-1]] || {} # BUG: needs to consider smell class AND subclass end end end end