roqua-support 0.3.4 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 145ad35d4dd5cca40ea55d6e49ebca48dbaae528732db8f96ead023fd8ea0ce3
-  data.tar.gz: bf7690b534eae2d73a27b14af058d880df87268afc1ce9f92f746f1fb7145317
+  metadata.gz: 774c4b5c612eff02c808d6dac48cc599664c641deb96b97c923a6d293875a5b4
+  data.tar.gz: 0b2dd8489e5ec1a6216db145f511d7313776efe7ea07db0604f4bb4c7bfeb3e1
 SHA512:
-  metadata.gz: b05f497f8685ffdbc830db90e1cbe8f92ae52858e6119eb5bcaa7857a2bdfc6c37903314bd131f6776e9cb18a2f20535dda0f3ca41bffaa3e25eec0d18035b44
-  data.tar.gz: 996d2195a27618321ed0ef27f55a60ff8cfe7ee8b2c62f060c3ebaa0e73342fe1ddfa879cacafcac5c23ea495331e81810c59cdeea01725408107f08a93cd604
+  metadata.gz: d69bb02afb3adb3bdf1dd64a526466d7b24f8f8b068829a8ce9dfafa2930d29de30cff451fec7ff2d91c933e390c27b2addb336124dd257856c3dc6c510d3b56
+  data.tar.gz: 316fedffd3ede41f162fb6b294b409b446b204e89412d29dd7f6cc7d07af300e6b3eb5abc8471cea6970944aae74b443adfcef4a3a25843d3b82d40ca39d24e9

data/.gitlab-ci.yml

@@ -5,44 +5,28 @@ before_script:
   - if [ -x "$(command -v apk)" ]; then apk add sqlite-dev; fi
   - export BUNDLE_PATH=$CI_PROJECT_DIR/.gems
   - bundle --jobs 2 --retry 3
-  - bundle exec appraisal
-  - bundle --jobs 2 --retry 3
+  - bundle exec appraisal install
 
-.ruby_25: &ruby_25
-  image: registry.roqua.nl/roqua/docker-base-images:ruby-2.5
-  cache:
-    key: ruby_25
-    paths:
-      - .gems
 
-.ruby_26: &ruby_26
-  image: registry.roqua.nl/roqua/docker-base-images:ruby-2.6-builder
+.ruby_27: &ruby_27
+  image: registry.roqua.nl/roqua/docker-base-images:ruby-2.7-builder
   cache:
-    key: ruby_26
+    key: ruby_27
     paths:
       - .gems
 
-rails_51_ruby_25:
-  <<: *ruby_25
-  script:
-    - bundle exec appraisal rails51 bundle exec rspec
 
-rails_52_ruby_25:
-  <<: *ruby_25
+rails_52_ruby_27:
+  <<: *ruby_27
   script:
     - bundle exec appraisal rails52 bundle exec rspec
 
-rails_52_ruby_26:
-  <<: *ruby_26
-  script:
-    - bundle exec appraisal rails52 bundle exec rspec
-
-rails_60_ruby_25:
-  <<: *ruby_25
+rails_60_ruby_27:
+  <<: *ruby_27
   script:
     - bundle exec appraisal rails60 bundle exec rspec
 
-rails_60_ruby_26:
-  <<: *ruby_26
+rails_61_ruby_27:
+  <<: *ruby_27
   script:
-    - bundle exec appraisal rails60 bundle exec rspec
+    - bundle exec appraisal rails61 bundle exec rspec

data/Appraisals

@@ -1,16 +1,15 @@
-appraise "rails51" do
-  gem "activesupport", "~> 5.1.0"
-  gem 'sqlite3', '~> 1.3.6'
-end
-
 appraise "rails52" do
+  gem "active_interaction", "~> 3.0"
   gem "activesupport", "~> 5.2.0"
-  gem 'sqlite3', '~> 1.4'
   gem 'appsignal', '~> 2.9.8' # test each minor versions we support
 end
 
 appraise "rails60" do
   gem "activesupport", "~> 6.0.0"
-  gem 'sqlite3', '~> 1.4'
   gem 'appsignal', '~> 2.10.6'
 end
+
+appraise "rails61" do
+  gem "activesupport", "~> 6.1.0"
+  gem 'appsignal', '~> 3.0.0'
+end

data/CHANGELOG.md

@@ -1,3 +1,22 @@
+## 0.4.2
+
+* Make sort_by_alphanum stable for chunks that parse to the same int like 04 and 4.
+
+## 0.4.1
+
+* Add timezone to logging timestamps. Use milliseconds instead of microseconds precision. 
+
+## 0.4.0
+
+* Remove support for ruby 2.5 and 2.6
+* Add stripped_string type (attributes api)
+* Add support for appsignal v3
+* Add support for active_interaction v4
+
+## 0.3.5
+
+* Update AppSignal gauge for every minute the probe runs
+
 ## 0.3.4
 
 * gemspec and tests: Removed rails 4.2 and 5.0, remove ruby 2.3 and 2.4, added ruby 2.6, allow appsignal 2.10

data/Gemfile

@@ -4,11 +4,12 @@ source 'https://rubygems.org'
 gemspec
 
 gem 'appraisal'
-gem 'roqua_styleguide', git: 'https://gitlab.roqua.nl/roqua/styleguide.git'
+# fix so long as we support ruby 2.6 and ealier.
+gem 'roqua_styleguide', git: 'https://gitlab.roqua.nl/roqua/styleguide.git', ref: '9662550'
 
 group :test do
+  gem 'sqlite3'
   gem 'actionpack', '>= 4.0'
-  gem 'active_interaction', '~> 3.0'
   gem 'climate_control' # For ENV modification in specs
   gem 'combustion', '~> 1.1.1'
   gem 'fakefs', require: 'fakefs/safe'
@@ -17,4 +18,4 @@ group :test do
   gem 'rspec-instrumentation-matcher'
   gem 'rspec-rails'
   gem 'pry'
-end
+end

data/Gemfile.lock

@@ -1,18 +1,19 @@
 GIT
   remote: https://gitlab.roqua.nl/roqua/styleguide.git
-  revision: 6d98107ef10d2dc5a4a785a4b0977f100475e007
+  revision: 9662550201987db20c03e300eddb34d8ad035613
+  ref: 9662550
   specs:
     roqua_styleguide (0.0.8)
-      rubocop (~> 0.80.0)
+      rubocop (~> 0.90.0)
       rubocop-rails
 
 PATH
   remote: .
   specs:
-    roqua-support (0.3.4)
-      active_interaction (~> 3.0)
-      activesupport (>= 5.1, < 6.1)
-      appsignal (>= 2.9, < 2.11)
+    roqua-support (0.4.2)
+      active_interaction (>= 3.0, < 5.0)
+      activesupport (>= 5.2, < 6.2)
+      appsignal (>= 2.9, < 3.1)
       naught (~> 1.0)
       with_advisory_lock (~> 3.2)
 
@@ -32,8 +33,9 @@ GEM
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    active_interaction (3.8.2)
-      activemodel (>= 4, < 7)
+    active_interaction (4.1.0)
+      activemodel (>= 5, < 8)
+      activesupport (>= 5, < 8)
     activemodel (6.0.2.2)
       activesupport (= 6.0.2.2)
     activerecord (6.0.2.2)
@@ -49,9 +51,9 @@ GEM
       bundler
       rake
       thor (>= 0.14.0)
-    appsignal (2.10.6)
+    appsignal (3.0.19)
       rack
-    ast (2.4.0)
+    ast (2.4.2)
     builder (3.2.4)
     climate_control (0.2.0)
     coderay (1.1.2)
@@ -59,7 +61,7 @@ GEM
       activesupport (>= 3.0.0)
       railties (>= 3.0.0)
       thor (>= 0.14.6)
-    concurrent-ruby (1.1.6)
+    concurrent-ruby (1.1.8)
     crass (1.0.6)
     delayed_job (4.1.8)
       activesupport (>= 3.0, < 6.1)
@@ -83,9 +85,8 @@ GEM
     guard-rspec (4.2.10)
       guard (~> 2.1)
       rspec (>= 2.14, < 4.0)
-    i18n (1.8.2)
+    i18n (1.8.10)
       concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.4)
     listen (3.2.1)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
@@ -95,7 +96,7 @@ GEM
     lumberjack (1.2.4)
     method_source (1.0.0)
     mini_portile2 (2.4.0)
-    minitest (5.14.0)
+    minitest (5.14.4)
     naught (1.1.0)
     nenv (0.3.0)
     nokogiri (1.10.9)
@@ -103,13 +104,13 @@ GEM
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    parallel (1.19.1)
-    parser (2.7.1.2)
-      ast (~> 2.4.0)
+    parallel (1.20.1)
+    parser (3.0.0.0)
+      ast (~> 2.4.1)
     pry (0.13.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    rack (2.2.2)
+    rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
     rails-dom-testing (2.0.3)
@@ -128,6 +129,7 @@ GEM
     rb-fsevent (0.10.4)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
+    regexp_parser (2.1.1)
     responders (3.0.0)
       actionpack (>= 5.0)
       railties (>= 5.0)
@@ -156,36 +158,39 @@ GEM
       rspec-mocks (~> 3.9)
       rspec-support (~> 3.9)
     rspec-support (3.9.3)
-    rubocop (0.80.1)
-      jaro_winkler (~> 1.5.1)
+    rubocop (0.90.0)
       parallel (~> 1.10)
-      parser (>= 2.7.0.1)
+      parser (>= 2.7.1.1)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.7)
       rexml
+      rubocop-ast (>= 0.3.0, < 1.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 1.7)
-    rubocop-rails (2.5.2)
-      activesupport
+      unicode-display_width (>= 1.4.0, < 2.0)
+    rubocop-ast (0.8.0)
+      parser (>= 2.7.1.5)
+    rubocop-rails (2.9.1)
+      activesupport (>= 4.2.0)
       rack (>= 1.1)
-      rubocop (>= 0.72.0)
-    ruby-progressbar (1.10.1)
+      rubocop (>= 0.90.0, < 2.0)
+    ruby-progressbar (1.11.0)
     shellany (0.0.1)
+    sqlite3 (1.4.2)
     thor (1.0.1)
     thread_safe (0.3.6)
     timecop (0.9.1)
-    tzinfo (1.2.7)
+    tzinfo (1.2.9)
       thread_safe (~> 0.1)
-    unicode-display_width (1.6.1)
+    unicode-display_width (1.7.0)
     with_advisory_lock (3.2.0)
       activerecord (>= 3.2)
-    zeitwerk (2.3.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   actionpack (>= 4.0)
-  active_interaction (~> 3.0)
   appraisal
   bundler (~> 2.0)
   climate_control
@@ -201,6 +206,7 @@ DEPENDENCIES
   rspec (>= 2.12.0, < 4.0)
   rspec-instrumentation-matcher
   rspec-rails
+  sqlite3
   timecop
 
 BUNDLED WITH

data/README.md

@@ -104,6 +104,29 @@ class ApiAreaController < ApplicationController
   ...
 ```
 
+### Types
+
+In initializer:
+
+```ruby
+require 'roqua/type'
+```
+
+In models:
+
+```ruby
+# strip whitespace and nil if blank.
+attribute :firstname, :stripped_string
+# strip, but don't nil if blank.
+attribute :firstname, :stripped_string, allow_empty: true 
+```
+
+#### option 2
+
+`require 'roqua/type/stripped_string'`
+
+`attribute :firstname, Roqua::Type::StrippedString.new(allow_empty: true)`
+
 # Delayed Job activity monitoring
 
 To add monitoring of whether Delayed Job keeps picking up jobs or checking for new jobs, add the following line to an initializer:

data/gemfiles/rails52.gemfile

@@ -3,14 +3,14 @@
 source "https://rubygems.org"
 
 gem "appraisal"
-gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git"
+gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git", ref: "9662550"
+gem "active_interaction", "~> 3.0"
 gem "activesupport", "~> 5.2.0"
-gem "sqlite3", "~> 1.4"
 gem "appsignal", "~> 2.9.8"
 
 group :test do
+  gem "sqlite3"
   gem "actionpack", ">= 4.0"
-  gem "active_interaction", "~> 3.0"
   gem "climate_control"
   gem "combustion", "~> 1.1.1"
   gem "fakefs", require: "fakefs/safe"

data/gemfiles/rails60.gemfile

@@ -3,14 +3,13 @@
 source "https://rubygems.org"
 
 gem "appraisal"
-gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git"
+gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git", ref: "9662550"
 gem "activesupport", "~> 6.0.0"
-gem "sqlite3", "~> 1.4"
 gem "appsignal", "~> 2.10.6"
 
 group :test do
+  gem "sqlite3"
   gem "actionpack", ">= 4.0"
-  gem "active_interaction", "~> 3.0"
   gem "climate_control"
   gem "combustion", "~> 1.1.1"
   gem "fakefs", require: "fakefs/safe"

data/gemfiles/{rails51.gemfile → rails61.gemfile}

@@ -3,13 +3,13 @@
 source "https://rubygems.org"
 
 gem "appraisal"
-gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git"
-gem "activesupport", "~> 5.1.0"
-gem "sqlite3", "~> 1.3.6"
+gem "roqua_styleguide", git: "https://gitlab.roqua.nl/roqua/styleguide.git", ref: "9662550"
+gem "activesupport", "~> 6.1.0"
+gem "appsignal", "~> 3.0.0"
 
 group :test do
+  gem "sqlite3"
   gem "actionpack", ">= 4.0"
-  gem "active_interaction", "~> 3.0"
   gem "climate_control"
   gem "combustion", "~> 1.1.1"
   gem "fakefs", require: "fakefs/safe"

data/lib/roqua/core_ext/active_interaction/filters/date_time_as_unix_extension.rb

@@ -1,11 +1,11 @@
 # allow datetimes to be given as unix times for activeinteractions
 module RoquaDateTimeAsUnixFilterExtension
-  def cast(value, interaction)
+  def cast(value, interaction, **options)
     case value
     when Numeric, /^[0-9]+$/
       Time.at(value.to_i).to_datetime
     when ''
-      super(nil, interaction)
+      super(nil, interaction, **options)
     else
       super
     end
@@ -15,12 +15,12 @@ ActiveInteraction::DateTimeFilter.include RoquaDateTimeAsUnixFilterExtension
 
 # allow datetimes to be given as unix times as string
 module RoquaTimeAsUnixFilterExtension
-  def cast(value, interaction)
+  def cast(value, interaction, **options)
     case value
     when /^[0-9]+$/
       Time.at(value.to_i)
     when ''
-      super(nil, interaction)
+      super(nil, interaction, **options)
     else
       super
     end

data/lib/roqua/core_ext/enumerable/sort_by_alphanum.rb

@@ -21,29 +21,45 @@ module Enumerable
 
   private
 
-  def grouped_compare(a, b)
-    loop {
-      a_chunk, a = extract_alpha_or_number_group(a)
-      b_chunk, b = extract_alpha_or_number_group(b)
+  ALL_NUM = /\d+/
+  ALL_ALPHA = /[A-Za-z]+/
+  NON_ALPHANUM = /[^A-Za-z0-9]+/
 
-      ret = if a_chunk =~ /\d/ and b_chunk =~ /\d/
+  def grouped_compare(a, b)
+    a_scanner = StringScanner.new(a)
+    b_scanner = StringScanner.new(b)
+    # each loop has to do exactly 1 non-nil-scan on both scanners or return a non-zero value.
+    loop do
+      ret = \
+        if a_scanner.eos?
+          -1
+        elsif (a_chunk = a_scanner.scan(ALL_NUM))
+          if (b_chunk = b_scanner.scan(ALL_NUM))
+            if a_chunk.to_i != b_chunk.to_i
               a_chunk.to_i <=> b_chunk.to_i
-            else
+            else # 03 vs 3
               a_chunk <=> b_chunk
             end
-
-      return -1 if a_chunk == ''
+          elsif b_scanner.scan(ALL_ALPHA)
+            -1
+          else # NON_ALPHANUM
+            1
+          end
+        elsif (a_chunk = a_scanner.scan(ALL_ALPHA))
+          if (b_chunk = b_scanner.scan(ALL_ALPHA))
+            a_chunk <=> b_chunk
+          else # ALL_NUM or NON_ALPHANUM
+            1
+          end
+        else # NON_ALPHANUM
+          a_chunk = a_scanner.scan(NON_ALPHANUM)
+          if (b_chunk = b_scanner.scan(NON_ALPHANUM))
+            a_chunk <=> b_chunk
+          else
+            -1
+          end
+        end
       return ret if ret != 0
-    }
-  end
-
-  def extract_alpha_or_number_group(item)
-    matchdata = /([A-Za-z]+|[\d]+)/.match(item)
-
-    if matchdata.nil?
-      ["", ""]
-    else
-      [matchdata[0], item = item[matchdata.offset(0)[1] .. -1]]
     end
   end
 end

data/lib/roqua/logging/roqua_logging_railtie.rb

@@ -7,6 +7,8 @@ class RoquaLoggingRailtie < Rails::Railtie
     def configure
       Roqua.logger = ActiveSupport::Logger.new(output_stream).tap do |logger|
         logger.formatter = Logger::Formatter.new
+        # Set the datetime_format to include timezone and use milliseconds instead of microseconds
+        logger.formatter.datetime_format = '%Y-%m-%dT%H:%M:%S.%L%z '
       end
 
       require 'roqua/support/request_logger'

data/lib/roqua/scheduling/scheduler.rb

@@ -11,6 +11,8 @@ class Roqua::Scheduling::Scheduler
           raise ex if Rails.env.test?
         end
       end
+
+      Appsignal.increment_counter("scheduler.completed", 1)
     end
   end
 
@@ -27,7 +29,10 @@ class Roqua::Scheduling::Scheduler
   end
 
   def advisory_lock_name
-    "#{ActiveRecord::Base.connection_config[:database]}_cron_lock"
+    db_name = ActiveRecord::Base.respond_to?(:connection_db_config) \
+              ? ActiveRecord::Base.connection_db_config.database
+              : ActiveRecord::Base.connection_config[:database]
+    "#{db_name}_cron_lock"
   end
 
   private

data/lib/roqua/support/errors.rb

@@ -82,7 +82,10 @@ module Roqua
             current_transaction.set_tags(labels)
             current_transaction.add_exception(exception)
           else
-            Appsignal.send_error exception, labels, namespace
+            Appsignal.send_error exception do |transaction|
+              transaction.set_tags labels
+              transaction.set_namespace namespace
+            end
           end
         end
       end

data/lib/roqua/type/stripped_string.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+# Strips whitespace and make nil when string becomes empty.
+module Roqua
+  module Type
+    class StrippedString < ActiveModel::Type::ImmutableString
+      # taken from strip_attributes gem
+      # Unicode invisible and whitespace characters. The POSIX character class
+      # [:space:] corresponds to the Unicode class Z ("separator"). We also
+      # include the following characters from Unicode class C ("control"), which
+      # are spaces or invisible characters that make no sense at the start or end
+      # of a string:
+      #   U+180E MONGOLIAN VOWEL SEPARATOR
+      #   U+200B ZERO WIDTH SPACE
+      #   U+200C ZERO WIDTH NON-JOINER
+      #   U+200D ZERO WIDTH JOINER
+      #   U+2060 WORD JOINER
+      #   U+FEFF ZERO WIDTH NO-BREAK SPACE
+      MULTIBYTE_WHITE = "\u180E\u200B\u200C\u200D\u2060\uFEFF"
+      MULTIBYTE_SPACE = /[[:space:]#{MULTIBYTE_WHITE}]/.freeze
+
+      def initialize(allow_empty: false)
+        super()
+        @allow_empty = allow_empty
+      end
+
+      def cast(value)
+        return unless value
+        value = super(value).gsub(/\A#{MULTIBYTE_SPACE}+|#{MULTIBYTE_SPACE}+\z/, "").freeze
+        value.blank? && !@allow_empty ? nil : value
+      end
+    end
+  end
+end

data/lib/roqua/type.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require "active_model/type"
+
+require_relative "./type/stripped_string"
+
+ActiveRecord::Type.register(:stripped_string, Roqua::Type::StrippedString)

data/lib/roqua/validators/subset_validator.rb

@@ -6,7 +6,7 @@ class SubsetValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
     return unless value
     unless value.all? { |element| options.fetch(:of).include? element }
-      record.errors[attribute] << (options[:message] || I18n.t('errors.messages.subset'))
+      record.errors.add attribute, (options[:message] || I18n.t('errors.messages.subset'))
     end
   end
 end

data/lib/roqua-support/version.rb

@@ -1,5 +1,5 @@
 module Roqua
   module Support
-    VERSION = '0.3.4'.freeze
+    VERSION = '0.4.2'.freeze
   end
 end

data/roqua-support.gemspec

@@ -18,12 +18,12 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.required_ruby_version = '>= 2.5'
-  gem.add_dependency 'active_interaction', '~> 3.0'
-  gem.add_dependency 'activesupport', '>= 5.1', '< 6.1'
+  gem.required_ruby_version = '>= 2.7'
+  gem.add_dependency 'active_interaction', '>= 3.0', '< 5.0'
+  gem.add_dependency 'activesupport', '>= 5.2', '< 6.2'
   gem.add_dependency 'naught', '~> 1.0'
   gem.add_dependency 'with_advisory_lock', '~> 3.2'
-  gem.add_dependency 'appsignal', '>= 2.9', '< 2.11'
+  gem.add_dependency 'appsignal', '>= 2.9', '< 3.1'
 
   gem.add_development_dependency 'bundler', '~> 2.0'
   gem.add_development_dependency 'delayed_job_active_record'

data/spec/roqua/core_ext/enumerable/sort_by_alphanum_spec.rb

@@ -11,8 +11,18 @@ describe Enumerable do
       expect(input.sort_by_alphanum(&:reverse)).to eq ["004some10thing", "004some11thing", "3another"]
     end
 
+    it 'treats non-alphanum as lower than alpha and num' do
+      input = %w[b3a b{c bԘb]  # curlies are above alpha in utf-8, Ԙ is multi-byte
+      expect(input.sort_by_alphanum).to eq %w[b{c bԘb b3a]
+    end
+
     it 'compares number chunks as integers' do
       expect(%w(004 3).sort_by_alphanum).to eq %w(3 004)
     end
+
+    it 'sorts identical integers by asci'  do
+      input = %w[b4e b0004e b04e b004e]
+      expect(input.sort_by_alphanum).to  eq %w[b0004e b004e b04e b4e]
+    end
   end
 end

data/spec/roqua/logging/roqua_logging_railtie_spec.rb

@@ -27,6 +27,12 @@ Rspec.describe RoquaLoggingRailtie do
         ActiveSupport::Logger.logger_outputs_to?(Roqua.logger.logger, STDOUT)
       ).to be_truthy
     end
+
+    it 'logs using a datetimeformat including timezone' do
+      time = Time.zone.local(2021,10,10,16,13)
+      got = Roqua.logger.logger.formatter.call 'INFO', time, 'progname', '{}'
+      expect(got).to match(/I, \[2021-10-10T16:13:00.000\+0000 #\d+]  INFO -- progname: {}/)
+    end
   end
 
   context 'when RAILS_LOG_TO_STDOUT_USING_ROQUA_LOGGER is blank' do

data/spec/roqua/scheduling/scheduler_spec.rb

@@ -51,7 +51,7 @@ describe Roqua::Scheduling::Scheduler do
   end
 
   it 'generates a database specific advisory lock name' do
-    expect(ActiveRecord::Base.connection_config[:database]).to eql(':memory:')
+    # expect(ActiveRecord::Base.connection_config[:database]).to eql(':memory:')
     expect(subject.advisory_lock_name).to eql ':memory:_cron_lock'
   end
 
@@ -86,6 +86,7 @@ describe Roqua::Scheduling::Scheduler do
     end
 
     it 'counts task calls' do
+      expect(Appsignal).to receive(:increment_counter).with('scheduler.completed', 1)
       expect(Appsignal).to receive(:increment_counter).with('scheduler.run_task.completed', 1, task_name: 'hourly')
       subject.ping
     end

data/spec/roqua/support/errors_spec.rb

@@ -106,16 +106,21 @@ describe 'Error reporting' do
     end
 
     it 'defaults to a background job' do
-      expect(Appsignal).to receive(:send_error).with(exception, {}, Appsignal::Transaction::BACKGROUND_JOB)
+      transaction = Appsignal::Transaction.new(SecureRandom.uuid, Appsignal::Transaction::FRONTEND, nil)
+      expect(Appsignal).to receive(:send_error).with(exception).and_yield(transaction)
       Roqua::Support::Errors.report exception
+      expect(transaction.namespace).to eq Appsignal::Transaction::BACKGROUND_JOB
     end
 
     it 'it will send an error under the provided category' do
+      transaction = Appsignal::Transaction.new(SecureRandom.uuid, Appsignal::Transaction::FRONTEND, nil)
       expect(Appsignal)
         .to receive(:send_error)
-        .with(exception, {a: 'b'}, Appsignal::Transaction::HTTP_REQUEST)
+        .with(exception).and_yield(transaction)
 
       Roqua::Support::Errors.report exception, a: 'b', namespace: :web
+      expect(transaction.namespace).to eq Appsignal::Transaction::HTTP_REQUEST
+      expect(transaction.tags).to eq a: 'b'
     end
 
     describe 'when a current transaction is present' do

data/spec/roqua/type/stripped_string_spec.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'roqua/type/stripped_string'
+
+RSpec.describe Roqua::Type::StrippedString do
+  let(:cls) do
+    Class.new(ActiveRecord::Base) do
+      self.table_name = 'delayed_jobs'
+
+      attribute :locked_by, Roqua::Type::StrippedString.new
+      attribute :queue, Roqua::Type::StrippedString.new(allow_empty: true)
+    end
+  end
+
+  it "strips whitespace from outside and not inside" do
+    expect(described_class.new.cast("   some name   \n  \t \u180E\u200B\u200C\u200D\u2060\uFEFF ")).to eq "some name"
+  end
+
+  it "nils empty values by default" do
+    expect(described_class.new.cast(" \n \t ")).to eq nil
+  end
+
+  it "keeps value empty if allow_empty is true" do
+    expect(described_class.new(allow_empty: true).cast(" \n \t ")).to eq ""
+  end
+
+  it "works on active_record" do
+    instance = cls.new(locked_by: "  \n  \t ", queue: "  \n  \t ", cron: "  \n  \t ")
+    expect(instance.locked_by).to be_nil
+    expect(instance.queue).to eq ""
+    expect(instance.cron).to eq "  \n  \t "
+  end
+end

metadata

@@ -1,49 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: roqua-support
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.4.2
 platform: ruby
 authors:
 - Marten Veldthuis
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-06 00:00:00.000000000 Z
+date: 2022-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_interaction
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
 - !ruby/object:Gem::Dependency
   name: naught
   requirement: !ruby/object:Gem::Requirement
@@ -81,7 +87,7 @@ dependencies:
         version: '2.9'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '3.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -91,7 +97,7 @@ dependencies:
         version: '2.9'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -188,9 +194,9 @@ files:
 - README.md
 - Rakefile
 - circle.yml
-- gemfiles/rails51.gemfile
 - gemfiles/rails52.gemfile
 - gemfiles/rails60.gemfile
+- gemfiles/rails61.gemfile
 - lib/roqua-support.rb
 - lib/roqua-support/railtie.rb
 - lib/roqua-support/version.rb
@@ -224,6 +230,8 @@ files:
 - lib/roqua/support/request_logger.rb
 - lib/roqua/support/stats.rb
 - lib/roqua/support/stats/hosted_graphite_backend.rb
+- lib/roqua/type.rb
+- lib/roqua/type/stripped_string.rb
 - lib/roqua/validators/subset_validator.rb
 - roqua-support.gemspec
 - spec/internal/config/balancer_state
@@ -250,6 +258,7 @@ files:
 - spec/roqua/support/request_logger_spec.rb
 - spec/roqua/support/stats_spec.rb
 - spec/roqua/support_spec.rb
+- spec/roqua/type/stripped_string_spec.rb
 - spec/roqua/validators/subset_validator_spec.rb
 - spec/spec_helper.rb
 - styleguide/ruby/rubocop.yml
@@ -257,7 +266,7 @@ homepage: https://github.com/roqua/roqua-support
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -265,15 +274,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Helper objects and proxies used by a lot of RoQua applications
 test_files:
@@ -301,5 +310,6 @@ test_files:
 - spec/roqua/support/request_logger_spec.rb
 - spec/roqua/support/stats_spec.rb
 - spec/roqua/support_spec.rb
+- spec/roqua/type/stripped_string_spec.rb
 - spec/roqua/validators/subset_validator_spec.rb
 - spec/spec_helper.rb