100% test coverage. (#88)

Author: ioquatix

.github/workflows/test-coverage.yaml

@@ -20,6 +20,7 @@ jobs:
           - macos
 
         ruby:
+          - "3.3"
           - ruby
 
     steps:

lib/console/event/spawn.rb

@@ -61,7 +61,7 @@ def initialize(environment, arguments, options)
 			#
 			# @parameter status [Process::Status] The status of the command.
 			def status=(status)
-				@end_time = Time.now
+				@end_time = self.now
 				@status = status
 			end
 
@@ -100,6 +100,12 @@ def emit(*arguments, **options)
 				options[:severity] ||= :info
 				super
 			end
+			
+			private
+			
+			def now
+				Clock.now
+			end
 		end
 	end
 end

lib/console/resolver.rb

@@ -103,8 +103,13 @@ def waiting?
 		#
 		# @parameter trace_point [TracePoint] The trace point that triggered the event.
 		def resolve(trace_point)
-			if block = @names.delete(trace_point.self.to_s)
-				block.call(trace_point.self)
+			resolve_class(trace_point.self)
+		end
+		
+		# @parameter klass [Class] The class that was resolved.
+		def resolve_class(klass)
+			if block = @names.delete(klass.to_s)
+				block.call(klass)
 			end
 
 			if @names.empty?

test/console/compatible/logger.rb

@@ -50,4 +50,11 @@
 
 		expect(stream.string).to be(:include?, '"request_id": 137')
 	end
+	
+	it "ignores messages below the log level" do
+		logger.level = Logger::WARN
+		
+		expect(logger.add(Logger::INFO, "Hello World")).to be == true
+		expect(stream.string).to be == ""
+	end
 end

test/console/event/failure.rb

@@ -14,6 +14,23 @@ def detailed_message(...)
 	end
 end
 
+class PlainError
+	def class
+		StandardError
+	end
+	
+	def message
+		"Plain error!"
+	end
+	
+	def backtrace
+		[]
+	end
+	
+	def cause
+	end
+end
+
 describe Console::Event::Failure do
 	include Sus::Fixtures::Console::CapturedLogger
 
@@ -59,5 +76,44 @@ def detailed_message(...)
 
 			expect(failure.exception).to be == error
 		end
+		
+		it "logs failures directly" do
+			Console::Event::Failure.log("It failed", error, name: "failure")
+			
+			expect(console_capture.last).to have_keys(
+				severity: be == :error,
+				subject: be == "It failed",
+				name: be == "failure",
+			)
+		end
+		
+		it "includes nested causes" do
+			begin
+				begin
+					raise RuntimeError, "Cause!"
+				rescue
+					raise TestError, "Test error!"
+				end
+			rescue TestError => error
+				expect(Console::Event::Failure.for(error).to_hash).to have_keys(
+					cause: have_keys(
+						class: be == "RuntimeError",
+						message: be =~ /Cause!/
+					)
+				)
+			end
+		end
+	end
+	
+	it "returns nil if the default root cannot be determined" do
+		expect(Dir).to receive(:getwd).and_raise(Errno::EMFILE)
+		
+		expect(subject.default_root).to be_nil
+	end
+	
+	it "uses the basic message when detailed messages are unavailable" do
+		expect(subject.new(PlainError.new).to_hash).to have_keys(
+			message: be == "Plain error!"
+		)
 	end
 end

test/console/event/spawn.rb

@@ -23,4 +23,38 @@
 			)
 		)
 	end
+	
+	it "records completion status and duration" do
+		event = subject.for("ls")
+		status = Object.new
+		
+		def status.to_i
+			0
+		end
+		
+		expect(event).to receive(:now).and_return(event.start_time + 1)
+		
+		event.status = status
+		
+		expect(event.end_time).to be == event.start_time + 1
+		expect(event.status).to be == status
+		expect(event.duration).to be == 1
+		expect(event.to_hash).to have_keys(
+			status: be == 0,
+			duration: be == 1
+		)
+	end
+	
+	it "uses the monotonic clock for completion time" do
+		event = subject.for("ls")
+		status = Object.new
+		
+		def status.to_i
+			0
+		end
+		
+		event.status = status
+		
+		expect(event.duration).to be >= 0
+	end
 end

test/console/filter.rb

@@ -59,6 +59,16 @@ def before
 		end
 	end
 
+	with "#all!" do
+		it "enables all messages" do
+			logger.level = Console::Logger::FATAL
+			logger.all!
+			
+			logger.debug(MySubject, "Hello World")
+			expect(output).to be(:include?, "Hello World")
+		end
+	end
+	
 	with "#clear" do
 		it "can't clear non-class subjects" do
 			expect do

test/console/output/default.rb

@@ -32,5 +32,11 @@ def object.tty?
 			output = subject.new(stream, env: {"TERM" => "xterm-256color", "DISPLAY" => ":0"})
 			expect(output).to be_a(Console::Output::Serialized)
 		end
+		
+		it "should use XTerm output for GitHub Actions" do
+			output = subject.new(stream, env: {"GITHUB_ACTIONS" => "true"})
+			expect(output).to be_a(Console::Output::Terminal)
+			expect(output.terminal).to be_a(Console::Terminal::XTerm)
+		end
 	end
 end

test/console/output/null.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+# Released under the MIT License.
+# Copyright, 2026, by Samuel Williams.
+
+require "console/output/null"
+
+describe Console::Output::Null do
+	let(:output) {subject.new}
+	
+	it "is its own last output" do
+		expect(output.last_output).to be_equal(output)
+	end
+	
+	it "ignores calls" do
+		expect(output.call("Hello World")).to be_nil
+	end
+end

test/console/output/sensitive.rb

@@ -40,4 +40,18 @@
 			expect(output).not.to be(:include?, "Samuel Williams")
 		end
 	end
+	
+	with "sensitive: callable" do
+		it "filters strings using the callable" do
+			logger.call("token", sensitive: ->(value){value.upcase})
+			
+			expect(output).to be(:include?, "TOKEN")
+		end
+	end
+	
+	it "redacts nested arguments" do
+		logger.call("subject", ["token"], {password: "token"}, :token)
+		
+		expect(output).not.to be(:include?, "token")
+	end
 end