inst-jobs 0.11.5 → 0.11.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8c54b2321b4cf650d162f7d8dad889cc1adb2ef3
-  data.tar.gz: 36e4af19910d4df69bd943d20c24bdc97df73b38
+  metadata.gz: e21f31dc7f5360307577a7c3e31d1747f8b7359c
+  data.tar.gz: 2191834aa82cf0a63065b9ac951f9aafbd78d391
 SHA512:
-  metadata.gz: 9c3b87f1a259e2316f15beb63ffe7311ed17fe4c113691150545ba8eff4f117ac0693bcf9fb8f8cc71b95502eec9054d783b5463a3ced59a82e26111506b6825
-  data.tar.gz: eed4955ae2845e22fc60a0140b13eaf0c66ae7d47cb756e17312d3e452ae5cbdab8d31aea2c751dd86eb93fc775dd2c2672ca28abbac74c6996024056d75e698
+  metadata.gz: 6558c7935c3dc4347a47685527fec8c5529b610649ee4a26ddeb10216ca5e15e8ef154b4b6bb87088a347a7327678b4c19015d68b6e5ae2e799b585802197010
+  data.tar.gz: ea259677667525224c10d0f73ee354b33e4e75b2fa78ae42ed406a42c293540f6cb3183c5a04da2b1ff3694e49f5e8b726132559575e62c8e02d7e1468f57cc7

data/db/migrate/20161206323555_add_back_default_string_limits_jobs.rb

@@ -0,0 +1,38 @@
+class AddBackDefaultStringLimitsJobs < ActiveRecord::Migration
+  def connection
+    Delayed::Job.connection
+  end
+
+  def up
+    drop_triggers
+
+    add_string_limit_if_missing :delayed_jobs, :queue
+    add_string_limit_if_missing :delayed_jobs, :locked_by
+    add_string_limit_if_missing :delayed_jobs, :tag
+    add_string_limit_if_missing :delayed_jobs, :strand
+    add_string_limit_if_missing :delayed_jobs, :source
+
+    add_string_limit_if_missing :failed_jobs, :queue
+    add_string_limit_if_missing :failed_jobs, :locked_by
+    add_string_limit_if_missing :failed_jobs, :tag
+    add_string_limit_if_missing :failed_jobs, :strand
+    add_string_limit_if_missing :failed_jobs, :source
+
+    readd_triggers
+  end
+
+  def drop_triggers
+    execute %{DROP TRIGGER delayed_jobs_before_insert_row_tr ON delayed_jobs}
+    execute %{DROP TRIGGER delayed_jobs_after_delete_row_tr ON delayed_jobs}
+  end
+
+  def readd_triggers
+    execute("CREATE TRIGGER delayed_jobs_before_insert_row_tr BEFORE INSERT ON delayed_jobs FOR EACH ROW WHEN (NEW.strand IS NOT NULL) EXECUTE PROCEDURE delayed_jobs_before_insert_row_tr_fn()")
+    execute("CREATE TRIGGER delayed_jobs_after_delete_row_tr AFTER DELETE ON delayed_jobs FOR EACH ROW WHEN (OLD.strand IS NOT NULL AND OLD.next_in_strand = 't') EXECUTE PROCEDURE delayed_jobs_after_delete_row_tr_fn()")
+  end
+
+  def add_string_limit_if_missing(table, column)
+    return if column_exists?(table, column, :string, limit: 255)
+    change_column table, column, :string, limit: 255
+  end
+end

data/lib/delayed/backend/base.rb

@@ -313,6 +313,9 @@ module Delayed
       rescue TypeError, LoadError, NameError => e
         raise DeserializationError,
           "Job failed to load: #{e.message}. Try to manually require the required file."
+      rescue ArgumentError, Psych::SyntaxError => e
+          raise DeserializationError,
+            "YAML parsing error: #{e.message}. Probably not recoverable."
       end
 
       def _yaml_deserialize(source)

data/lib/delayed/version.rb

@@ -1,3 +1,3 @@
 module Delayed
-  VERSION = "0.11.5"
+  VERSION = "0.11.6"
 end

data/spec/shared/shared_backend.rb

@@ -83,6 +83,11 @@ shared_examples_for 'a backend' do
     lambda { job.payload_object.perform }.should raise_error(Delayed::Backend::DeserializationError)
   end
 
+  it "should raise an DeserializationError when the handler is invalid YAML" do
+    job = Delayed::Job.new :handler => %{test: ""11"}
+    lambda { job.payload_object.perform }.should raise_error(Delayed::Backend::DeserializationError, /parsing error/)
+  end
+
   describe "find_available" do
     it "should not find failed jobs" do
       @job = create_job :attempts => 50

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inst-jobs
 version: !ruby/object:Gem::Version
-  version: 0.11.5
+  version: 0.11.6
 platform: ruby
 authors:
 - Tobias Luetke
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-11 00:00:00.000000000 Z
+date: 2016-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: after_transaction_commit
@@ -45,14 +45,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 3.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 3.2.0
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -278,6 +278,7 @@ files:
 - db/migrate/20150807133223_add_max_concurrent_to_jobs.rb
 - db/migrate/20151123210429_add_expires_at_to_jobs.rb
 - db/migrate/20151210162949_improve_max_concurrent.rb
+- db/migrate/20161206323555_add_back_default_string_limits_jobs.rb
 - lib/delayed/backend/active_record.rb
 - lib/delayed/backend/base.rb
 - lib/delayed/backend/redis/bulk_update.lua
@@ -360,7 +361,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.4
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Instructure-maintained fork of delayed_job