twiglet 3.0.8 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aadbe159aee695cfd454e0f006ab5eab2b3c06d01ecaa80d3f6b1b69c0d21d5e
-  data.tar.gz: 53907b03fb29af1fe41d8262619a6d5c84dab554b4e309d7336d969c11600a44
+  metadata.gz: 120d52bf23086d2db6f65f068aa087835914bcfe52c3ef1f8b6b886f946e9dcd
+  data.tar.gz: 8355c113e09882c4b1346111bba954b82f81d69c0290d1055dca4bb61174fc09
 SHA512:
-  metadata.gz: d5e68a4b343c861c7236e4a301365d738221bbe5eb0b23426e3cba84fad2358046d1b868bef5248c1c3398f4c2cfd132445259b7db976937024f73663699c2d4
-  data.tar.gz: 28b619fd981103d5baea53d07e032e52e21cf88e84b5229d1ca1ff61cedc284d6825ebf26fb977f060d3af4c34f276b8a69c733e66321952a726d452dc82164b
+  metadata.gz: 65ca30c0dcf26b8190b7840b70bd5a183b481333bc98af0abfed2835a5a25ec664e2b3c24f10d90f40a6a1d5a18fb90d9f97c2c2aae0ab49b123558049cc11da
+  data.tar.gz: 5936e07ac730cd18b39b5777a3fd1a7b8d805852726ad2732d46d6acdb1c7ef820f7dacf87ba68a85352dd3653c0e49b132a90ad71a40dd7ebdc3744ff9e68b8

data/Gemfile

@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
 source 'https://rubygems.org'
+
+gem 'json-schema'
+
 group :development, :test do
   gem 'minitest'
   gem 'rake'

data/lib/twiglet/formatter.rb

@@ -1,24 +1,28 @@
 require 'logger'
 require_relative '../hash_extensions'
 require_relative 'message'
+require_relative 'validator'
 
 module Twiglet
   class Formatter < ::Logger::Formatter
     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

@@ -26,8 +26,19 @@ module Twiglet
       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)
-      super(output, formatter: formatter, level: level)
+      @validator = Validator.from_file('lib/twiglet/validation_schema.json')
+
+      @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)

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,25 @@
+# frozen_string_literal: true
+
+require 'json-schema'
+require 'json'
+
+module Twiglet
+  class Validator
+    attr_accessor :custom_error_handler
+
+    def initialize(schema)
+      @schema = schema
+      @custom_error_handler = ->(e) { raise e }
+    end
+
+    def self.from_file(file_path)
+      new(JSON.parse(File.read(file_path)))
+    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.1.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({}))
   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,27 @@ 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
+
   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)
+  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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: twiglet
 version: !ruby/object:Gem::Version
-  version: 3.0.8
+  version: 3.1.0
 platform: ruby
 authors:
 - Simply Business
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-11 00:00:00.000000000 Z
+date: 2021-02-15 00:00:00.000000000 Z
 dependencies: []
 description: Like a log, only smaller.
 email:
@@ -42,12 +42,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: