More test coverage.

Author: ioquatix

lib/console/logger.rb

@@ -19,12 +19,17 @@ class Logger < Filter[debug: 0, info: 1, warn: 2, error: 3, fatal: 4]
 		# Set the default log level based on `$DEBUG` and `$VERBOSE`.
 		# You can also specify CONSOLE_LEVEL=debug or CONSOLE_LEVEL=info in environment.
 		# https://mislav.net/2011/06/ruby-verbose-mode/ has more details about how it all fits together.
-		def self.default_log_level(env = ENV)
+		#
+		# @parameter env [Hash] The environment to read the log level from.
+		# @parameter verbose [Boolean] The verbose flag.
+		# @parameter debug [Boolean] The debug flag.
+		# @returns [Integer] The default log level.
+		def self.default_log_level(env = ENV, verbose: $VERBOSE, debug: $DEBUG)
 			if level = env["CONSOLE_LEVEL"]
 				LEVELS[level.to_sym] || level.to_i
-			elsif $DEBUG
+			elsif debug
 				DEBUG
-			elsif $VERBOSE.nil?
+			elsif verbose.nil?
 				WARN
 			else
 				INFO

test/console/.config/console_debug.rb

@@ -0,0 +1,4 @@
+# Always log at debug level.
+def log_level(env = ENV)
+	:debug
+end

test/console/capture.rb

@@ -10,6 +10,45 @@
 	let(:capture) {subject.new}
 	let(:logger) {Console::Logger.new(capture)}
 
+	with "#each" do
+		it "can iterate over log buffer" do
+			logger.info("Hello World!")
+			
+			capture.each do |entry|
+				expect(entry).to have_keys(
+					severity: be == :info,
+					subject: be == "Hello World!"
+				)
+			end
+		end
+	end
+	
+	with "#first" do
+		it "can access first log entry" do
+			logger.info("Goodbye World!")
+			logger.info("Hello World!")
+			
+			first = capture.first
+			expect(first).to have_keys(
+				severity: be == :info,
+				subject: be == "Goodbye World!"
+			)
+		end
+	end
+	
+	with "#last" do
+		it "can access last log entry" do
+			logger.info("Goodbye World!")
+			logger.info("Hello World!")
+			
+			last = capture.last
+			expect(last).to have_keys(
+				severity: be == :info,
+				subject: be == "Hello World!"
+			)
+		end
+	end
+	
 	with "#clear" do
 		it "can clear log buffer" do
 			logger.info("Hello World!")

test/console/config.rb

@@ -23,4 +23,14 @@
 			expect(logger).to be_a Console::Logger
 		end
 	end
+	
+	with ".load" do
+		let(:path) {File.expand_path(".config/console_debug.rb", __dir__)}
+		
+		it "can load configuration" do
+			config = subject.load(path)
+			
+			expect(config.log_level).to be == :debug
+		end
+	end
 end

test/console/logger.rb

@@ -106,36 +106,17 @@
 	end
 
 	describe ".default_log_level" do
-		def before
-			@debug = $DEBUG
-			@verbose = $VERBOSE
-			super
-		end
-		
-		def after(error = nil)
-			$DEBUG = @debug
-			$VERBOSE = @verbose
-			super
-		end
-		
 		it "should set default log level" do
-			$DEBUG = false
-			$VERBOSE = 0
-			
-			expect(Console::Logger.default_log_level).to be == Console::Logger::INFO
+			expect(Console::Logger.default_log_level(verbose: false, debug: false)).to be == Console::Logger::INFO
 		end
 
 		it "should set default log level based on $DEBUG" do
-			$DEBUG = true
-			
-			expect(Console::Logger.default_log_level).to be == Console::Logger::DEBUG
+			expect(Console::Logger.default_log_level(debug: true)).to be == Console::Logger::DEBUG
 		end
 
 		it "should set default log level based on $VERBOSE" do
-			$DEBUG = false
-			$VERBOSE = true
-			
-			expect(Console::Logger.default_log_level).to be == Console::Logger::INFO
+			expect(Console::Logger.default_log_level(verbose: true, debug: false)).to be == Console::Logger::INFO
+			expect(Console::Logger.default_log_level(verbose: nil, debug: false)).to be == Console::Logger::INFO
 		end
 
 		it "can get log level from ENV" do

test/progress.rb

@@ -11,6 +11,12 @@
 
 	let(:progress) {console_logger.progress("My Measurement", 100)}
 
+	with ".now" do
+		it "can measure time" do
+			expect(Console::Progress.now).to be_a(Float)
+		end
+	end
+	
 	with "#mark" do
 		it "can mark progress" do
 			progress.mark("Hello World!")