Create XMLMiniEngineTest; base class for all tests

activesupport/test/xml_mini/jdom_engine_test.rb

@@ -1,14 +1,9 @@
-if RUBY_PLATFORM.include?("java")
+require_relative "xml_mini_engine_test"
-  require "abstract_unit"
-  require "active_support/xml_mini"
-  require "active_support/core_ext/hash/conversions"
-  require_relative "./common"
 
-  class JDOMEngineTest < ActiveSupport::TestCase
+XMLMiniEngineTest.run_with_platform("java") do
+  class JDOMEngineTest < XMLMiniEngineTest
     FILES_DIR = File.dirname(__FILE__) + "/../fixtures/xml"
 
-    include CommonXMLMiniAdapterTest
-
     def test_not_allowed_to_expand_entities_to_files
       attack_xml = <<-EOT
       <!DOCTYPE member [

activesupport/test/xml_mini/libxml_engine_test.rb

@@ -1,16 +1,7 @@
-begin
+require_relative "xml_mini_engine_test"
-  require "libxml"
-rescue LoadError
-  # Skip libxml tests
-else
-  require "abstract_unit"
-  require "active_support/xml_mini"
-  require "active_support/core_ext/hash/conversions"
-  require_relative "./common"
-
-  class LibxmlEngineTest < ActiveSupport::TestCase
-    include CommonXMLMiniAdapterTest
 
+XMLMiniEngineTest.run_with_gem("libxml") do
+  class LibxmlEngineTest < XMLMiniEngineTest
     def setup
       super
       LibXML::XML::Error.set_handler(&lambda { |error| }) #silence libxml, exceptions will do

activesupport/test/xml_mini/libxmlsax_engine_test.rb

@@ -1,16 +1,7 @@
-begin
+require_relative "xml_mini_engine_test"
-  require "libxml"
-rescue LoadError
-  # Skip libxml tests
-else
-  require "abstract_unit"
-  require "active_support/xml_mini"
-  require "active_support/core_ext/hash/conversions"
-  require_relative "./common"
-
-  class LibXMLSAXEngineTest < ActiveSupport::TestCase
-    include CommonXMLMiniAdapterTest
 
+XMLMiniEngineTest.run_with_gem("libxml") do
+  class LibXMLSAXEngineTest < XMLMiniEngineTest
     private
 
       def engine

activesupport/test/xml_mini/nokogiri_engine_test.rb

@@ -1,16 +1,7 @@
-begin
+require_relative "xml_mini_engine_test"
-  require "nokogiri"
-rescue LoadError
-  # Skip nokogiri tests
-else
-  require "abstract_unit"
-  require "active_support/xml_mini"
-  require "active_support/core_ext/hash/conversions"
-  require_relative "./common"
-
-  class NokogiriEngineTest < ActiveSupport::TestCase
-    include CommonXMLMiniAdapterTest
 
+XMLMiniEngineTest.run_with_gem("nokogiri") do
+  class NokogiriEngineTest < XMLMiniEngineTest
     private
 
       def engine

activesupport/test/xml_mini/nokogirisax_engine_test.rb

@@ -1,16 +1,7 @@
-begin
+require_relative "xml_mini_engine_test"
-  require "nokogiri"
-rescue LoadError
-  # Skip nokogiri tests
-else
-  require "abstract_unit"
-  require "active_support/xml_mini"
-  require "active_support/core_ext/hash/conversions"
-  require_relative "./common"
-
-  class NokogiriSAXEngineTest < ActiveSupport::TestCase
-    include CommonXMLMiniAdapterTest
 
+XMLMiniEngineTest.run_with_gem("nokogiri") do
+  class NokogiriSAXEngineTest < XMLMiniEngineTest
     private
 
       def engine

activesupport/test/xml_mini/rexml_engine_test.rb

@@ -1,10 +1,6 @@
-require "abstract_unit"
+require_relative "xml_mini_engine_test"
-require "active_support/xml_mini"
-require_relative "./common"
-
-class REXMLEngineTest < ActiveSupport::TestCase
-  include CommonXMLMiniAdapterTest
 
+class REXMLEngineTest < XMLMiniEngineTest
   def test_default_is_rexml
     assert_equal ActiveSupport::XmlMini_REXML, ActiveSupport::XmlMini.backend
   end

activesupport/test/xml_mini/xml_mini_engine_test.rb

@@ -1,4 +1,19 @@
-module CommonXMLMiniAdapterTest
+require "abstract_unit"
+require "active_support/xml_mini"
+require "active_support/core_ext/hash/conversions"
+
+class XMLMiniEngineTest < ActiveSupport::TestCase
+  def self.run_with_gem(gem_name)
+    require gem_name
+    yield
+  rescue LoadError
+    # Skip tests unless gem is available
+  end
+
+  def self.run_with_platform(platform_name)
+    yielf if RUBY_PLATFORM.include?(platform_name)
+  end
+
   def setup
     @default_backend = ActiveSupport::XmlMini.backend
     ActiveSupport::XmlMini.backend = engine