require_relative '../../spec_helper'
require_relative '../../../lib/reek/smells/class_variable'
require_relative '../../../lib/reek/context/module_context'
require_relative 'smell_detector_shared'

RSpec.describe Reek::Smells::ClassVariable do
  before :each do
    @source_name = 'dummy_source'
    @detector = build(:smell_detector, smell_type: :ClassVariable, source: @source_name)
    @class_variable = '@@things'
  end

  it_should_behave_like 'SmellDetector'

  context 'with no class variables' do
    it 'records nothing in the class' do
      exp = ast(:class, :Fred)
      expect(@detector.examine_context(Reek::Context::CodeContext.new(nil, exp))).to be_empty
    end

    it 'records nothing in the module' do
      exp = ast(:module, :Fred)
      expect(@detector.examine_context(Reek::Context::CodeContext.new(nil, exp))).to be_empty
    end
  end

  context 'with one class variable' do
    shared_examples_for 'one variable found' do
      before :each do
        ast = Reek::Source::SourceCode.from(@src).syntax_tree
        @smells = @detector.examine_context(Reek::Context::CodeContext.new(nil, ast))
      end

      it 'records only that class variable' do
        expect(@smells.length).to eq(1)
      end

      it 'records the variable name' do
        expect(@smells[0].parameters[:name]).to eq(@class_variable)
      end
    end

    ['class', 'module'].each do |scope|
      context "declared in a #{scope}" do
        before :each do
          @src = "#{scope} Fred; #{@class_variable} = {}; end"
        end

        it_should_behave_like 'one variable found'
      end

      context "used in a #{scope}" do
        before :each do
          @src = "#{scope} Fred; def jim() #{@class_variable} = {}; end; end"
        end

        it_should_behave_like 'one variable found'
      end

      context "indexed in a #{scope}" do
        before :each do
          @src = "#{scope} Fred; def jim() #{@class_variable}[mash] = {}; end; end"
        end

        it_should_behave_like 'one variable found'
      end

      context "declared and used in a #{scope}" do
        before :each do
          @src = "
            #{scope} Fred
              #{@class_variable} = {}
              def jim() #{@class_variable} = {}; end
            end
          "
        end

        it_should_behave_like 'one variable found'
      end

      context "used twice in a #{scope}" do
        before :each do
          @src = "
            #{scope} Fred
              def jeff() #{@class_variable} = {}; end
              def jim()  #{@class_variable} = {}; end
            end
          "
        end

        it_should_behave_like 'one variable found'
      end
    end
  end

  it 'reports the correct fields' do
    src = <<-EOS
      module Fred
        #{@class_variable} = {}
      end
    EOS
    ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
    @warning = @detector.examine_context(ctx)[0]
    expect(@warning.source).to eq(@source_name)
    expect(@warning.smell_category).to eq(described_class.smell_category)
    expect(@warning.smell_type).to eq(described_class.smell_type)
    expect(@warning.parameters[:name]).to eq(@class_variable)
    expect(@warning.lines).to eq([2])
  end
end