gouda 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8d5d39b730d429fe834fc88ecfe67e3a1bad397022715f6f52a1aa5e050df8b
-  data.tar.gz: b6cb85e539a387f75e3e84ffe756d54a991755dbd3c4335d67b476d70770eb97
+  metadata.gz: cbebec9ae881152e66d8a8e763d74e2314c6694cc46a1b547323bb4fe01fd505
+  data.tar.gz: f9836ba9594cf2473485efbe6be8dd6b001e0e514315cb84170a62d525adbaf3
 SHA512:
-  metadata.gz: 919a70fddf01f87880289e977b853aad5b01470e4f9b46ef1e8c5867ee50c4751f1ccd6ddc05146c40af83f8c82106d980feb93e022c4871114d7bb01361cee5
-  data.tar.gz: 466fb4a016ff74b481b936445ea66ed85981520147831b4cb11ec08446b7e7f7962974041f153448248558190ff0b0afb427b37e21af8b915337885ceb37f250
+  metadata.gz: 2bf05ef7dcfe8cc682f54da18f8ad8a415f5db3f103928188ad04ab714f4a2799b4998b25c5d53432776d4f913e22fe0a761bd1fe11ea2eb17763f89201cb809
+  data.tar.gz: 636a2aa356b4fed48dece844dec2f26d46a430d6e4a0d956c56d5c5121d30f8d3fb8f87174b82e4d7ab6a58f3e0bcd99d3e07c18c631390cc6233b8cac2d48ba

data/.ruby-version

@@ -1 +1 @@
-3.3.1
+3.3.3

data/CHANGELOG.md

@@ -15,3 +15,11 @@
 ## [0.1.3] - 2023-06-11
 
 - Allow the Rails app to boot even if there is no database yet
+
+## [0.1.4] - 2023-06-14
+
+- Rescue NoDatabaseError at scheduler update.
+- Include tests in gem, for sake of easier debugging.
+- Reduce logging in local test runs.
+- Bump local ruby version to 3.3.3
+

data/gouda.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |spec|
   spec.metadata["changelog_uri"] = "https://github.com/cheddar-me/gouda/CHANGELOG.md"
 
   spec.files = Dir.chdir(File.expand_path(__dir__)) do
-    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{\A(?:test|spec|features)/}) }
+    `git ls-files -z`.split("\x0")
   end
 
   spec.add_dependency "activerecord", "~> 7"

data/lib/gouda/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Gouda
-  VERSION = "0.1.3"
+  VERSION = "0.1.4"
 end

data/test/gouda/concurrency_extension_test.rb

@@ -0,0 +1,160 @@
+# frozen_string_literal: true
+
+require "gouda/test_helper"
+
+class GoudaConcurrencyExtensionTest < ActiveSupport::TestCase
+  include AssertHelper
+  class TestJobWithoutConcurrency < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+  end
+
+  class TestJobWithPerformConcurrency < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+    gouda_control_concurrency_with(perform_limit: 1)
+
+    def perform(*args)
+    end
+  end
+
+  setup do
+    @adapter ||= Gouda::Adapter.new
+    Gouda::Railtie.initializers.each(&:run)
+  end
+
+  test "gouda_control_concurrency_with with just perform_limit sets a perform concurrency key and no enqueue concurrency key" do
+    job = TestJobWithPerformConcurrency.new
+    assert_nil job.enqueue_concurrency_key
+    assert job.execution_concurrency_key
+  end
+
+  test "gouda_control_concurrency_with with just perform_limit makes the perform concurrency key dependent on job params" do
+    job1 = TestJobWithPerformConcurrency.new(1, 2, :something)
+    assert job1.execution_concurrency_key
+
+    job2 = TestJobWithPerformConcurrency.new(1, 2, :something)
+    assert_equal job2.execution_concurrency_key, job1.execution_concurrency_key
+
+    job3 = TestJobWithPerformConcurrency.new(1, 2, :something_else)
+    refute_equal job3.execution_concurrency_key, job1.execution_concurrency_key
+  end
+
+  class TestJobWithCommonConcurrency < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+    gouda_control_concurrency_with(total_limit: 1)
+
+    def perform(*args)
+    end
+  end
+
+  test "gouda_control_concurrency_with with total_limit sets a perform concurrency key and an enqueue concurrency key" do
+    job = TestJobWithCommonConcurrency.new
+    assert job.enqueue_concurrency_key
+    assert job.execution_concurrency_key
+  end
+
+  test "gouda_control_concurrency_with with total_limit makes the perform concurrency key dependent on job params" do
+    job1 = TestJobWithCommonConcurrency.new(1, 2, :something)
+    assert job1.execution_concurrency_key
+
+    job2 = TestJobWithCommonConcurrency.new(1, 2, :something)
+    assert_equal job2.execution_concurrency_key, job1.execution_concurrency_key
+
+    job3 = TestJobWithCommonConcurrency.new(1, 2, :something_else)
+    refute_equal job3.execution_concurrency_key, job1.execution_concurrency_key
+  end
+
+  test "gouda_control_concurrency_with with total_limit makes the enqueue concurrency key dependent on job params" do
+    job1 = TestJobWithCommonConcurrency.new(1, 2, :something)
+    assert job1.enqueue_concurrency_key
+
+    job2 = TestJobWithCommonConcurrency.new(1, 2, :something)
+    assert_equal job2.enqueue_concurrency_key, job1.enqueue_concurrency_key
+
+    job3 = TestJobWithCommonConcurrency.new(1, 2, :something_else)
+    refute_equal job3.enqueue_concurrency_key, job1.enqueue_concurrency_key
+  end
+
+  class TestJobWithEnqueueConcurrency < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+    gouda_control_concurrency_with(enqueue_limit: 1)
+
+    def perform(*args)
+    end
+  end
+
+  test "gouda_control_concurrency_with with enqueue_limit sets a perform concurrency key and an enqueue concurrency key" do
+    job = TestJobWithEnqueueConcurrency.new
+    assert job.enqueue_concurrency_key
+    assert_nil job.execution_concurrency_key
+  end
+
+  test "gouda_control_concurrency_with with enqueue_limit makes the enqueue concurrency key dependent on job params" do
+    job1 = TestJobWithEnqueueConcurrency.new(1, 2, :something)
+    assert job1.enqueue_concurrency_key
+
+    job2 = TestJobWithEnqueueConcurrency.new(1, 2, :something)
+    assert_equal job2.enqueue_concurrency_key, job1.enqueue_concurrency_key
+
+    job3 = TestJobWithEnqueueConcurrency.new(1, 2, :something_else)
+    refute_equal job3.enqueue_concurrency_key, job1.enqueue_concurrency_key
+  end
+
+  class TestJobWithCustomKey < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+    gouda_control_concurrency_with total_limit: 1, key: "42"
+  end
+
+  test "can use an arbitrary string as the custom key" do
+    job = TestJobWithCustomKey.new
+    assert_equal "42", job.enqueue_concurrency_key
+    assert_equal "42", job.execution_concurrency_key
+  end
+
+  class TestJobWithCustomKeyProc < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+    gouda_control_concurrency_with total_limit: 1, key: -> { @ivar }
+
+    def initialize(...)
+      super
+      @ivar = "123"
+    end
+  end
+
+  test "can use a proc that gets instance_exec'd as the custom key" do
+    job = TestJobWithCustomKeyProc.new
+    assert_equal "123", job.enqueue_concurrency_key
+    assert_equal "123", job.execution_concurrency_key
+  end
+
+  class TestJobWithWithUnconfiguredConcurrency < ActiveJob::Base
+    self.queue_adapter = Gouda::Adapter.new
+    include Gouda::ActiveJobExtensions::Concurrency
+  end
+
+  test "validates arguments" do
+    assert_raises ArgumentError do
+      TestJobWithWithUnconfiguredConcurrency.gouda_control_concurrency_with
+    end
+
+    assert_raises ArgumentError do
+      TestJobWithWithUnconfiguredConcurrency.gouda_control_concurrency_with total_limit: 2
+    end
+
+    assert_raises ArgumentError do
+      TestJobWithWithUnconfiguredConcurrency.gouda_control_concurrency_with perform_limit: 2
+    end
+
+    assert_raises ArgumentError do
+      TestJobWithWithUnconfiguredConcurrency.gouda_control_concurrency_with enqueue_limit: 2
+    end
+
+    assert_raises ArgumentError do
+      TestJobWithWithUnconfiguredConcurrency.gouda_control_concurrency_with total_limit: 2, bollocks: 4
+    end
+  end
+end