DelayedJob conflicts about email-spec HOT 9 CLOSED

I looked into this a while ago when someone else had the same issue. It's odd that ThinkingSphinx is loading a lib into memory that is not being used. (Apparently a ticket is open to resolve that.) I looked into ways to address this but I could not come up with a nice clean way due to the way the DelayedJob support was originally added.

One easy fix might be to undefined the Delayed const before you load email-spec. Doing that would prevent email-spec from loading the background process module.

So.. in short there is way, currently, to disable this. I would like to see a patch that forces DelatedJob people to require 'email-spec/delayed_job' or something. I went down the path of creating this but it didn't work out. Too much module magic. :( Anyways, patches are welcome as I probably won't be addressing this myself anytime soon.

from email-spec.

er... I meant to say.. "in short there is no way, currently, to disable this"

from email-spec.

Ah I figured it out. Since background_processes.rb is using Delayed::Job not ::Delayed::Job, this solves the problem without affecting anything outside of email-spec:

module EmailSpec
  module Delayed
    class Job
      def self.work_off
        # nothing
      end
    end
  end
end

I think I'm happy with that little hack for the time being :)

I was trying to do what you were describing: have people manually require 'email_spec/background_processes' in their env.rb... but I'm also having a ton a trouble getting the module magic to work... my meta-programming skills are failing me...

To me it seems like it should be as simple as getting rid of "include EmailSpec::BackgroundProcesses::Compatibility" in deliveries.rb:68 and getting rid of "require 'email_spec/background_processes'" on email_spec.rb:8 and changing background_processes.rb to insert itself into EmailSpec::Deliveries... I tried

EmailSpec::Deliveries.class_eval do
  include EmailSpec::BackgroundProcesses::DelayedJob
end

but it doesn't seem to insert the EmailSpec::BackgroundProcesses::DelayedJob methods into the chain... oh well...

from email-spec.

Right, I ran into the same issue. I was able to reproduce it in an irb session. Basically, mixing in modules after the fact effects method delegation differently than as if it happens inline (as presently defined). I think a redesign of how that is all working would be in order. I'm thinking more objects as strategies and less modules and method chaining. :) Glad you got an acceptable work-around for now though.

from email-spec.

I'm working through this and a few other issues now. Stay tuned.

from email-spec.

Fix available at http://github.com/KieranP/email-spec/commit/51c8c8e916c320064d4211740f426827ab31e6ed

from email-spec.

For the ticket record, the check for Delayed was too wide, so also added a check for Delayed::Job or Delayed::Worker, which should help reduce conflicts.

from email-spec.

I haven't had this problem since I started using the gem version of email_spec 0.4.0 http://rubygems.org/gems/email_spec/versions/0.4.0

from email-spec.

Merged in all of Kieran's awesome patches and released 0.6.0.

from email-spec.