coaster 0.5.8 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5bddf807a6efc4c93fd95053bfc91f1d14a43f6c3c5eecd441d4f80b5a5e9a89
-  data.tar.gz: 4f57d7add97d24fabb8f150022603abae485441bff00c4eb3462032e5e84d3ee
+  metadata.gz: 7bb245ac61cf472e066e9f33934edfb00b715b07a2745ba30dcee503d973fbee
+  data.tar.gz: 133b5ae2c21130ff0b9ed27d03388b5f682b7ff95f31c2c455f2a36b62084cf4
 SHA512:
-  metadata.gz: 93ba4e67c7ca68f5ef3165a30a15f8158bcf57b4ee4f809c2aacfcdbab17bc9967ddcd2657a75808d43f92f4a611a7df184b02ed1c6fcb484f497394ef15ce3a
-  data.tar.gz: 813985347ab24389dca77903263031e08f4c349fc66b0d94bf07b683069c3390331fbf40103f185fb4a92ba78f44b1a6d46ef6f9e8903ebb42ec3022044ce140
+  metadata.gz: c26dfe88f571581a60fac9c5f7a4594ce6929e7dbee7aab4bd3b134e29e595ab8f3ee2dd912110625a810deef1bf5ef0c67e8935c4b9d9a5e1fd459a8a23b2ca
+  data.tar.gz: 983374e824e12b9da051ec0a246e65a6aaa0fe35c1bb75561aaff3398f4ffcd9f63009b6db97f23ff163ba073563400be07368103f9595b28049e489a7f39e3c

data/lib/coaster/core_ext/object_translation.rb

@@ -3,8 +3,9 @@ require 'i18n'
 class Object
   class << self
     def _translate(*args)
-      options = args.last.is_a?(Hash) ? args.pop.dup : {}
+      options = args.last.is_a?(Hash) ? args.pop : {}
       options.merge!(_translate_params)
+      options = options.to_hash.symbolize_keys!
 
       key = args.shift
       subkey = nil
@@ -33,7 +34,6 @@ class Object
       end
       options[:tkey] ||= key
       options.merge!(throw: true)
-      options.symbolize_keys!
       result = catch(:exception) do
         I18n.t(key, *args, options)
       end
@@ -58,6 +58,7 @@ class Object
           _translate(subkey, *args, options)
         end
       else
+        result = result.dup if result.frozen?
         result.instance_variable_set(:@translated, true)
         result.instance_variable_set(:@tkey, options[:tkey])
         result
@@ -75,7 +76,7 @@ class Object
   # Foo::Bar.new._translate(:force)    #=> ignore 'message' even if message exists
   #
   def _translate(*args)
-    options = args.last.is_a?(Hash) ? args.pop.dup : {}
+    options = (args.last.is_a?(Hash) ? args.pop : {}).with_indifferent_access
     key = args.shift || (respond_to?(:tkey) ? tkey : nil)
 
     if respond_to?(:description) && description.present? && description != 'false' && description != self.class.name

data/lib/coaster/core_ext/standard_error.rb

@@ -26,42 +26,43 @@ class StandardError
     @tags = {}
     @level = 'error'
     @attributes = HashWithIndifferentAccess.new
+    @attributes.merge!(cause.attributes || {}) if cause && cause.respond_to?(:attributes)
     @tkey = nil
 
     case message
-      when Exception
-        msg = message
-        set_backtrace(message.backtrace)
-      when StandardError
-        @fingerprint = message.fingerprint
-        @tags = message.tags
-        @level = message.level
-        @tkey = message.tkey
-        @attributes = message.attributes
-        msg = message
-        set_backtrace(message.backtrace)
-      when Hash then
-        hash = message.with_indifferent_access rescue message
-        msg = hash.delete(:m)
-        msg = hash.delete(:msg) || msg
-        msg = hash.delete(:message) || msg
-        @fingerprint = hash.delete(:fingerprint) || hash.delete(:fingerprints)
-        @tags = hash.delete(:tags) || hash.delete(:tag)
-        @level = hash.delete(:level) || hash.delete(:severity) || @level
-        @tkey = hash.delete(:tkey)
-        msg = cause.message if msg.nil? && cause
-        @attributes.merge!(hash)
-      when String then
-        msg = message
-      when FalseClass, NilClass then
-        msg = ''
-      else
-        msg = message
+    when Exception
+      msg = message
+      set_backtrace(message.backtrace)
+    when StandardError
+      @fingerprint = message.fingerprint
+      @tags = message.tags
+      @level = message.level
+      @tkey = message.tkey
+      @attributes = message.attributes
+      msg = message
+      set_backtrace(message.backtrace)
+    when Hash then
+      hash = message.with_indifferent_access rescue message
+      msg = hash.delete(:m)
+      msg = hash.delete(:msg) || msg
+      msg = hash.delete(:message) || msg
+      @fingerprint = hash.delete(:fingerprint) || hash.delete(:fingerprints)
+      @tags = hash.delete(:tags) || hash.delete(:tag)
+      @level = hash.delete(:level) || hash.delete(:severity) || @level
+      @tkey = hash.delete(:tkey)
+      msg = cause.message if msg.nil? && cause
+      @attributes.merge!(hash)
+    when String then
+      msg = message
+    when FalseClass, NilClass then
+      msg = ''
+    else
+      msg = message
     end
 
     @fingerprint = [] unless @fingerprint.is_a?(Array)
     @tags = {} unless @tags.is_a?(Hash)
-    msg ||= self.class.title
+    msg ||= self.class._translate
     super(msg)
   end
 

data/lib/coaster/version.rb

@@ -1,3 +1,3 @@
 module Coaster
-  VERSION = '0.5.8'
+  VERSION = '1.0.1'
 end

data/test/locales/en.yml

@@ -5,8 +5,12 @@ en:
       SampleObject:
         self: 'Coaster SampleObject Translated'
         title: 'Coaster SampleObject Title (%{tkey})'
+        interpolation: 'Coaster SampleObject interpolation test %{test_this}'
       TestStandardError:
+        ExampleError:
+          self: 'Test example error'
         SampleError:
+          self: 'Test sample error'
           test: 'Test this translation'
           title: 'Test this title'
   sample:

data/test/test_object_translation.rb

@@ -48,6 +48,10 @@ module Coaster
       assert_equal 'Coaster SampleObject Title (class.Coaster.Inherited.title)', Inherited._translate(:title)
     end
 
+    def test_interpolation
+      assert_equal 'Coaster SampleObject interpolation test this interpolated', SampleObject._translate('.interpolation', {test_this: 'this interpolated'}.with_indifferent_access)
+    end
+
     def teardown
       I18n.locale = nil
       I18n.default_locale = nil

data/test/test_standard_error.rb

@@ -39,20 +39,16 @@ module Coaster
         raise ExampleError, {wat: 'cha'}
       end
     rescue => e
-      assert_equal({
-        "wat"=>"cha",
-        "type"=>"Coaster::TestStandardError::ExampleError",
-        "status"=>20,
-        "http_status"=>500,
-        "message"=>"Test this title",
-        "cause"=>{
-          "frog"=>"rams",
-          "type"=>"Coaster::TestStandardError::SampleError",
-          "status"=>10,
-          "http_status"=>500,
-          "message"=>"Test this title"
-        }
-      }, e.to_hash)
+      assert_equal e.to_hash['wat'], 'cha'
+      assert_equal e.to_hash['type'], 'Coaster::TestStandardError::ExampleError'
+      assert_equal e.to_hash['status'], 20
+      assert_equal e.to_hash['http_status'], 500
+      assert_equal e.to_hash['message'], 'Test sample error'
+      assert_equal e.to_hash['cause']['frog'], 'rams'
+      assert_equal e.to_hash['cause']['type'], 'Coaster::TestStandardError::SampleError'
+      assert_equal e.to_hash['cause']['status'], 10
+      assert_equal e.to_hash['cause']['http_status'], 500
+      assert_equal e.to_hash['cause']['message'], 'Test sample error'
     end
 
     def test_cause_attributes
@@ -62,7 +58,9 @@ module Coaster
         raise ExampleError, {wat: 'cha'}
       end
     rescue => e
-      assert_equal({frog: 'rams', wat: 'cha'}.with_indifferent_access, e.attr)
+      assert_equal e.cause.attr['frog'], 'rams'
+      assert_equal e.attr['frog'], 'rams'
+      assert_equal e.attr['wat'], 'cha'
     end
 
     def test_to_detail
@@ -74,15 +72,15 @@ module Coaster
     rescue => e
       detail = <<-LOG
 [Coaster::TestStandardError::ExampleError] status:20
-	MESSAGE: Test this title
+	MESSAGE: Test sample error
 	@fingerprint: []
 	@tags: {}
 	@level: \"error\"
-	@attributes: {\"wat\"=>\"cha\"}
+	@attributes: {\"frog\"=>\"rams\", \"wat\"=>\"cha\"}
 	@tkey: nil
 	@raven: {}
 	CAUSE: [Coaster::TestStandardError::SampleError] status:10
-		MESSAGE: Test this title
+		MESSAGE: Test sample error
 		@fingerprint: []
 		@tags: {}
 		@level: \"error\"
@@ -108,7 +106,7 @@ LOG
     def test_title_missing
       raise UntitledError, 'untitled'
     rescue => e
-      assert_equal e.title, nil
+      assert_nil e.title
     end
 
     def root_cause_sample1
@@ -138,7 +136,8 @@ LOG
     def test_raven_notes
       raise SampleError, m: 'foofoo', something: 'other'
     rescue => e
-      assert_equal e.notes(the_other: 'something'), {extra: {something: 'other', the_other: 'something'}, fingerprint: [], tags: {}, level: 'error'}.with_indifferent_access
+      assert_equal e.notes(the_other: 'something')[:extra][:something], 'other'
+      assert_equal e.notes(the_other: 'something')[:extra][:the_other], 'something'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coaster
 version: !ruby/object:Gem::Version
-  version: 0.5.8
+  version: 1.0.1
 platform: ruby
 authors:
 - buzz jung
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-17 00:00:00.000000000 Z
+date: 2020-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement