@mperham I can change to `to_s` but we are already using `class.name` in
other places. Should they be updated separately in a new pull request?


On Thu, Jun 27, 2013 at 12:09 PM, Mike Perham <notifications@github.com>wrote:

> In test/test_retry.rb:
>
> > @@ -179,10 +179,36 @@ def @redis.with; yield self; end
> >        let(:worker) { Minitest::Mock.new }
> >        let(:msg){ {"class"=>"Bob", "args"=>[1, 2, "foo"], "queue"=>"default", "error_message"=>"kerblammo!", "error_class"=>"RuntimeError", "failed_at"=>Time.now.utc, "retry"=>3, "retry_count"=>3} }
> >
> > -      it 'calls worker retries_exhausted after too many retries' do
> > -        worker.expect(:retries_exhausted, true, [1,2,"foo"])
> > -        task_misbehaving_worker
> > -        worker.verify
> > +      describe "worker method" do
> > +        let(:worker) do
> > +          klass = Class.new do
> > +            include Sidekiq::Worker
> > +
> > +            def self.name; "Worker"; end
>
> You should be able to use class.to_s instead of class.name, iirc.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mperham/sidekiq/pull/1030/files#r4919142>
> .
>



-- 
Justin Mazzi