Rubocop fixes for test/test_log_adapter.rb
This commit is contained in:
parent
5a1a8c9c82
commit
ad98883843
|
@ -76,7 +76,6 @@ AllCops:
|
||||||
- test/test_filters.rb
|
- test/test_filters.rb
|
||||||
- test/test_kramdown.rb
|
- test/test_kramdown.rb
|
||||||
- test/test_liquid_renderer.rb
|
- test/test_liquid_renderer.rb
|
||||||
- test/test_log_adapter.rb
|
|
||||||
- test/test_new_command.rb
|
- test/test_new_command.rb
|
||||||
- test/test_page.rb
|
- test/test_page.rb
|
||||||
- test/test_path_sanitization.rb
|
- test/test_path_sanitization.rb
|
||||||
|
|
|
@ -1,12 +1,15 @@
|
||||||
require 'helper'
|
require "helper"
|
||||||
|
|
||||||
class TestLogAdapter < JekyllUnitTest
|
class TestLogAdapter < JekyllUnitTest
|
||||||
class LoggerDouble
|
class LoggerDouble
|
||||||
attr_accessor :level
|
attr_accessor :level
|
||||||
|
|
||||||
def debug(*); end
|
def debug(*); end
|
||||||
|
|
||||||
def info(*); end
|
def info(*); end
|
||||||
|
|
||||||
def warn(*); end
|
def warn(*); end
|
||||||
|
|
||||||
def error(*); end
|
def error(*); end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -58,8 +61,9 @@ class TestLogAdapter < JekyllUnitTest
|
||||||
should "call #debug on writer return true" do
|
should "call #debug on writer return true" do
|
||||||
writer = LoggerDouble.new
|
writer = LoggerDouble.new
|
||||||
logger = Jekyll::LogAdapter.new(writer)
|
logger = Jekyll::LogAdapter.new(writer)
|
||||||
allow(writer).to receive(:debug).with('topic '.rjust(20) + 'log message').and_return(true)
|
allow(writer).to receive(:debug).with("topic ".rjust(20) +
|
||||||
assert logger.debug('topic', 'log message')
|
"log message").and_return(true)
|
||||||
|
assert logger.debug("topic", "log message")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -67,8 +71,9 @@ class TestLogAdapter < JekyllUnitTest
|
||||||
should "call #info on writer return true" do
|
should "call #info on writer return true" do
|
||||||
writer = LoggerDouble.new
|
writer = LoggerDouble.new
|
||||||
logger = Jekyll::LogAdapter.new(writer)
|
logger = Jekyll::LogAdapter.new(writer)
|
||||||
allow(writer).to receive(:info).with('topic '.rjust(20) + 'log message').and_return(true)
|
allow(writer).to receive(:info).with("topic ".rjust(20) +
|
||||||
assert logger.info('topic', 'log message')
|
"log message").and_return(true)
|
||||||
|
assert logger.info("topic", "log message")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -76,8 +81,9 @@ class TestLogAdapter < JekyllUnitTest
|
||||||
should "call #warn on writer return true" do
|
should "call #warn on writer return true" do
|
||||||
writer = LoggerDouble.new
|
writer = LoggerDouble.new
|
||||||
logger = Jekyll::LogAdapter.new(writer)
|
logger = Jekyll::LogAdapter.new(writer)
|
||||||
allow(writer).to receive(:warn).with('topic '.rjust(20) + 'log message').and_return(true)
|
allow(writer).to receive(:warn).with("topic ".rjust(20) +
|
||||||
assert logger.warn('topic', 'log message')
|
"log message").and_return(true)
|
||||||
|
assert logger.warn("topic", "log message")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -85,16 +91,17 @@ class TestLogAdapter < JekyllUnitTest
|
||||||
should "call #error on writer return true" do
|
should "call #error on writer return true" do
|
||||||
writer = LoggerDouble.new
|
writer = LoggerDouble.new
|
||||||
logger = Jekyll::LogAdapter.new(writer)
|
logger = Jekyll::LogAdapter.new(writer)
|
||||||
allow(writer).to receive(:error).with('topic '.rjust(20) + 'log message').and_return(true)
|
allow(writer).to receive(:error).with("topic ".rjust(20) +
|
||||||
assert logger.error('topic', 'log message')
|
"log message").and_return(true)
|
||||||
|
assert logger.error("topic", "log message")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context "#abort_with" do
|
context "#abort_with" do
|
||||||
should "call #error and abort" do
|
should "call #error and abort" do
|
||||||
logger = Jekyll::LogAdapter.new(LoggerDouble.new)
|
logger = Jekyll::LogAdapter.new(LoggerDouble.new)
|
||||||
allow(logger).to receive(:error).with('topic', 'log message').and_return(true)
|
allow(logger).to receive(:error).with("topic", "log message").and_return(true)
|
||||||
assert_raises(SystemExit) { logger.abort_with('topic', 'log message') }
|
assert_raises(SystemExit) { logger.abort_with("topic", "log message") }
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -105,7 +112,7 @@ class TestLogAdapter < JekyllUnitTest
|
||||||
|
|
||||||
should "store each log value in the array" do
|
should "store each log value in the array" do
|
||||||
logger = Jekyll::LogAdapter.new(LoggerDouble.new)
|
logger = Jekyll::LogAdapter.new(LoggerDouble.new)
|
||||||
values = %w{one two three four}
|
values = %w(one two three four)
|
||||||
logger.debug(values[0])
|
logger.debug(values[0])
|
||||||
logger.info(values[1])
|
logger.info(values[1])
|
||||||
logger.warn(values[2])
|
logger.warn(values[2])
|
||||||
|
|
Loading…
Reference in New Issue