mirror of
https://github.com/mperham/sidekiq.git
synced 2022-11-09 13:52:34 -05:00
Minimize Ruby warnings in test suite
This commit is contained in:
parent
e216aa41ca
commit
257b96094f
4 changed files with 29 additions and 75 deletions
3
Rakefile
3
Rakefile
|
@ -1,8 +1,7 @@
|
||||||
require 'bundler/gem_tasks'
|
require 'bundler/gem_tasks'
|
||||||
require 'rake/testtask'
|
require 'rake/testtask'
|
||||||
Rake::TestTask.new(:test) do |test|
|
Rake::TestTask.new(:test) do |test|
|
||||||
#SO MUCH NOISE
|
test.warning = true
|
||||||
#test.warning = true
|
|
||||||
test.pattern = 'test/**/test_*.rb'
|
test.pattern = 'test/**/test_*.rb'
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
@ -174,6 +174,7 @@ class TestApi < Sidekiq::Test
|
||||||
before do
|
before do
|
||||||
ActiveJob::Base.queue_adapter = :sidekiq
|
ActiveJob::Base.queue_adapter = :sidekiq
|
||||||
ActiveJob::Base.logger = nil
|
ActiveJob::Base.logger = nil
|
||||||
|
end
|
||||||
|
|
||||||
class ApiMailer < ActionMailer::Base
|
class ApiMailer < ActionMailer::Base
|
||||||
def test_email(*)
|
def test_email(*)
|
||||||
|
@ -184,7 +185,6 @@ class TestApi < Sidekiq::Test
|
||||||
def perform(*)
|
def perform(*)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
|
||||||
|
|
||||||
class ApiWorker
|
class ApiWorker
|
||||||
include Sidekiq::Worker
|
include Sidekiq::Worker
|
||||||
|
|
|
@ -117,11 +117,13 @@ class TestExtensions < Sidekiq::Test
|
||||||
|
|
||||||
Sidekiq.instance_eval { remove_instance_variable :@delay_removed }
|
Sidekiq.instance_eval { remove_instance_variable :@delay_removed }
|
||||||
# Reload modified modules
|
# Reload modified modules
|
||||||
|
silence_warnings do
|
||||||
load 'sidekiq/extensions/action_mailer.rb'
|
load 'sidekiq/extensions/action_mailer.rb'
|
||||||
load 'sidekiq/extensions/active_record.rb'
|
load 'sidekiq/extensions/active_record.rb'
|
||||||
load 'sidekiq/extensions/generic_proxy.rb'
|
load 'sidekiq/extensions/generic_proxy.rb'
|
||||||
load 'sidekiq/extensions/class_methods.rb'
|
load 'sidekiq/extensions/class_methods.rb'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
|
|
|
@ -414,75 +414,28 @@ class TestWeb < Sidekiq::Test
|
||||||
2.times { add_retry }
|
2.times { add_retry }
|
||||||
3.times { add_scheduled }
|
3.times { add_scheduled }
|
||||||
4.times { add_worker }
|
4.times { add_worker }
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'works' do
|
||||||
get '/stats'
|
get '/stats'
|
||||||
@response = Sidekiq.load_json(last_response.body)
|
@response = Sidekiq.load_json(last_response.body)
|
||||||
end
|
|
||||||
|
|
||||||
it 'can refresh dashboard stats' do
|
|
||||||
assert_equal 200, last_response.status
|
assert_equal 200, last_response.status
|
||||||
end
|
|
||||||
|
|
||||||
describe "for sidekiq" do
|
|
||||||
it 'are namespaced' do
|
|
||||||
assert_includes @response.keys, "sidekiq"
|
assert_includes @response.keys, "sidekiq"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports processed' do
|
|
||||||
assert_equal 5, @response["sidekiq"]["processed"]
|
assert_equal 5, @response["sidekiq"]["processed"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports failed' do
|
|
||||||
assert_equal 2, @response["sidekiq"]["failed"]
|
assert_equal 2, @response["sidekiq"]["failed"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports busy' do
|
|
||||||
assert_equal 4, @response["sidekiq"]["busy"]
|
assert_equal 4, @response["sidekiq"]["busy"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports processes' do
|
|
||||||
assert_equal 1, @response["sidekiq"]["processes"]
|
assert_equal 1, @response["sidekiq"]["processes"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports retries' do
|
|
||||||
assert_equal 2, @response["sidekiq"]["retries"]
|
assert_equal 2, @response["sidekiq"]["retries"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports scheduled' do
|
|
||||||
assert_equal 3, @response["sidekiq"]["scheduled"]
|
assert_equal 3, @response["sidekiq"]["scheduled"]
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports latency' do
|
|
||||||
assert_equal 0, @response["sidekiq"]["default_latency"]
|
assert_equal 0, @response["sidekiq"]["default_latency"]
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe "for redis" do
|
|
||||||
it 'are namespaced' do
|
|
||||||
assert_includes @response.keys, "redis"
|
assert_includes @response.keys, "redis"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports version' do
|
|
||||||
assert_includes @response["redis"].keys, "redis_version"
|
assert_includes @response["redis"].keys, "redis_version"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports uptime' do
|
|
||||||
assert_includes @response["redis"].keys, "uptime_in_days"
|
assert_includes @response["redis"].keys, "uptime_in_days"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports connected clients' do
|
|
||||||
assert_includes @response["redis"].keys, "connected_clients"
|
assert_includes @response["redis"].keys, "connected_clients"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports user memory' do
|
|
||||||
assert_includes @response["redis"].keys, "used_memory_human"
|
assert_includes @response["redis"].keys, "used_memory_human"
|
||||||
end
|
|
||||||
|
|
||||||
it 'reports memory peak' do
|
|
||||||
assert_includes @response["redis"].keys, "used_memory_peak_human"
|
assert_includes @response["redis"].keys, "used_memory_peak_human"
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
|
||||||
|
|
||||||
describe 'stats/queues' do
|
describe 'stats/queues' do
|
||||||
include Sidekiq::Util
|
include Sidekiq::Util
|
||||||
|
|
Loading…
Add table
Reference in a new issue