SOL-127: Add Ruby 3 support to gem

.github/CODEOWNERS

@@ -0,0 +1,3 @@
+# Team Sol is taking ownership over delayed job infrastructure.
+# This will notify us of any requested changes to anything
+* @cultureamp/team-sol

.github/pull_request_checklist.md

@@ -0,0 +1,27 @@
+# Pull Request Checklist Details
+
+Here are some useful questions to ask before merging a Pull Request.
+
+## [Security](https://cultureamp.atlassian.net/wiki/spaces/SEC/pages/963641883/Security+Partnership+Process)
+- [Do we need an AppSec review?](https://cultureamp.atlassian.net/servicedesk/customer/portal/5/group/39/create/461)
+- Have we modified authentication, authorization, filtering?
+- Could there be any PII leaks through tracking or logging?
+
+## Acceptance Criteria
+- What is the acceptance criteria?
+- How can we verify the PR has the desired effect?
+- Are we sure there are no undesired side effects?
+
+## Tests
+- Do we have appropriate tests?
+- How much of the new code do they cover?
+- What gaps do you need to call out?
+
+## Logging
+- Are we logging the right things?
+- Are we using structured logging?
+
+## Involve Others
+- [Do we need a Change Management Request?](https://cultureamp.atlassian.net/servicedesk/customer/portal/30)
+- Does the tech lead need to see this?
+- Do other teams need to see this?

.github/pull_request_template.md

@@ -0,0 +1,35 @@
+<!--
+
+ Just fill in only the relevant sections unless you are linking to trello/jira
+
+-->
+
+## Purpose
+<!--
+What is the objective of the Pull Request?
+-->
+
+## Context
+<!--
+Why are we making this change? 
+Is there jira/trello, bugsnag or slack link you can put here?
+If it was a bug, what caused it?
+-->
+
+## Changes
+<!--
+What were the changes you made? (Omit section if this is clear from the commits already.)
+-->
+
+## Verification
+<!--
+Any exceptions to testing norms?
+Where should QA focus?
+-->
+
+## Checklist
+- [Security](pull_request_checklist.md#security)
+- [Acceptance Criteria](pull_request_checklist.md#acceptance%20criteria)
+- [Tests](pull_request_checklist.md#tests)
+- [Logging](pull_request_checklist.md#logging)
+- [Involve Others](pull_request_checklist.md#involve%20others)

.github/teams.yml

@@ -0,0 +1,34 @@
+Krypton:
+  - '@ater1'
+  - '@braden-m'
+  - '@caitlinmwillington'
+  - '@chy-m'
+  - '@dan-lennox'
+  - '@jchillemi'
+  - '@paulofla'
+  - '@Xuester'
+Mercury:
+  - '@dNitza'
+  - '@Gwash3189'
+  - '@markreid'
+  - '@RohanSharma4'
+  - '@zoe0119'
+  - '@Moskvyak'
+  - '@felixsebastian'
+  - '@mike-kent-ca'
+  - '@dscultureamp'
+Sol:
+  - '@juliohr'
+  - '@dabarrell'
+  - '@m-dwyer'
+  - '@princeprescott'
+  - '@deepsainath'
+Earthlings:
+  - '@AndrewMcCraeCA'
+  - '@quangkhoa'
+  - '@sumandeep0808'
+  - '@Confidenceman02'
+  - '@fugu-chop'
+  - '@dalebaldwin'
+  - '@talbet'
+

.github/workflows/rubocop.yml

@@ -8,10 +8,10 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Ruby 2.7
+    - name: Set up Ruby 3.0
       uses: ruby/setup-ruby@v1
       with:
-        ruby-version: 2.7
+        ruby-version: 3.0
     - name: Generate lockfile for cache key
       run: bundle lock
     - name: Cache gems

.github/workflows/ruby.yml

@@ -0,0 +1,19 @@
+name: Tests
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 3.0
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rspec

.github/workflows/team-labeler.yml

@@ -0,0 +1,11 @@
+name: Add team label to PR
+on: pull_request
+
+jobs:
+  label:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: JulienKode/[email protected]
+      if: github.event.action == 'opened'
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}

.rubocop.yml

@@ -13,10 +13,7 @@ AllCops:
     - 'Gemfile'
     - 'Rakefile'
     - 'delayed_job.gemspec'
-  TargetRubyVersion: 2.1
-
-RedundantBlockCall:
-  Enabled: false
+  TargetRubyVersion: 3.0
 
 BlockLength:
   Enabled: false
@@ -133,9 +130,6 @@ Style/SymbolArray:
 SymbolProc:
   Enabled: false
 
-TrailingCommaInLiteral:
-  Enabled: false
-
 TrailingCommaInArguments:
   Enabled: false
 
@@ -144,3 +138,15 @@ YAMLLoad:
 
 ZeroLengthPredicate:
   Enabled: false
+
+Style/IfUnlessModifier:
+  Enabled: false
+
+Style/CaseLikeIf:
+  Enabled: false
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+Migration/DepartmentName:
+  Enabled: false

Gemfile

@@ -70,7 +70,7 @@ group :test do
 end
 
 group :rubocop do
-  gem 'rubocop', '>= 0.25', '< 0.49'
+  gem 'rubocop', '~>1.27.0'
 end
 
 gemspec

catalog-info-component.yaml

@@ -0,0 +1,19 @@
+apiVersion: backstage.io/v1alpha1
+kind: Component
+metadata:
+  name: delayed_job
+  description: Forked delayed_job gem with ruby 3 support
+  links:
+    - title: Github
+      url: https://github.com/cultureamp/delayed_job
+  tags:
+    - users-internal
+    - camp-engagement
+    - data-none
+  annotations:
+    github.com/project-slug: cultureamp/delayed_job
+    github.com/team-slug: cultureamp/sol
+spec:
+  type: library
+  owner: sol
+  lifecycle: development

catalog-info.yaml

@@ -0,0 +1,9 @@
+apiVersion: backstage.io/v1alpha1
+kind: Location
+metadata:
+  name: delayed_job-location
+  tags:
+    - camp-engagement
+spec:
+  targets:
+    - ./catalog-info-component.yaml # this is your main component file

delayed_job.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |spec|
   spec.test_files     = Dir.glob('spec/**/*')
   spec.version        = '4.1.10'
   spec.metadata       = {
-    'changelog_uri'   => 'https://github.com/collectiveidea/delayed_job/blob/master/CHANGELOG.md',
+    'changelog_uri' => 'https://github.com/collectiveidea/delayed_job/blob/master/CHANGELOG.md',
     'bug_tracker_uri' => 'https://github.com/collectiveidea/delayed_job/issues',
     'source_code_uri' => 'https://github.com/collectiveidea/delayed_job'
   }

lib/delayed/message_sending.rb

@@ -7,8 +7,8 @@ def initialize(payload_class, target, options)
     end
 
     # rubocop:disable MethodMissing
-    def method_missing(method, *args)
-      Job.enqueue({:payload_object => @payload_class.new(@target, method.to_sym, args)}.merge(@options))
+    def method_missing(method, *args, **kwargs)
+      Job.enqueue({:payload_object => @payload_class.new(@target, method.to_sym, args, kwargs)}.merge(@options))
     end
     # rubocop:enable MethodMissing
   end

lib/delayed/performable_mailer.rb

@@ -3,7 +3,7 @@
 module Delayed
   class PerformableMailer < PerformableMethod
     def perform
-      mailer = object.send(method_name, *args)
+      mailer = super
       mailer.respond_to?(:deliver_now) ? mailer.deliver_now : mailer.deliver
     end
   end