twiglet 3.0.8 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aadbe159aee695cfd454e0f006ab5eab2b3c06d01ecaa80d3f6b1b69c0d21d5e
-  data.tar.gz: 53907b03fb29af1fe41d8262619a6d5c84dab554b4e309d7336d969c11600a44
+  metadata.gz: 6970a1b98fbd77737c3742429691e3c03c7966e894e2f6a523824df785941858
+  data.tar.gz: 3a878416f432a4c9e38b0299afb6204fd2202168646d028298d40ab2c6cc7746
 SHA512:
-  metadata.gz: d5e68a4b343c861c7236e4a301365d738221bbe5eb0b23426e3cba84fad2358046d1b868bef5248c1c3398f4c2cfd132445259b7db976937024f73663699c2d4
-  data.tar.gz: 28b619fd981103d5baea53d07e032e52e21cf88e84b5229d1ca1ff61cedc284d6825ebf26fb977f060d3af4c34f276b8a69c733e66321952a726d452dc82164b
+  metadata.gz: 8a2e0c5bd321f495f4cf2c5d69a8caaff889e352305c5a900c1c7415e8637d0127729c91bd030353c1dd3b56e9c5623880f46849d3ff3eac1d502c3df8752c54
+  data.tar.gz: 0076c02f4070de277e488278f9f858d529b0ab39d8f8c07bc59a9d583c40421f0231b769a7ffb6a6ad1b422ed17f22ee4cd2a7eb68e436dc3b2b6c70c0246c8e

data/.github/workflows/{version-update.yml → dobby-actions.yml}

@@ -1,4 +1,4 @@
-name: "version update action"
+name: "dobby action"
 on:
   issue_comment:
     types: [created]
@@ -8,8 +8,9 @@ jobs:
     if: startsWith(github.event.comment.body, '/dobby')
 
     steps:
-      - name: 'bump version'
-        uses: simplybusiness/dobby@v1.0.0
+      - name: bump version
+        uses: simplybusiness/dobby@v2.0.0
         env:
-          ACCESS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          DOBBY_APP_ID: ${{ secrets.DOBBY_APP_ID }}
+          DOBBY_PRIVATE_KEY: ${{ secrets.DOBBY_PRIVATE_KEY }}
           VERSION_FILE_PATH: lib/twiglet/version.rb

data/Gemfile

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
 source 'https://rubygems.org'
-group :development, :test do
-  gem 'minitest'
-  gem 'rake'
-  gem 'simplecov', '0.17.1'
-  gem 'simplycop'
-end
+
+gemspec

data/lib/twiglet/formatter.rb

@@ -7,18 +7,21 @@ module Twiglet
     Hash.include HashExtensions
 
     def initialize(service_name,
-                   default_properties: {},
+                   validator:, default_properties: {},
                    now: -> { Time.now.utc })
       @service_name = service_name
       @now = now
       @default_properties = default_properties
+      @validator = validator
 
       super()
     end
 
     def call(severity, _time, _progname, msg)
       level = severity.downcase
-      log(level: level, message: Message.new(msg))
+      message = Message.new(msg)
+      @validator.validate(message)
+      log(level: level, message: message)
     end
 
     private

data/lib/twiglet/logger.rb

@@ -2,10 +2,10 @@
 
 require 'logger'
 require 'time'
-require 'json'
 require_relative 'formatter'
 require_relative '../hash_extensions'
 require_relative 'message'
+require_relative 'validator'
 
 module Twiglet
   class Logger < ::Logger
@@ -13,23 +13,35 @@ module Twiglet
 
     def initialize(
       service_name,
-      default_properties: {},
-      now: -> { Time.now.utc },
-      output: $stdout,
-      level: Logger::DEBUG
+      **args
     )
       @service_name = service_name
-      @now = now
-      @output = output
-      @level = level
+      default_properties = args.delete(:default_properties) || {}
+      @args = args
+
+      now = args.fetch(:now, -> { Time.now.utc })
+      output = args.fetch(:output, $stdout)
+      level = args.fetch(:level, Logger::DEBUG)
+      validation_schema = args.fetch(:validation_schema, File.read("#{__dir__}/validation_schema.json"))
 
       raise 'Service name is mandatory' \
         unless service_name.is_a?(String) && !service_name.strip.empty?
 
-      formatter = Twiglet::Formatter.new(service_name, default_properties: default_properties, now: now)
+      @validator = Validator.new(validation_schema)
+
+      formatter = Twiglet::Formatter.new(
+        service_name,
+        default_properties: default_properties,
+        now: now,
+        validator: @validator
+      )
       super(output, formatter: formatter, level: level)
     end
 
+    def configure_validation_error_response(&block)
+      @validator.custom_error_handler = block
+    end
+
     def error(message = nil, error = nil, &block)
       if error
         error_fields = {
@@ -48,10 +60,7 @@ module Twiglet
     def with(default_properties)
       Logger.new(
         @service_name,
-        default_properties: default_properties,
-        now: @now,
-        output: @output,
-        level: @level
+        **@args.merge(default_properties: default_properties)
       )
     end
 

data/lib/twiglet/message.rb

@@ -10,18 +10,6 @@ module Twiglet
       else
         super(msg)
       end
-
-      validate!
-    end
-
-    private
-
-    def validate!
-      raise 'Message must be initialized with a String or a non-empty Hash' if empty?
-
-      raise 'Log object must have a \'message\' property' unless self[:message]
-
-      raise 'The \'message\' property of the log object must not be empty' if self[:message].strip.empty?
     end
   end
 end

data/lib/twiglet/validation_schema.json

@@ -0,0 +1,10 @@
+{
+  "type": "object",
+  "required": ["message"],
+  "properties": {
+    "message": {
+      "type": "string",
+      "minLength": 1
+    }
+  }
+}

data/lib/twiglet/validator.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'json-schema'
+require 'json'
+
+module Twiglet
+  class Validator
+    attr_accessor :custom_error_handler
+
+    def initialize(schema)
+      @schema = JSON.parse(schema)
+      @custom_error_handler = ->(e) { raise e }
+    end
+
+    def validate(message)
+      JSON::Validator.validate!(@schema, message)
+    rescue JSON::Schema::ValidationError => e
+      custom_error_handler.call(e)
+    end
+  end
+end

data/lib/twiglet/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Twiglet
-  VERSION = '3.0.8'
+  VERSION = '3.2.0'
 end

data/test/formatter_test.rb

@@ -3,11 +3,12 @@
 require 'minitest/autorun'
 require 'json'
 require_relative '../lib/twiglet/formatter'
+require_relative '../lib/twiglet/validator'
 
 describe Twiglet::Formatter do
   before do
     @now = -> { Time.utc(2020, 5, 11, 15, 1, 1) }
-    @formatter = Twiglet::Formatter.new('petshop', now: @now)
+    @formatter = Twiglet::Formatter.new('petshop', now: @now, validator: Twiglet::Validator.new({}.to_json))
   end
 
   it 'initializes an instance of a Ruby Logger Formatter' do

data/test/logger_test.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'minitest/autorun'
+require 'minitest/mock'
 require_relative '../lib/twiglet/logger'
 
 LEVELS = [
@@ -42,11 +43,17 @@ describe Twiglet::Logger do
 
   describe 'JSON logging' do
     it 'should throw an error with an empty message' do
-      assert_raises RuntimeError do
+      assert_raises JSON::Schema::ValidationError, "The property '#/message' was not of a minimum string length of 1" do
         @logger.info({ message: '' })
       end
     end
 
+    it 'should throw an error if message is missing' do
+      assert_raises JSON::Schema::ValidationError, "The property '#/message' was not of a minimum string length of 1" do
+        @logger.info({ foo: 'bar' })
+      end
+    end
+
     it 'should log mandatory attributes' do
       @logger.error({ message: 'Out of pets exception' })
       actual_log = read_json(@buffer)
@@ -239,7 +246,7 @@ describe Twiglet::Logger do
 
   describe 'text logging' do
     it 'should throw an error with an empty message' do
-      assert_raises RuntimeError do
+      assert_raises JSON::Schema::ValidationError, "The property '#/message' was not of a minimum string length of 1" do
         @logger.info('')
       end
     end
@@ -352,6 +359,82 @@ describe Twiglet::Logger do
     end
   end
 
+  describe 'configuring error response' do
+    it 'blows up by default' do
+      assert_raises JSON::Schema::ValidationError,
+                    "The property '#/message' of type boolean did not match the following type: string" do
+        @logger.debug(message: true)
+      end
+    end
+
+    it 'silently swallows errors when configured to do so' do
+      mock = Minitest::Mock.new
+
+      @logger.configure_validation_error_response do |_e|
+        mock.notify_error("Logging schema validation error")
+      end
+
+      mock.expect(:notify_error, nil, ["Logging schema validation error"])
+      nonconformant_log = { message: true }
+      @logger.debug(nonconformant_log)
+    end
+  end
+
+  describe 'validation schema' do
+    before do
+      validation_schema = <<-JSON
+        {
+          "type": "object",
+          "required": ["pet"],
+          "properties": {
+            "pet": {
+              "type": "object",
+              "required": ["name", "best_boy_or_girl?"],
+              "properties": {
+                "name": {
+                  "type": "string",
+                  "minLength": 1
+                },
+                "best_boy_or_girl?": {
+                  "type": "boolean"
+                }
+              }
+            }
+          }
+        }
+      JSON
+
+      @logger = Twiglet::Logger.new(
+        'petshop',
+        now: @now,
+        output: @buffer,
+        validation_schema: validation_schema
+      )
+    end
+
+    it 'allows for the configuration of custom validation rules' do
+      @logger.debug(
+        {
+          pet: { name: 'Davis', best_boy_or_girl?: true, species: 'dog' }
+        }
+      )
+      log = read_json(@buffer)
+
+      assert_equal true, log[:pet][:best_boy_or_girl?]
+    end
+
+    it 'raises when custom validation rules are broken' do
+      nonconformant = {
+        pet: { name: 'Davis' }
+      }
+
+      assert_raises JSON::Schema::ValidationError,
+                    "The property '#/pet' did not contain a required property of 'best_boy_or_girl?'" do
+        @logger.debug(nonconformant)
+      end
+    end
+  end
+
   private
 
   def read_json(buffer)

data/test/message_test.rb

@@ -2,24 +2,6 @@ require 'minitest/autorun'
 require_relative '../lib/twiglet/message'
 
 describe Twiglet::Message do
-  it 'raises if message is empty' do
-    assert_raises RuntimeError do
-      Twiglet::Message.new('   ')
-    end
-  end
-
-  it 'raises if message is not provided' do
-    assert_raises RuntimeError do
-      Twiglet::Message.new(foo: 'bar')
-    end
-  end
-
-  it 'raises on unrecognized inputs' do
-    assert_raises RuntimeError do
-      Twiglet::Message.new(OpenStruct.new(message: 'hello'))
-    end
-  end
-
   it 'returns a message hash from a string' do
     assert_equal Twiglet::Message.new('hello, world'), { message: 'hello, world' }
   end

data/test/validator_test.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require 'minitest/autorun'
+require_relative '../lib/twiglet/validator'
+
+describe Twiglet::Validator do
+  let(:valid)
+
+  before do
+    schema = {
+      "type" => "object",
+      "required" => ["message"],
+      "properties" => {
+        "message" => {
+          "type" => "string"
+        }
+      }
+    }
+
+    @validator = Twiglet::Validator.new(schema.to_json)
+  end
+
+  it 'does not raise when validation passes' do
+    assert_equal(@validator.validate({ message: 'this is my message', foo: 'bar' }), true)
+  end
+
+  it 'raises when validation fails' do
+    assert_raises JSON::Schema::ValidationError do
+      @validator.validate({ message: true })
+    end
+  end
+
+  it 'is a no-op when validator is configured to swallow errors' do
+    @validator.custom_error_handler = ->(e) { puts e }
+
+    assert_nil(@validator.validate({ message: true }))
+  end
+end

data/twiglet.gemspec

@@ -21,4 +21,10 @@ Gem::Specification.new do |gem|
   gem.required_ruby_version = '>= 2.6'
 
   gem.license               = 'Copyright SimplyBusiness'
+
+  gem.add_runtime_dependency     'json-schema'
+  gem.add_development_dependency 'minitest'
+  gem.add_development_dependency 'rake'
+  gem.add_development_dependency 'simplecov', '0.17.1'
+  gem.add_development_dependency 'simplycop'
 end

metadata

@@ -1,15 +1,85 @@
 --- !ruby/object:Gem::Specification
 name: twiglet
 version: !ruby/object:Gem::Version
-  version: 3.0.8
+  version: 3.2.0
 platform: ruby
 authors:
 - Simply Business
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-11 00:00:00.000000000 Z
-dependencies: []
+date: 2021-02-16 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: json-schema
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+- !ruby/object:Gem::Dependency
+  name: simplycop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Like a log, only smaller.
 email:
 - tech@simplybusiness.co.uk
@@ -19,10 +89,10 @@ extra_rdoc_files: []
 files:
 - ".github/CODEOWNERS"
 - ".github/dependabot.yml"
+- ".github/workflows/dobby-actions.yml"
 - ".github/workflows/gem-publish.yml"
 - ".github/workflows/ruby.yml"
 - ".github/workflows/version-forget-me-not.yml"
-- ".github/workflows/version-update.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".ruby-version"
@@ -42,12 +112,15 @@ files:
 - lib/twiglet/formatter.rb
 - lib/twiglet/logger.rb
 - lib/twiglet/message.rb
+- lib/twiglet/validation_schema.json
+- lib/twiglet/validator.rb
 - lib/twiglet/version.rb
 - test/formatter_test.rb
 - test/hash_extensions_test.rb
 - test/logger_test.rb
 - test/message_test.rb
 - test/test_coverage.rb
+- test/validator_test.rb
 - twiglet.gemspec
 homepage: https://github.com/simplybusiness/twiglet-ruby
 licenses: