ceedling 1.1.10 → 1.2.0

data/spec/units/plugins/report_tests_ide_stdout_spec.rb

@@ -0,0 +1,140 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_helper'
+require 'ceedling/constants'
+require 'ceedling/exceptions'
+require 'ceedling/reportinator'
+require 'ceedling/plugins/plugin'
+require 'ceedling/plugins/plugin_reportinator'
+require 'ceedling/templateinator'
+require 'template_render_cases'
+require 'ceedling/yaml_wrapper'
+
+$: << File.expand_path('../../../../plugins/report_tests_ide_stdout/lib', __FILE__)
+$: << File.expand_path('../../../../lib/ceedling/plugins', __FILE__)
+require 'report_tests_ide_stdout'
+
+# Renders the plugin's template asset through the real PluginReportinator pipeline, with
+# the plugin itself writing each failed and ignored test's lines
+describe ReportTestsIdeStdout do
+  before(:each) do
+    @loginator = double('loginator')
+    allow(@loginator).to receive(:decorate) { |str, _label| str }
+    allow(@loginator).to receive(:log) { |output, *_| @captured = output }
+
+    @reportinator = PluginReportinator.new(
+      { plugin_reportinator_helper: nil, plugin_manager: nil, reportinator: Reportinator.new,
+        loginator: @loginator, templateinator: Templateinator.new }
+    )
+
+    file_wrapper = double('file_wrapper')
+    allow(file_wrapper).to receive(:exist?) { |path| File.exist?( path ) }
+    allow(file_wrapper).to receive(:read) { |path| File.read( path ) }
+
+    setupinator = double( 'setupinator', config_hash: { report_tests_ide_stdout: config } )
+
+    @system_objects = { plugin_reportinator: @reportinator, file_wrapper: file_wrapper, setupinator: setupinator,
+                        configurator: nil, file_path_utils: nil, rake_invocation_tracker: nil }
+    @reportinator.set_system_objects( @system_objects )
+  end
+
+  let(:root_path) { File.join( TemplateRenderCases::ROOT, 'plugins', 'report_tests_ide_stdout' ) }
+
+  # The plugin's own defaults, read as Ceedling reads them, as a project that sets nothing gets them
+  let(:config) do
+    defaults = YamlWrapper.new( { file_wrapper: nil } ).load_string( File.read( File.join( root_path, 'config', 'defaults.yml' ) ) )
+    defaults[:report_tests_ide_stdout]
+  end
+
+  def plugin
+    return @system_objects[:report_tests_ide_stdout] ||= described_class.new( @system_objects, 'report_tests_ide_stdout', root_path )
+  end
+
+  # Newlines normalized as the golden spec does, so a checkout's line endings don't matter
+  def render(hash)
+    template = TemplateRenderCases.normalize_newlines( plugin.send(:load_template) )
+    @reportinator.run_report( template, hash )
+    return @captured
+  end
+
+  let(:results) do
+    {
+      counts: { total: 2, passed: 0, failed: 1, ignored: 1, stdout: 1 },
+      successes: [],
+      failures: [ { source: { file: 'test/TestFoo.c' },
+                    collection: [ { test: 'test_c', line: 30, message: "Expected 1\nWas 2" } ] } ],
+      ignores:   [ { source: { file: 'test/TestFoo.c' },
+                     collection: [ { test: 'test_d', line: 40, message: '' } ] } ],
+      stdout:    [ { source: { file: 'test/TestFoo.c' }, collection: [ 'debug line' ] } ]
+    }
+  end
+
+  def report(results, context: TEST_SYM)
+    return render( { context: context, results: results } )
+  end
+
+  it 'registers the template in its own assets directory' do
+    expect(@reportinator).to receive(:register_test_results_template).with( File.read( File.join( root_path, 'assets', 'test_results.template' ) ) )
+    plugin
+  end
+
+  it 'lists stdout output by source file' do
+    expect(report(results)).to include("test/TestFoo.c: \"debug line\"")
+  end
+
+  it 'reports "No tests executed." when nothing ran' do
+    empty = results.merge(counts: { total: 0, passed: 0, failed: 0, ignored: 0, stdout: 0 }, failures: [], ignores: [], stdout: [])
+    expect(report(empty)).to include('No tests executed.')
+  end
+
+  it 'reports gcc lines by default' do
+    expect(report(results)).to include(
+      "test/TestFoo.c:40:1: warning: test_d\n",
+      "test/TestFoo.c:30:1: error: test_c: Expected 1\n    Was 2\n"
+    )
+  end
+
+  # Coverage plugins render their results through this plugin's template
+  it 'gives a coverage context its banners and the same lines' do
+    output = report(results, context: :gcov)
+
+    expect(output).to include( 'GCOV: FAILED TEST SUMMARY' )
+    expect(output).to include( "\ntest/TestFoo.c:30:1: error: test_c: Expected 1\n" )
+  end
+
+  it 'rejects a configuration it cannot report with' do
+    config[:format] = :msvc
+    expect { plugin }.to raise_error( CeedlingException, /:report_tests_ide_stdout ↳ :format/ )
+  end
+
+  it "decorates the summary banner the same way the pretty report does" do
+    report(results)
+    expect(@loginator).to have_received(:decorate).with('OVERALL TEST SUMMARY', LogLabels::FAIL)
+  end
+
+  context 'with :format :legacy' do
+    let(:config) { { format: :legacy } }
+
+    # Every line of a multi-line message stays a link an IDE can follow
+    it 'reindents a multi-line failure message, prefixing every continuation line with file:line:test:' do
+      expect(report(results)).to include(
+        "test/TestFoo.c:30:test_c: \"Expected 1\ntest/TestFoo.c:30:test_c: Was 2\""
+      )
+    end
+
+    # The golden fixtures hold the report as ERB rendered it, so the plugin's own report
+    # must match them byte for byte
+    TemplateRenderCases.cases.each_pair do |case_name, hash|
+      it "renders #{case_name} exactly as ERB did" do
+        fixture = File.join( TemplateRenderCases::FIXTURES_PATH, "ide__#{case_name}.txt" )
+
+        expect(render( Marshal.load( Marshal.dump( hash ) ) ).b).to eq( File.binread( fixture ) )
+      end
+    end
+  end
+end

data/spec/units/plugins/report_tests_log_factory_spec.rb

@@ -0,0 +1,210 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_helper'
+require 'ceedling/constants'
+require 'ceedling/plugins/plugin'
+require 'ceedling/config/config_walkinator'
+
+$: << File.expand_path('../../../../plugins/report_tests_log_factory/lib', __FILE__)
+require 'report_tests_log_factory'
+require 'tests_reporter'
+
+describe ReportTestsLogFactory do
+  before(:each) do
+    @plugin = described_class.allocate
+    @plugin.instance_variable_set(:@ceedling, { config_walkinator: ConfigWalkinator.new })
+  end
+
+  describe '#setup' do
+    before(:each) do
+      @loginator = double('loginator')
+      ceedling = @plugin.instance_variable_get(:@ceedling)
+      ceedling[:loginator]    = @loginator
+      ceedling[:reportinator] = double('reportinator')
+      ceedling[:file_wrapper] = double('file_wrapper')
+    end
+
+    def set_up(reports)
+      setupinator = double('setupinator', config_hash: { report_tests_log_factory: { reports: reports } })
+      @plugin.instance_variable_get(:@ceedling)[:setupinator] = setupinator
+      @plugin.setup
+    end
+
+    it 'loads a reporter for each configured report' do
+      set_up(['json', 'JUnit'])
+
+      reporters = @plugin.instance_variable_get(:@reporters)
+      expect(reporters.map(&:class)).to eq([JsonTestsReporter, JunitTestsReporter])
+    end
+
+    it 'loads a report named more than once only once' do
+      set_up(['json', 'JSON'])
+
+      expect(@plugin.instance_variable_get(:@reporters).length).to eq(1)
+    end
+
+    it 'reports nothing at all when no reports are configured' do
+      set_up([])
+
+      expect(@loginator).to_not receive(:log)
+      @plugin.post_build(0)
+      @plugin.summary
+    end
+  end
+
+  describe '#load_reporters' do
+    it 'builds a working instance of a known built-in reporter, with its own configuration injected' do
+      reporters = @plugin.send(:load_reporters, ['json'], { json: { filename: 'custom.json' } })
+
+      expect(reporters.length).to eq(1)
+      expect(reporters.first).to be_a(JsonTestsReporter)
+      expect(reporters.first.filename).to eq('custom.json')
+    end
+  end
+
+  describe '#generate_report_name' do
+    before(:each) do
+      configurator = double('configurator')
+      allow(configurator).to receive(:project_name).and_return('MyProject')
+      @plugin.instance_variable_get(:@ceedling)[:configurator] = configurator
+    end
+
+    it 'uses the project name unprefixed for the default test context' do
+      expect(@plugin.send(:generate_report_name, TEST_SYM)).to eq('MyProject')
+    end
+
+    it 'prefixes with the bracketed, uppercased context for any non-default context' do
+      expect(@plugin.send(:generate_report_name, :gcov)).to eq('[GCOV] MyProject')
+    end
+
+    it 'falls back to the default report name for a blank project name' do
+      allow(@plugin.instance_variable_get(:@ceedling)[:configurator]).to receive(:project_name).and_return('  ')
+      expect(@plugin.send(:generate_report_name, TEST_SYM)).to eq(TestsReporter::DEFAULT_REPORT_NAME)
+    end
+  end
+
+  # A configured report name maps to a reporter class by convention
+  describe '.reporter_class_name' do
+    it 'appends TestsReporter to a capitalized single-word report' do
+      expect(described_class.reporter_class_name('json')).to eq('JsonTestsReporter')
+    end
+
+    it 'camel-cases a snake-case report' do
+      expect(described_class.reporter_class_name('fancy_shmancy')).to eq('FancyShmancyTestsReporter')
+    end
+  end
+
+  describe 'build hooks' do
+    let(:reporter) { double('reporter', filename: 'tests_report.json', write: nil) }
+
+    before(:each) do
+      stub_const('PROJECT_BUILD_ARTIFACTS_ROOT', 'build/artifacts')
+
+      @loginator           = double('loginator', log: nil)
+      @reportinator        = double('reportinator', generate_heading: '', generate_progress: '')
+      @plugin_reportinator = double('plugin_reportinator')
+      @configurator        = double('configurator', project_name: 'MyProject')
+      allow(@plugin_reportinator).to receive(:assemble_test_results) { |*args| { from: args.first } }
+
+      ceedling = @plugin.instance_variable_get(:@ceedling)
+      ceedling[:plugin_reportinator] = @plugin_reportinator
+      ceedling[:configurator]        = @configurator
+
+      @plugin.instance_variable_set(:@enabled, true)
+      @plugin.instance_variable_set(:@reporters, [reporter])
+      @plugin.instance_variable_set(:@build_results, {})
+      @plugin.instance_variable_set(:@mutex, Mutex.new)
+      @plugin.instance_variable_set(:@loginator, @loginator)
+      @plugin.instance_variable_set(:@reportinator, @reportinator)
+    end
+
+    def run_build(context, start_s: 100, end_s: 160, results: ['a.pass'])
+      @plugin.pre_test_build(context, start_s) if start_s
+      results.each { |path| @plugin.post_test_fixture_execute(context: context, result_file: path) }
+      @plugin.post_test_build(context, end_s) if end_s
+      @plugin.post_build(200)
+    end
+
+    it "writes each reporter's report for each context under that context's artifacts directory" do
+      expect(reporter).to receive(:write).with(hash_including(filepath: 'build/artifacts/test/tests_report.json', name: 'MyProject'))
+      expect(reporter).to receive(:write).with(hash_including(filepath: 'build/artifacts/gcov/tests_report.json', name: '[GCOV] MyProject'))
+
+      run_build(TEST_SYM)
+      @plugin.pre_test_build(:gcov, 0)
+      @plugin.post_test_fixture_execute(context: :gcov, result_file: 'g.pass')
+      @plugin.post_build(200)
+    end
+
+    it "reports from every result file a context's test fixtures produced" do
+      expect(reporter).to receive(:write).with(hash_including(results: { from: ['a.pass', 'b.pass'] }))
+      run_build(TEST_SYM, results: ['a.pass', 'b.pass'])
+    end
+
+    it 'reports the build duration from the test build start and end' do
+      expect(reporter).to receive(:write).with(hash_including(duration_s: 60))
+      run_build(TEST_SYM)
+    end
+
+    # `ceedling summary` reaches post_test_fixture_execute without pre_test_build
+    it 'reports no duration when the test build start is unknown' do
+      expect(reporter).to receive(:write).with(hash_including(duration_s: nil))
+      run_build(TEST_SYM, start_s: nil)
+    end
+
+    it 'writes nothing when no test results were collected' do
+      expect(reporter).to_not receive(:write)
+      @plugin.post_build(200)
+    end
+
+    it 'does nothing at all when no reports are configured' do
+      @plugin.instance_variable_set(:@enabled, false)
+      expect(reporter).to_not receive(:write)
+      run_build(TEST_SYM)
+    end
+  end
+
+  # `ceedling summary` reports on results already on disk from an earlier build
+  describe '#summary' do
+    let(:reporter) { double('reporter', filename: 'tests_report.json', write: nil) }
+
+    before(:each) do
+      stub_const('PROJECT_BUILD_ARTIFACTS_ROOT', 'build/artifacts')
+      stub_const('PROJECT_TEST_RESULTS_PATH', 'build/test/results')
+      stub_const('COLLECTION_ALL_TESTS', ['test/test_a.c'])
+
+      @file_path_utils     = double('file_path_utils')
+      @plugin_reportinator = double('plugin_reportinator')
+      allow(@file_path_utils).to receive(:form_pass_results_filelist)
+        .with('build/test/results', ['test/test_a.c']).and_return(['build/test/results/test_a.pass'])
+      allow(@plugin_reportinator).to receive(:assemble_test_results)
+        .with(['build/test/results/test_a.pass'], { boom: false }).and_return(:assembled)
+
+      @plugin.instance_variable_get(:@ceedling).merge!(
+        file_path_utils: @file_path_utils, plugin_reportinator: @plugin_reportinator,
+        configurator: double('configurator', project_name: 'MyProject')
+      )
+      @plugin.instance_variable_set(:@enabled, true)
+      @plugin.instance_variable_set(:@reporters, [reporter])
+      @plugin.instance_variable_set(:@loginator, double('loginator', log: nil))
+      @plugin.instance_variable_set(:@reportinator, double('reportinator', generate_heading: '', generate_progress: ''))
+    end
+
+    it "writes each configured report for the test context from the results on disk, with no duration" do
+      expect(reporter).to receive(:write).with(
+        name: 'MyProject', filepath: 'build/artifacts/test/tests_report.json', results: :assembled, duration_s: nil
+      )
+      @plugin.summary
+    end
+
+    it 'does nothing when no reports are configured' do
+      @plugin.instance_variable_set(:@enabled, false)
+      expect(reporter).to_not receive(:write)
+      @plugin.summary
+    end
+  end
+end

data/spec/units/plugins/report_tests_pretty_stdout_spec.rb

@@ -0,0 +1,83 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_helper'
+require 'ceedling/constants'
+require 'ceedling/reportinator'
+require 'ceedling/plugins/plugin_reportinator'
+require 'ceedling/templateinator'
+
+# Renders the real template asset through the real PluginReportinator
+# pipeline -- not a reimplementation of the template's own logic -- so this
+# proves actual rendered output, including the exact multi-line message
+# reindentation that must survive the upcoming refactor to shared helpers
+# byte-for-byte.
+describe "report_tests_pretty_stdout template" do
+  before(:each) do
+    @loginator = double('loginator')
+    allow(@loginator).to receive(:decorate) { |str, _label| str }
+
+    @plugin_reportinator = PluginReportinator.new(
+      {
+        plugin_reportinator_helper: nil, plugin_manager: nil,
+        reportinator: Reportinator.new, loginator: @loginator,
+        templateinator: Templateinator.new
+      }
+    )
+    @plugin_reportinator.set_system_objects({ plugin_reportinator: @plugin_reportinator })
+
+    template = File.read(File.expand_path('../../../../plugins/report_tests_pretty_stdout/assets/test_results.template', __FILE__))
+    @plugin_reportinator.register_test_results_template(template)
+  end
+
+  def render(results)
+    captured = nil
+    allow(@loginator).to receive(:log) { |output, *_| captured = output }
+    @plugin_reportinator.run_test_results_report({ context: TEST_SYM, results: results })
+    captured
+  end
+
+  let(:results) do
+    {
+      counts: { total: 4, passed: 2, failed: 1, ignored: 1, stdout: 0 },
+      successes: [],
+      failures: [ { source: { file: 'test/TestFoo.c' },
+                    collection: [ { test: 'test_c', line: 30, message: "Expected 1\nWas 2" } ] } ],
+      ignores:   [ { source: { file: 'test/TestFoo.c' },
+                     collection: [ { test: 'test_d', line: 40, message: '' } ] } ],
+      stdout: []
+    }
+  end
+
+  it 'reindents a multi-line failure message so continuation lines align under the line-number header' do
+    output = render(results)
+
+    expect(output).to include("Test: test_c\n  At line (30): \"Expected 1\n                Was 2\"")
+  end
+
+  it 'reports TESTED/PASSED/FAILED/IGNORED counts and decorates the summary banner red on any failure' do
+    output = render(results)
+
+    expect(output).to include("TESTED:  4")
+    expect(output).to include("PASSED:  2")
+    expect(output).to include("FAILED:  1")
+    expect(output).to include("IGNORED: 1")
+    expect(@loginator).to have_received(:decorate).with('OVERALL TEST SUMMARY', LogLabels::FAIL)
+  end
+
+  it 'decorates the summary banner green when nothing failed' do
+    passing = results.merge(counts: { total: 1, passed: 1, failed: 0, ignored: 0, stdout: 0 }, failures: [])
+    render(passing)
+
+    expect(@loginator).to have_received(:decorate).with('OVERALL TEST SUMMARY', LogLabels::PASS)
+  end
+
+  it 'reports "No tests executed." when nothing ran' do
+    output = render(results.merge(counts: { total: 0, passed: 0, failed: 0, ignored: 0, stdout: 0 }, failures: [], ignores: []))
+    expect(output).to include('No tests executed.')
+  end
+end

data/spec/units/plugins/report_tests_raw_output_log_spec.rb

@@ -0,0 +1,98 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_helper'
+require 'ceedling/constants'
+require 'ceedling/plugins/plugin'
+
+$: << File.expand_path('../../../../plugins/report_tests_raw_output_log/lib', __FILE__)
+require 'report_tests_raw_output_log'
+
+PROJECT_BUILD_ARTIFACTS_ROOT = 'artifacts' unless defined?(PROJECT_BUILD_ARTIFACTS_ROOT)
+
+describe ReportTestsRawOutputLog do
+  before(:each) do
+    @file_wrapper = double('file_wrapper')
+    @loginator    = double('loginator')
+    @reportinator = double('reportinator')
+
+    allow(@reportinator).to receive(:generate_heading) { |msg| "HEADING: #{msg}" }
+    allow(@reportinator).to receive(:generate_progress) { |msg| "PROGRESS: #{msg}" }
+    allow(@loginator).to receive(:log)
+
+    @plugin = described_class.allocate
+    @plugin.instance_variable_set(:@raw_output, {})
+    @plugin.instance_variable_set(:@mutex, Mutex.new)
+    @plugin.instance_variable_set(:@file_wrapper, @file_wrapper)
+    @plugin.instance_variable_set(:@loginator, @loginator)
+    @plugin.instance_variable_set(:@reportinator, @reportinator)
+  end
+
+  # A delta build reports a skipped test's results from the last build, without running it
+  describe '#post_test_fixture_execute' do
+    it 'collects nothing for a test reported from cached results' do
+      @plugin.post_test_fixture_execute( { context: :test, test_name: 'test_a', shell_result: nil } )
+      expect(@plugin.instance_variable_get(:@raw_output)).to be_empty
+    end
+  end
+
+  # Raw output is whatever a test executable printed besides its results. The scan skips
+  # blank lines and test-result lines, and stops at the summary footer.
+  describe '#extract_output' do
+    it 'keeps stray console output while skipping blank lines, test-result lines, and everything after the footer' do
+      raw = <<~OUTPUT
+        test/TestFoo.c:10:test_something:PASS
+
+        Debug: entering state X
+        test/TestFoo.c:20:test_other:FAIL
+        -----------------------
+        TESTED:  2
+      OUTPUT
+
+      output = @plugin.send(:extract_output, raw)
+
+      expect(output).to eq(["Debug: entering state X\n"])
+    end
+
+    it 'returns an empty array when there is no extra output at all' do
+      raw = <<~OUTPUT
+        test/TestFoo.c:10:test_something:PASS
+        -----------------------
+        TESTED:  1
+      OUTPUT
+
+      expect(@plugin.send(:extract_output, raw)).to eq([])
+    end
+  end
+
+  describe '#post_build' do
+    it 'logs that no extra output was produced and never touches the filesystem when nothing was collected' do
+      expect(@loginator).to receive(:log).with("Tests produced no extra console output.\n")
+      expect(@file_wrapper).not_to receive(:mkdir)
+      expect(@file_wrapper).not_to receive(:write)
+
+      @plugin.post_build(0)
+    end
+  end
+
+  # A log reaches disk only through the injected file layer
+  describe '#post_build' do
+    it 'joins one test executable\'s collected output lines and hands them to file_wrapper.write' do
+      @plugin.instance_variable_get(:@raw_output)[:test] = {
+        'TestFoo' => ["Debug: entering state X\n", "Debug: leaving state X\n"]
+      }
+
+      expect(@file_wrapper).to receive(:mkdir).with(File.join(PROJECT_BUILD_ARTIFACTS_ROOT, 'test'))
+      expect(@file_wrapper).to receive(:write).with(
+        File.join(PROJECT_BUILD_ARTIFACTS_ROOT, 'test', 'TestFoo.raw.log'),
+        "Debug: entering state X\nDebug: leaving state X\n"
+      )
+
+      @plugin.post_build(0)
+    end
+  end
+end

data/spec/units/plugins/report_tests_stdout_plugin_spec.rb

@@ -0,0 +1,145 @@
+# =========================================================================
+#   Ceedling - Test-Centered Build System for C
+#   ThrowTheSwitch.org
+#   Copyright (c) 2010-26 Mike Karlesky, Mark VanderVoord, & Greg Williams
+#   SPDX-License-Identifier: MIT
+# =========================================================================
+
+require 'spec_helper'
+require 'ceedling/constants'
+require 'ceedling/plugins/report_tests_stdout_plugin'
+
+# Ceedling normally derives these from project configuration at runtime.
+PROJECT_TEST_RESULTS_PATH = 'build/test/results' unless defined?(PROJECT_TEST_RESULTS_PATH)
+COLLECTION_ALL_TESTS = [] unless defined?(COLLECTION_ALL_TESTS)
+
+describe ReportTestsStdoutPlugin do
+  before(:each) do
+    @plugin_reportinator     = double('plugin_reportinator')
+    @rake_invocation_tracker = double('rake_invocation_tracker')
+    @configurator            = double('configurator')
+    @file_path_utils         = double('file_path_utils')
+
+    @plugin = described_class.allocate
+    @plugin.instance_variable_set(:@result_list, [])
+    @plugin.instance_variable_set(:@mutex, Mutex.new)
+    @plugin.instance_variable_set(:@configurator, @configurator)
+    @plugin.instance_variable_set(:@plugin_reportinator, @plugin_reportinator)
+    @plugin.instance_variable_set(:@file_path_utils, @file_path_utils)
+    @plugin.instance_variable_set(:@rake_invocation_tracker, @rake_invocation_tracker)
+  end
+
+  describe '#post_test_fixture_execute' do
+    it 'collects a result file that lives under the project test results path' do
+      @plugin.post_test_fixture_execute(result_file: File.join(PROJECT_TEST_RESULTS_PATH, 'TestFoo.pass'))
+      expect(@plugin.instance_variable_get(:@result_list)).to eq([File.join(PROJECT_TEST_RESULTS_PATH, 'TestFoo.pass')])
+    end
+
+    it 'ignores a result file outside the project test results path' do
+      @plugin.post_test_fixture_execute(result_file: 'somewhere/else/TestFoo.pass')
+      expect(@plugin.instance_variable_get(:@result_list)).to be_empty
+    end
+
+    it 'never adds the same result file twice' do
+      path = File.join(PROJECT_TEST_RESULTS_PATH, 'TestFoo.pass')
+      @plugin.post_test_fixture_execute(result_file: path)
+      @plugin.post_test_fixture_execute(result_file: path)
+      expect(@plugin.instance_variable_get(:@result_list)).to eq([path])
+    end
+
+    # Regression lock for #104: a literal `[`/`]` in the build root (legal
+    # filename characters) must not silently break this match the way it
+    # once did when this comparison was regex-based.
+    it 'still matches correctly when the project test results path itself contains brackets' do
+      bracket_path = File.join(PROJECT_TEST_RESULTS_PATH, '[unit]', 'TestFoo.pass')
+      @plugin.post_test_fixture_execute(result_file: bracket_path)
+      expect(@plugin.instance_variable_get(:@result_list)).to eq([bracket_path])
+    end
+  end
+
+  describe '#post_build' do
+    it 'does nothing when no test task was invoked' do
+      allow(@rake_invocation_tracker).to receive(:test_task_invoked?).and_return(false)
+      expect(@plugin_reportinator).not_to receive(:run_test_results_report)
+
+      @plugin.post_build(0)
+    end
+
+    it 'does nothing when raw test results display is configured on' do
+      allow(@rake_invocation_tracker).to receive(:test_task_invoked?).and_return(true)
+      allow(@configurator).to receive(:plugins_display_raw_test_results).and_return(true)
+      expect(@plugin_reportinator).not_to receive(:run_test_results_report)
+
+      @plugin.post_build(0)
+    end
+
+    it 'assembles results and runs the report at the shared floor verbosity' do
+      @plugin.instance_variable_set(:@result_list, ['TestFoo.pass'])
+      allow(@rake_invocation_tracker).to receive(:test_task_invoked?).and_return(true)
+      allow(@configurator).to receive(:plugins_display_raw_test_results).and_return(false)
+      allow(@plugin_reportinator).to receive(:assemble_test_results).with(['TestFoo.pass']).and_return({ marker: :results })
+      allow(@plugin_reportinator).to receive(:test_results_floor_verbosity).and_return(Verbosity::ERRORS)
+
+      expect(@plugin_reportinator).to receive(:run_test_results_report).with(
+        { context: TEST_SYM, results: { marker: :results } }, Verbosity::ERRORS
+      )
+
+      @plugin.post_build(0)
+    end
+  end
+
+  describe '#summary' do
+    it 'does nothing when raw test results display is configured on' do
+      allow(@configurator).to receive(:plugins_display_raw_test_results).and_return(true)
+      expect(@plugin_reportinator).not_to receive(:run_test_results_report)
+
+      @plugin.summary
+    end
+
+    it 'assembles results tolerantly (never raising) from every known test, not just those already collected' do
+      allow(@configurator).to receive(:plugins_display_raw_test_results).and_return(false)
+      allow(@file_path_utils).to receive(:form_pass_results_filelist)
+        .with(PROJECT_TEST_RESULTS_PATH, COLLECTION_ALL_TESTS).and_return(['TestFoo.pass', 'TestBar.pass'])
+      allow(@plugin_reportinator).to receive(:assemble_test_results)
+        .with(['TestFoo.pass', 'TestBar.pass'], { boom: false }).and_return({ marker: :results })
+
+      expect(@plugin_reportinator).to receive(:run_test_results_report).with(
+        { context: TEST_SYM, results: { marker: :results } }
+      )
+
+      @plugin.summary
+    end
+  end
+
+  # Ceedling's own template assets were renamed when ERB was removed, since
+  # template.erb described neither the contents nor the engine any longer. An
+  # out-of-tree plugin subclass may still ship only the old filename, so the old
+  # name is still honored.
+  describe '#load_template' do
+    before(:each) do
+      @file_wrapper = double('file_wrapper')
+      @plugin.instance_variable_set(:@plugin_root_path, '/plugin')
+      @plugin.instance_variable_set(:@ceedling, { file_wrapper: @file_wrapper })
+    end
+
+    def load_template
+      return @plugin.send( :load_template )
+    end
+
+    it 'reads the current asset name when it is present' do
+      allow(@file_wrapper).to receive(:exist?).with('/plugin/assets/test_results.template').and_return( true )
+
+      expect(@file_wrapper).to receive(:read).with('/plugin/assets/test_results.template').and_return( 'current' )
+
+      expect(load_template).to eq( 'current' )
+    end
+
+    it 'falls back to the historical name when only that one is present' do
+      allow(@file_wrapper).to receive(:exist?).with('/plugin/assets/test_results.template').and_return( false )
+
+      expect(@file_wrapper).to receive(:read).with('/plugin/assets/template.erb').and_return( 'legacy' )
+
+      expect(load_template).to eq( 'legacy' )
+    end
+  end
+end