service_skeleton 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2eb043ee79208c4b1b457b1b9deb2b5901119339c9789d8a6d511870bc038e46
-  data.tar.gz: 6848edcb6fe369e5a6ce0dcf823780ee8047f8953b40ea261e03aa0f47314d27
+  metadata.gz: f06ab46da1ee2c8366933e34efe02686cfca1758cff0667fb05b6ef5a42e9db6
+  data.tar.gz: 17cc7ba27c8cd63f6e74831e69da68c860803af15e77eb6f4a52686754107138
 SHA512:
-  metadata.gz: 43fe6a4e0ad9adb50cd78c09c41063df614a0116c194283710660b24f2877994038bb818e35ad78eb7b72da62c3ec7e75646fc2bfae5c6f85ef5fdd7a5289dba
-  data.tar.gz: 9aacd383f2d867a51b19adc8c1d501ad383497d13a72de0b8fa9992301b029cd23e60aaa3c72511e98f240ed060dd1dab6b8b38a89a9de3cce19a40c8c696f39
+  metadata.gz: c3137b6133e59bd2c9aa990bd85216b6acdf09a1d6174ae0fa06a1c2ff55e898f266f131a69457dd39f363fcdef35d67d0f9fd1e9255f6a8e9b4ebdd2f4a5ed9
+  data.tar.gz: dbba001733bcf0717673335d1ff52d6eb33b0a88b602c7f3be2d378addb37c0da585fba5c15d6ddff106f0dd69663935d541583ea521376b8136711eef53f3b8

data/.github/workflows/ci.yml

@@ -4,23 +4,41 @@ on:
   pull_request:
   push:
     branches:
-      - master
       - main
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: "3.2"
+          bundler-cache: true
+
+      - name: Lint
+        run: bundle exec rubocop
+
   build:
     runs-on: ubuntu-latest
 
     strategy:
+      fail-fast: false
+
       matrix:
         ruby:
-          - 2.5
-          - 2.6
-          - 2.7
-          - 3.0
+          - "2.5"
+          - "2.6"
+          - "2.7"
+          - "3.0"
+          - "3.1"
+          - "3.2"
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - name: Setup ruby
         uses: ruby/setup-ruby@v1
@@ -28,22 +46,19 @@ jobs:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
 
-      - name: Lint
-        run: bundle exec rubocop
-
       - name: Tests
         run: bundle exec rake test
 
   publish:
-    if: github.event_name == 'push' && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/master')
-    needs: build
+    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
+    needs: [lint, build]
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - name: Release Gem
-        uses: discourse/publish-rubygems-action@v2-beta
+        uses: discourse/publish-rubygems-action@v3
         env:
           RUBYGEMS_API_KEY: ${{ secrets.RUBYGEMS_API_KEY }}
           GIT_EMAIL: team@discourse.org

data/.rubocop.yml

@@ -8,4 +8,4 @@ inherit_mode:
 RSpec/MessageSpies:
   EnforcedStyle: receive
   Exclude:
-    - "ultravisor/**/*"
+    - "ultravisor/**/*"

data/service_skeleton.gemspec

@@ -3,7 +3,7 @@
 Gem::Specification.new do |s|
   s.name = "service_skeleton"
 
-  s.version = '2.1.0'
+  s.version = '2.2.0'
 
   s.platform = Gem::Platform::RUBY
 
@@ -39,7 +39,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rake'
   s.add_development_dependency 'redcarpet'
   s.add_development_dependency 'rspec'
-  s.add_development_dependency 'rubocop-discourse', '~> 2.4.1'
+  s.add_development_dependency 'rubocop-discourse', '~> 3.4.1'
   s.add_development_dependency 'simplecov'
   s.add_development_dependency 'yard'
 end

data/ultravisor/lib/ultravisor/child/call_receiver.rb

@@ -11,4 +11,5 @@ class Ultravisor::Child::CallReceiver < BasicObject
     @blk.call(callback)
     rv_q.pop.tap { |rv| ::Kernel.raise ::Ultravisor::ChildRestartedError.new if rv == rv_fail }
   end
+  ruby2_keywords :method_missing if respond_to?(:ruby2_keywords, true)
 end

data/ultravisor/lib/ultravisor/child/cast_receiver.rb

@@ -8,4 +8,5 @@ class Ultravisor::Child::CastReceiver < BasicObject
     castback = ::Ultravisor::Child::Cast.new(name, args, blk)
     @blk.call(castback)
   end
+  ruby2_keywords :method_missing if respond_to?(:ruby2_keywords, true)
 end

data/ultravisor/spec/ultravisor/child/cast_spec.rb

@@ -106,6 +106,30 @@ describe Ultravisor::Child do
 
         expect { child.cast.to_s }.to_not raise_error
       end
+
+      context "with keyword arguments" do
+        class Child
+          def run
+          end
+
+          def process
+            process_castcall
+          end
+
+          def kwarg(message:)
+            raise "#{message} was passed"
+          end
+        end
+
+        let(:args) do
+          { id: :child, klass: Child, method: :run, enable_castcall: true, access: :unsafe }
+        end
+
+        it "forwards them" do
+          child.cast.kwarg(message: "hey")
+          expect { child.unsafe_instance.process }.to raise_error("hey was passed")
+        end
+      end
     end
   end
 end

data/ultravisor/spec/ultravisor/child/restart_delay_spec.rb

@@ -10,7 +10,7 @@ describe Ultravisor::Child do
   let(:mock_class) { Class.new.tap { |k| k.class_eval { def run; end } } }
 
   describe "#restart_delay" do
-    context "by default" do
+    context "with default args" do
       let(:args) { base_args }
 
       it "returns the default delay" do

data/ultravisor/spec/ultravisor/child/shutdown_spec.rb

@@ -77,7 +77,7 @@ describe Ultravisor::Child do
           expect(@thread).to have_received(:join).with(0.05)
         end
 
-        context "the worker doesn't finish quickly enough" do
+        context "when the worker doesn't finish quickly enough" do
           before(:each) do
             allow(mock_instance).to receive(:run) { sleep 15 }
           end

data/ultravisor/spec/ultravisor/child/unsafe_instance_spec.rb

@@ -10,7 +10,7 @@ describe Ultravisor::Child do
   let(:child) { Ultravisor::Child.new(**args) }
 
   describe "#unsafe_instance" do
-    context "by default" do
+    context "with default args" do
       it "explodes" do
         expect { child.unsafe_instance }.to raise_error(Ultravisor::ThreadSafetyError)
       end

data/ultravisor/spec/ultravisor/run_spec.rb

@@ -69,13 +69,13 @@ describe Ultravisor do
         ultravisor.run
       end
 
-      context "that terminates" do
+      context "when the child terminates" do
         before(:each) do
           allow(ultravisor.instance_variable_get(:@queue)).to receive(:pop).and_return(child, :shutdown)
           allow(ultravisor).to receive(:sleep)
         end
 
-        context "within the limits of its restart policy" do
+        context "when in the limits of its restart policy" do
           it "spawns the child again" do
             expect(child).to receive(:spawn).exactly(:twice)
 
@@ -89,7 +89,7 @@ describe Ultravisor do
           end
         end
 
-        context "too often for its restart policy" do
+        context "when it terminates too often for its restart policy" do
           before(:each) do
             allow(child).to receive(:restart?).and_raise(Ultravisor::BlownRestartPolicyError)
             allow(logger).to receive(:error)

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: service_skeleton
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Matt Palmer
 - Sam Saffron
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-25 00:00:00.000000000 Z
+date: 2023-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: frankenstein
@@ -199,14 +199,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.1
+        version: 3.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.1
+        version: 3.4.1
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -321,7 +321,7 @@ files:
 homepage: https://github.com/discourse/service_skeleton
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -336,8 +336,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: The bare bones of a service
 test_files: []