beaker 2.30.0 → 2.30.1

data/lib/beaker/version.rb

@@ -1,5 +1,5 @@
 module Beaker
   module Version
-    STRING = '2.30.0'
+    STRING = '2.30.1'
   end
 end

data/spec/beaker/test_suite_spec.rb

@@ -1,20 +1,317 @@
-require "spec_helper"
-require "beaker/test_suite"
+require 'spec_helper'
+require 'fileutils'
 
 module Beaker
   describe TestSuite do
-    context "runner" do
-      it "returns the native test suite class when the 'native' runner name is provided" do
-        expect(Beaker::TestSuite.runner("native")).to be == Beaker::Runner::Native::TestSuite
+
+    context 'new' do
+      let(:test_dir) { 'tmp/tests' }
+
+      let(:options)  { {'name' => create_files(@files)} }
+      let(:rb_test)  { File.expand_path(test_dir + '/my_ruby_file.rb')    }
+      let(:pl_test)  { File.expand_path(test_dir + '/my_perl_file.pl')    }
+      let(:sh_test)  { File.expand_path(test_dir + '/my_shell_file.sh')   }
+
+      it 'fails without test files' do
+        expect { Beaker::TestSuite.new('name', 'hosts', Hash.new, Time.now, :stop_on_error) }.to raise_error
+      end
+
+      it 'includes specific files as test file when explicitly passed' do
+        @files = [ rb_test ]
+        ts = Beaker::TestSuite.new('name', 'hosts', options, Time.now, :stop_on_error)
+
+        tfs = ts.instance_variable_get(:@test_files)
+        expect(tfs).to include rb_test
+      end
+
+      it 'defaults to :slow fail_mode if not provided through parameter or options' do
+        @files = [ rb_test ]
+        ts = Beaker::TestSuite.new('name', 'hosts', options, Time.now)
+        tfm = ts.instance_variable_get(:@fail_mode)
+        expect(tfm).to be == :slow
       end
 
-      it "returns the minitest test suite class when the 'minitest' runner name is provided" do
-        expect(Beaker::TestSuite.runner("minitest")).to be == Beaker::Runner::MiniTest::TestSuite
+      it 'uses provided parameter fail_mode' do
+        @files = [ rb_test ]
+        ts = Beaker::TestSuite.new('name', 'hosts', options, Time.now, :fast)
+        tfm = ts.instance_variable_get(:@fail_mode)
+        expect(tfm).to be == :fast
       end
 
-      it "returns nil when unknown runner name is provided" do
-        expect(Beaker::TestSuite.runner("unknown")).to be_nil
+      it 'uses options fail_mode if fail_mode parameter is not provided' do
+        @files = [ rb_test ]
+        options[:fail_mode] = :fast
+        ts = Beaker::TestSuite.new('name', 'hosts', options, Time.now)
+        tfm = ts.instance_variable_get(:@fail_mode)
+        expect(tfm).to be == :fast
       end
     end
+
+    context 'run' do
+
+      let( :options )     { make_opts.merge({ :logger => double().as_null_object, 'name' => create_files(@files), :log_dated_dir => '.', :xml_dated_dir => '.'}) }
+      let(:broken_script) { "raise RuntimeError" }
+      let(:fail_script)   { "raise Beaker::DSL::Outcomes::FailTest" }
+      let(:okay_script)   { "true" }
+      let(:rb_test)       { 'my_ruby_file.rb'     }
+      let(:pl_test)       { '/my_perl_file.pl'    }
+      let(:sh_test)       { '/my_shell_file.sh'   }
+      let(:hosts)         { make_hosts() }
+
+      it 'fails fast if fail_mode != :slow and runtime error is raised' do
+        allow( Logger ).to receive('new')
+        @files = [ rb_test, pl_test, sh_test]
+        File.open(rb_test, 'w') { |file| file.write(broken_script) }
+        File.open(pl_test, 'w') { |file| file.write(okay_script) }
+        File.open(sh_test, 'w') { |file| file.write(okay_script) }
+
+        ts = Beaker::TestSuite.new( 'name', hosts, options, Time.now, :stop )
+        tsr = ts.instance_variable_get( :@test_suite_results )
+        allow( tsr ).to receive(:write_junit_xml).and_return( true )
+        allow( tsr ).to receive(:summarize).and_return( true )
+
+        ts.run
+        expect( tsr.errored_tests ).to be === 1
+        expect( tsr.failed_tests ).to be === 0
+        expect( tsr.test_count ).to be === 1
+        expect( tsr.passed_tests).to be === 0
+
+      end
+
+      it 'fails fast if fail_mode != :slow and fail test is raised' do
+        allow( Logger ).to receive('new')
+        @files = [ rb_test, pl_test, sh_test]
+        File.open(rb_test, 'w') { |file| file.write(fail_script) }
+        File.open(pl_test, 'w') { |file| file.write(okay_script) }
+        File.open(sh_test, 'w') { |file| file.write(okay_script) }
+
+        ts = Beaker::TestSuite.new( 'name', hosts, options, Time.now, :stop )
+        tsr = ts.instance_variable_get( :@test_suite_results )
+        allow( tsr ).to receive(:write_junit_xml).and_return( true )
+        allow( tsr ).to receive(:summarize).and_return( true )
+
+        ts.run
+        expect( tsr.errored_tests ).to be === 0
+        expect( tsr.failed_tests ).to be === 1
+        expect( tsr.test_count ).to be === 1
+        expect( tsr.passed_tests).to be === 0
+
+      end
+
+      it 'fails slow if fail_mode = :slow, even if a test fails and there is a runtime error' do
+        allow( Logger ).to receive('new')
+        @files = [ rb_test, pl_test, sh_test]
+        File.open(rb_test, 'w') { |file| file.write(broken_script) }
+        File.open(pl_test, 'w') { |file| file.write(fail_script) }
+        File.open(sh_test, 'w') { |file| file.write(okay_script) }
+
+        ts = Beaker::TestSuite.new( 'name', hosts, options, Time.now, :slow )
+        tsr = ts.instance_variable_get( :@test_suite_results )
+        allow( tsr ).to receive(:write_junit_xml).and_return( true )
+        allow( tsr ).to receive(:summarize).and_return( true )
+
+        ts.run
+        expect( tsr.errored_tests ).to be === 1
+        expect( tsr.failed_tests ).to be === 1
+        expect( tsr.test_count ).to be === 3
+        expect( tsr.passed_tests).to be === 1
+
+      end
+
+
+    end
+
+    describe TestSuite::TestSuiteResult do
+
+      let( :options )           { make_opts.merge({ :logger => double().as_null_object }) }
+      let( :hosts )             { make_hosts() }
+      let( :testcase1 )         { Beaker::TestCase.new( hosts, options[:logger], options) }
+      let( :testcase2 )         { Beaker::TestCase.new( hosts, options[:logger], options) }
+      let( :testcase3 )         { Beaker::TestCase.new( hosts, options[:logger], options) }
+      let( :test_suite_result ) { TestSuite::TestSuiteResult.new( options, "my_suite") }
+
+      it 'supports adding test cases' do
+        expect( test_suite_result.test_count ).to be === 0
+        test_suite_result.add_test_case( testcase1 )
+        expect( test_suite_result.test_count ).to be === 1
+      end
+
+      it 'calculates passed tests' do
+        testcase1.instance_variable_set(:@test_status, :pass)
+        testcase2.instance_variable_set(:@test_status, :pass)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.passed_tests ).to be == 2
+      end
+
+      it 'calculates failed tests' do
+        testcase1.instance_variable_set(:@test_status, :pass)
+        testcase2.instance_variable_set(:@test_status, :pass)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.failed_tests ).to be == 1
+      end
+
+      it 'calculates errored tests' do
+        testcase1.instance_variable_set(:@test_status, :error)
+        testcase2.instance_variable_set(:@test_status, :pass)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.errored_tests ).to be == 1
+      end
+
+      it 'calculates skipped tests' do
+        testcase1.instance_variable_set(:@test_status, :error)
+        testcase2.instance_variable_set(:@test_status, :skip)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.skipped_tests ).to be == 1
+      end
+
+      it 'calculates pending tests' do
+        testcase1.instance_variable_set(:@test_status, :error)
+        testcase2.instance_variable_set(:@test_status, :pending)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.pending_tests ).to be == 1
+      end
+
+      it 'calculates sum_failed as a sum of errored and failed TestCases' do
+        testcase1.instance_variable_set(:@test_status, :error)
+        testcase2.instance_variable_set(:@test_status, :pending)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.sum_failed ).to be == 2
+      end
+
+      it 'reports success with no errors/failures' do
+        testcase1.instance_variable_set(:@test_status, :pass)
+        testcase2.instance_variable_set(:@test_status, :pending)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.success? ).to be == false
+      end
+
+      it 'reports failed if any tests error/fail' do
+        testcase1.instance_variable_set(:@test_status, :pass)
+        testcase2.instance_variable_set(:@test_status, :pending)
+        testcase3.instance_variable_set(:@test_status, :fail)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.failed? ).to be == true
+      end
+
+      it 'can calculate the sum of all TestCase runtimes' do
+        testcase1.instance_variable_set(:@runtime, 1)
+        testcase2.instance_variable_set(:@runtime, 10)
+        testcase3.instance_variable_set(:@runtime, 100)
+        test_suite_result.add_test_case( testcase1 )
+        test_suite_result.add_test_case( testcase2 )
+        test_suite_result.add_test_case( testcase3 )
+        expect( test_suite_result.elapsed_time ).to be == 111
+      end
+
+      describe '#write_junit_xml' do
+        let( :options )     { make_opts.merge({ :logger => double().as_null_object, 'name' => create_files(@files), :log_dated_dir => '.', :xml_dated_dir => '.'}) }
+        let(:rb_test)       { 'my_ruby_file.rb'     }
+
+        it 'doesn\'t re-order test cases themselves on time_sort' do
+          nokogiri_mock = Hash.new
+          allow( nokogiri_mock ).to receive( :add_child )
+          allow( Nokogiri::XML::Node ).to receive( :new ) { nokogiri_mock }
+          allow( LoggerJunit ).to receive( :write_xml ).and_yield( Object.new, nokogiri_mock )
+
+          @files = [ rb_test, rb_test, rb_test]
+          ts = Beaker::TestSuite.new( 'name', hosts, options, Time.now, :fast )
+          tsr = ts.instance_variable_get( :@test_suite_results )
+
+          allow( tsr ).to receive( :start_time ).and_return(0)
+          allow( tsr ).to receive( :stop_time ).and_return(10)
+          expect( tsr.instance_variable_get( :@logger ) ).to receive( :error ).never
+
+          test_cases = []
+          3.times do
+            tc = Beaker::TestCase.new( hosts, options[:logger], options, rb_test)
+            allow( tc ).to receive( :sublog ).and_return( false )
+            test_cases << tc
+          end
+          test_cases[0].instance_variable_set(:@runtime, 3)
+          test_cases[1].instance_variable_set(:@runtime, 301)
+          test_cases[2].instance_variable_set(:@runtime, 101)
+          test_cases.map { |tc| tsr.add_test_case( tc ) }
+
+          original_testcase_order = test_suite_result.instance_variable_get( :@test_cases ).dup
+          tsr.write_junit_xml( 'fakeFilePath07', 'fakeFileToLink09', true )
+          after_testcase_order = test_suite_result.instance_variable_get( :@test_cases ).dup
+          expect( after_testcase_order ).to be === original_testcase_order
+        end
+
+      end
+
+
+    end
+
+    describe '#log_path' do
+      let( :sh_test   ) { '/my_shell_file.sh'   }
+      let( :files     ) { @files ? @files : [sh_test] }
+      let( :options   ) { make_opts.merge({ :logger => double().as_null_object, 'name' => create_files(files) }) }
+      let( :hosts     ) { make_hosts() }
+      let( :testsuite ) { Beaker::TestSuite.new( 'name', hosts, options, Time.now, :stop ) }
+
+      it 'returns the simple joining of the log dir & file as required' do
+        expect(testsuite.log_path('foo.txt', 'man/date')).to be === 'man/date/foo.txt'
+      end
+
+      describe 'builds the base directory correctly' do
+        # the base directory is where the latest symlink itself should live
+
+        it 'in the usual case' do
+          expect( File.symlink?('man/latest') ).to be_falsy
+          testsuite.log_path('foo.txt', 'man/date')
+          expect( File.symlink?('man/latest') ).to be_truthy
+        end
+
+        it 'if given a nested directory' do
+          expect( File.symlink?('a/latest') ).to be_falsy
+          testsuite.log_path('foo.txt', 'a/b/c/d/e/f')
+          expect( File.symlink?('a/latest') ).to be_truthy
+        end
+
+      end
+
+      describe 'builds the symlink directory correctly' do
+        # the symlink directory is where the symlink points to
+
+        it 'in the usual case' do
+          expect( File.symlink?('d/latest') ).to be_falsy
+          testsuite.log_path('foo.txt', 'd/e')
+          expect( File.readlink('d/latest') ).to be === 'e'
+        end
+
+        it 'if given a nested directory' do
+          expect( File.symlink?('f/latest') ).to be_falsy
+          testsuite.log_path('foo.txt', 'f/g/h/i/j/k')
+          expect( File.readlink('f/latest') ).to be === 'g/h/i/j/k'
+        end
+
+      end
+
+    end
+
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker
 version: !ruby/object:Gem::Version
-  version: 2.30.0
+  version: 2.30.1
 platform: ruby
 authors:
 - Puppetlabs
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-02 00:00:00.000000000 Z
+date: 2015-12-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -662,9 +662,6 @@ files:
 - lib/beaker/perf.rb
 - lib/beaker/platform.rb
 - lib/beaker/result.rb
-- lib/beaker/runner/mini_test/test_suite.rb
-- lib/beaker/runner/native/test_case.rb
-- lib/beaker/runner/native/test_suite.rb
 - lib/beaker/shared.rb
 - lib/beaker/shared/error_handler.rb
 - lib/beaker/shared/host_manager.rb
@@ -728,7 +725,6 @@ files:
 - spec/beaker/logger_junit_spec.rb
 - spec/beaker/logger_spec.rb
 - spec/beaker/network_manager_spec.rb
-- spec/beaker/options/beaker_options_spec.rb
 - spec/beaker/options/command_line_parser_spec.rb
 - spec/beaker/options/data/badyaml.cfg
 - spec/beaker/options/data/hosts.cfg
@@ -741,8 +737,6 @@ files:
 - spec/beaker/options/presets_spec.rb
 - spec/beaker/perf_spec.rb
 - spec/beaker/platform_spec.rb
-- spec/beaker/runner/native/test_case_spec.rb
-- spec/beaker/runner/native/test_suite_spec.rb
 - spec/beaker/shared/error_handler_spec.rb
 - spec/beaker/shared/host_manager_spec.rb
 - spec/beaker/shared/repetition_spec.rb

data/lib/beaker/runner/mini_test/test_suite.rb

@@ -1,58 +0,0 @@
-module Beaker
-  module Runner
-    module MiniTest
-      class TestSuite
-        attr_reader :name, :options, :fail_mode # TODO ?
-
-        #Create {TestSuite} instance
-        #@param [String] name The name of the {TestSuite}
-        #@param [Array<Host>] hosts An Array of Hosts to act upon.
-        #@param [Hash{Symbol=>String}] options Options for this object
-        #@option options [Logger] :logger The Logger object to report information to
-        #@option options [String] :log_dir The directory where text run logs will be written
-        #@option options [String] :xml_dir The directory where JUnit XML file will be written
-        #@option options [String] :xml_file The name of the JUnit XML file to be written to
-        #@option options [String] :project_root The full path to the Beaker lib directory
-        #@option options [String] :xml_stylesheet The path to a stylesheet to be applied to the generated XML output
-        #@param [Symbol] fail_mode One of :slow, :fast
-        #@param [Time] timestamp Beaker execution start time
-        def initialize(name, hosts, options, timestamp, fail_mode=nil)
-          # TODO ?
-        end
-
-        def run
-          # TODO ?
-        end
-
-        #Execute all the TestCases in this suite.
-        #This is a wrapper that catches any failures generated during TestSuite::run.
-        def run_and_raise_on_failure
-          # TODO ?
-        end
-
-        # Gives a full file path for output to be written to, maintaining the latest symlink
-        # @param [String] name The file name that we want to write to.
-        # @param [String] log_dir The desired output directory.
-        #                         A symlink will be made from ./basedir/latest to that.
-        # @example
-        #   log_path('output.txt', 'log/2014-06-02_16_31_22')
-        #
-        #     This will create the structure:
-        #
-        #   ./log/2014-06-02_16_31_22/output.txt
-        #   ./log/latest -> 2014-06-02_16_31_22
-        #
-        # @example
-        #   log_path('foo.log', 'log/man/date')
-        #
-        #     This will create the structure:
-        #
-        #   ./log/man/date/foo.log
-        #   ./log/latest -> man/date
-        def log_path(name, log_dir)
-          # TODO ?
-        end
-      end
-    end
-  end
-end

data/lib/beaker/runner/native/test_case.rb

@@ -1,193 +0,0 @@
-[ 'host', 'dsl' ].each do |lib|
-  require "beaker/#{lib}"
-end
-
-require 'tempfile'
-require 'benchmark'
-require 'stringio'
-require 'rbconfig'
-
-module Beaker
-  module Runner
-    module Native
-      # This class represents a single test case. A test case is necessarily
-      # contained all in one file though may have multiple dependent examples.
-      # They are executed in order (save for any teardown procs registered
-      # through {Beaker::DSL::Structure#teardown}) and once completed
-      # the status of the TestCase is saved. Instance readers/accessors provide
-      # the test case access to various details of the environment and suite
-      # the test case is running within.
-      #
-      # See {Beaker::DSL} for more information about writing tests
-      # using the DSL.
-      class TestCase
-        include ::Beaker::DSL
-
-        # The Exception raised by Ruby's STDLIB's test framework (Ruby 1.9)
-        TEST_EXCEPTION_CLASS = ::MiniTest::Assertion
-
-        # Necessary for implementing {Beaker::DSL::Helpers#confine}.
-        # Assumed to be an array of valid {Beaker::Host} objects for
-        # this test case.
-        attr_accessor :hosts
-
-        # Necessary for many methods in {Beaker::DSL}. Assumed to be
-        # an instance of {Beaker::Logger}.
-        attr_accessor :logger
-
-        # Necessary for many methods in {Beaker::DSL::Helpers}.  Assumed to be
-        # a hash.
-        attr_accessor :metadata
-
-        #The full log for this test
-        attr_accessor :sublog
-
-        #The result for the last command run
-        attr_accessor :last_result
-
-        # A Hash of 'product name' => 'version installed', only set when
-        # products are installed via git or PE install steps. See the 'git' or
-        # 'pe' directories within 'ROOT/setup' for examples.
-        attr_reader :version
-
-        # Parsed command line options.
-        attr_reader :options
-
-        # The path to the file which contains this test case.
-        attr_reader :path
-
-        # I don't know why this is here
-        attr_reader :fail_flag
-
-        # The user that is running this tests home directory, needed by 'net/ssh'.
-        attr_reader :usr_home
-
-        # A Symbol denoting the status of this test (:fail, :pending,
-        # :skipped, :pass).
-        attr_reader :test_status
-
-        # The exception that may have stopped this test's execution.
-        attr_reader :exception
-
-        # @deprecated
-        # The amount of time taken to execute the test. Unused, probably soon
-        # to be removed or refactored.
-        attr_reader :runtime
-
-        # An Array of Procs to be called after test execution has stopped
-        # (whether by exception or not).
-        attr_reader :teardown_procs
-
-        # @deprecated
-        # Legacy accessor from when test files would only contain one remote
-        # action.  Contains the Result of the last call to utilize
-        # {Beaker::DSL::Helpers#on}.  Do not use as it is not safe
-        # in test files that use multiple calls to
-        # {Beaker::DSL::Helpers#on}.
-        attr_accessor :result
-
-        # @param [Hosts,Array<Host>] these_hosts The hosts to execute this test
-        #                                        against/on.
-        # @param [Logger] logger A logger that implements
-        #                        {Beaker::Logger}'s interface.
-        # @param [Hash{Symbol=>String}] options Parsed command line options.
-        # @param [String] path The local path to a test file to be executed.
-        def initialize(these_hosts, logger, options={}, path=nil)
-          @hosts   = these_hosts
-          @logger = logger
-          @sublog = ""
-          @options = options
-          @path    = path
-          @usr_home = options[:home]
-          @test_status = :pass
-          @exception = nil
-          @runtime = nil
-          @teardown_procs = []
-          @metadata = {}
-          set_current_test_filename(@path ? File.basename(@path, '.rb') : nil)
-
-
-          #
-          # We put this on each wrapper (rather than the class) so that methods
-          # defined in the tests don't leak out to other tests.
-          class << self
-            def run_test
-              @logger.start_sublog
-              @logger.last_result = nil
-
-              set_current_step_name(nil)
-
-              #add arbitrary role methods
-              roles = []
-              @hosts.each do |host|
-                roles << host[:roles]
-              end
-              add_role_def( roles.flatten.uniq )
-
-              @runtime = Benchmark.realtime do
-                begin
-                  test = File.read(path)
-                  eval test,nil,path,1
-                rescue FailTest, TEST_EXCEPTION_CLASS => e
-                  @test_status = :fail
-                  @exception   = e
-                rescue PendingTest
-                  @test_status = :pending
-                rescue SkipTest
-                  @test_status = :skip
-                rescue StandardError, ScriptError, SignalException => e
-                  log_and_fail_test(e)
-                ensure
-                  @teardown_procs.each do |teardown|
-                    begin
-                      teardown.call
-                    rescue StandardError, SignalException, TEST_EXCEPTION_CLASS => e
-                      log_and_fail_test(e)
-                    end
-                  end
-                end
-              end
-              @sublog = @logger.get_sublog
-              @last_result = @logger.last_result
-              return self
-            end
-
-            private
-
-            # Log an error and mark the test as failed, passing through an
-            # exception so it can be displayed at the end of the total run.
-            #
-            # We break out the complete exception backtrace and log each line
-            # individually as well.
-            #
-            # @param exception [Exception] exception to fail with
-            def log_and_fail_test(exception)
-              logger.error("#{exception.class}: #{exception.message}")
-              bt = exception.backtrace
-              logger.pretty_backtrace(bt).each_line do |line|
-                logger.error(line)
-              end
-              @test_status = :error
-              @exception   = exception
-            end
-          end
-        end
-
-        # The TestCase as a hash
-        # @api public
-        # @note The visibility and semantics of this method are valid, but the
-        #   structure of the Hash it returns may change without notice
-        #
-        # @return [Hash] A Hash representation of this test.
-        def to_hash
-          hash = {}
-          hash['HOSTS'] = {}
-          @hosts.each do |host|
-            hash['HOSTS'][host.name] = host.overrides
-          end
-          hash
-        end
-      end
-    end
-  end
-end