DRY analyzer specs

These pieces were repeated pretty much verbatim between the different
analyzer specs. That seems especially silly since this is, after all, an
engine for pointing out code duplication.

Author: wfleming

spec/cc/engine/analyzers/javascript/main_spec.rb

@@ -6,15 +6,7 @@
 require 'tmpdir'
 
 RSpec.describe CC::Engine::Analyzers::Javascript::Main do
-  around do |example|
-    Dir.mktmpdir do |directory|
-      @code = directory
-
-      Dir.chdir(directory) do
-        example.run
-      end
-    end
-  end
+  include AnalyzerSpec
 
   describe "#run" do
     it "prints an issue" do
@@ -68,21 +60,6 @@
     expect(issues.length).to eq 1
   end
 
-  def create_source_file(path, content)
-    File.write(File.join(@code, path), content)
-  end
-
-  def run_engine(config = nil)
-    io = StringIO.new
-
-    engine = ::CC::Engine::Analyzers::Javascript::Main.new(engine_config: config)
-    reporter = ::CC::Engine::Analyzers::Reporter.new(double(concurrency: 2), engine, io)
-
-    reporter.run
-
-    io.string
-  end
-
   def engine_conf
     CC::Engine::Analyzers::EngineConfig.new({
       'config' => {

spec/cc/engine/analyzers/python/main_spec.rb

@@ -6,15 +6,7 @@
 require "tmpdir"
 
 RSpec.describe CC::Engine::Analyzers::Python::Main do
-  around do |example|
-    Dir.mktmpdir do |directory|
-      @code = directory
-
-      Dir.chdir(directory) do
-        example.run
-      end
-    end
-  end
+  include AnalyzerSpec
 
   describe "#run" do
     it "prints an issue" do
@@ -46,21 +38,6 @@
     end
   end
 
-  def create_source_file(path, content)
-    File.write(File.join(@code, path), content)
-  end
-
-  def run_engine(config = nil)
-    io = StringIO.new
-
-    engine = ::CC::Engine::Analyzers::Python::Main.new(engine_config: config)
-    reporter = ::CC::Engine::Analyzers::Reporter.new(double(concurrency: 2), engine, io)
-
-    reporter.run
-
-    io.string
-  end
-
   def engine_conf
     CC::Engine::Analyzers::EngineConfig.new({
       "config" => {

spec/cc/engine/analyzers/ruby/main_spec.rb

@@ -5,15 +5,7 @@
 require 'tmpdir'
 
 RSpec.describe CC::Engine::Analyzers::Ruby::Main do
-  around do |example|
-    Dir.mktmpdir do |directory|
-      @code = directory
-
-      Dir.chdir(directory) do
-        example.run
-      end
-    end
-  end
+  include AnalyzerSpec
 
   describe "#run" do
     it "prints an issue" do
@@ -35,7 +27,7 @@
           end
       EORUBY
 
-      result = run_engine.strip
+      result = run_engine(engine_conf).strip
       json = JSON.parse(result)
 
       expect(json["type"]).to eq("issue")
@@ -60,24 +52,12 @@
       EORUBY
 
       expect {
-        expect(run_engine).to eq("")
+        expect(run_engine(engine_conf)).to eq("")
       }.to output(/Skipping file/).to_stderr
     end
   end
 
-  def create_source_file(path, content)
-    File.write(File.join(@code, path), content)
-  end
-
-  def run_engine(config = {})
-    io = StringIO.new
-
-    config = CC::Engine::Analyzers::EngineConfig.new(config)
-    engine = ::CC::Engine::Analyzers::Ruby::Main.new(engine_config: config)
-    reporter = ::CC::Engine::Analyzers::Reporter.new(double(concurrency: 2), engine, io)
-
-    reporter.run
-
-    io.string
+  def engine_conf
+    CC::Engine::Analyzers::EngineConfig.new({})
   end
 end

spec/support/analyzer_spec.rb

@@ -0,0 +1,28 @@
+module AnalyzerSpec
+  def self.included(base)
+    base.around do |example|
+      Dir.mktmpdir do |directory|
+        @code = directory
+
+        Dir.chdir(directory) do
+          example.run
+        end
+      end
+    end
+  end
+
+  def create_source_file(path, content)
+    File.write(File.join(@code, path), content)
+  end
+
+  def run_engine(config = nil)
+    io = StringIO.new
+
+    engine = described_class.new(engine_config: config)
+    reporter = ::CC::Engine::Analyzers::Reporter.new(double(concurrency: 2), engine, io)
+
+    reporter.run
+
+    io.string
+  end
+end