coaster 1.3.4 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '095940670cdfde3eac7e9c3ab8818386b9070cf4d19ac405e144296e2bdb2c25'
-  data.tar.gz: 468278966fcdad4349921586d9520a2dd0b84188f52f5f089678f09b88ea7ef8
+  metadata.gz: ca831c98247a51e387740d4a2b15d9d7c31a37a4a119e69bf9081cc6ef08a438
+  data.tar.gz: dedfc0e5be84563640b98ac5808dda34c735ac2ca40edac5c76d906d69c9b63d
 SHA512:
-  metadata.gz: 9a332cd5b18dd1963a010293352c80939d99c968f4c83d52f02aebf0d50d444eb2ed82ed0590afded4eed2712b58eb550b7ca1f1f90389d42011b920435a7929
-  data.tar.gz: 4fe3cddcca27bc17f0c189e9eb829b68f52f07b9a4b4c6c2a9dcbbaca883c2d2cb4e89f134c159fc0a94888049ac990dbb6c26d7811681f32d995a4e8dcbc22d
+  metadata.gz: 4d8591789c4e587d37b88033881c6031d7ce231915058b2eac1b041c5e2d8933a03808a157a6f8cf17b3b5541c0ee25b1fd3b24992956467bfda165369c22968
+  data.tar.gz: f778ee607e1f0a249518df4292427e4fd812ebbca8868b40908e593dce6670d4a818b166a58aa24cb5f195f9159119983684ff1fee0cf6c2273bf02d54a949a6

data/lib/coaster/core_ext/standard_error.rb

@@ -53,17 +53,23 @@ class StandardError
       msg = message
       set_backtrace(message.backtrace)
     when Hash then
+      @coaster = true # coaster 확장을 사용한 에러임을 확인할 수 있음.
       hash = message.with_indifferent_access rescue message
       msg = hash.delete(:m)
       msg = hash.delete(:msg) || msg
       msg = hash.delete(:message) || msg
       hash[:description] ||= hash.delete(:desc) if hash[:desc].present?
-      hash[:dev_message] ||= hash.delete(:dm) if hash[:dm].present?
       @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)
       @attributes.merge!(hash)
+      if @attributes[:description] == :translate
+        @attributes.delete(:description)
+        @attributes[:description] = _translate
+      end
+      msg = "#{_translate} (#{msg || self.class.name})"
+      msg = "#{msg} {#{cause.message}}" if cause
     when String then
       msg = message
     when FalseClass, NilClass then
@@ -74,7 +80,7 @@ class StandardError
 
     @fingerprint = [] unless @fingerprint.is_a?(Array)
     @tags = {} unless @tags.is_a?(Hash)
-    msg = cause.message if msg.blank? && cause
+    msg = "{#{cause.message}}" if msg.blank? && cause
     super(msg)
   end
 
@@ -118,8 +124,6 @@ class StandardError
   # error message is not user friendly in many cases.
   def description; attributes[:description] || attributes[:desc] end
   alias_method :desc, :description
-  def dev_message; attributes[:dev_message] end
-  alias_method :dm, :dev_message
 
   def _translate(*args)
     return description if description.present?
@@ -132,9 +136,9 @@ class StandardError
 
   # user friendly message, for overid
   def user_message
-    return description if description.present?
-    return _translate if tkey.present? || dev_message.present?
-    "#{_translate} (#{to_origin_s})"
+    return _translate if description.present? || tkey.present?
+    return "#{_translate} (#{message})" unless defined?(@coaster)
+    message
   end
 
   # another user friendly messages
@@ -144,15 +148,6 @@ class StandardError
     attributes[:descriptions]
   end
 
-  # https://github.com/getsentry/sentry-ruby/blob/fbbc7a51ed10684d0e8b7bd9d2a1b65a7351c9ef/lib/raven/event.rb#L162
-  # sentry message use `to_s` method
-  # https://ruby-doc.org/core-2.5.1/Exception.html#method-i-to_s
-  alias to_origin_s to_s
-
-  def to_s
-    "#{_translate} (#{dev_message || to_origin_s})"
-  end
-
   def to_hash
     hash = attributes.merge(
       type: self.class.name, status: status,
@@ -176,7 +171,7 @@ class StandardError
     lg = "[#{self.class.name}] status:#{status}"
     lg += "\n\tMESSAGE: #{safe_message.gsub(/\n/, "\n\t\t")}"
     instance_variables.each do |var|
-      if var.to_s.start_with?('@_')
+      if var.to_s.start_with?('@_') || var.to_s == '@coaster'
         next
       elsif var.to_s == '@spell_checker'
         next

data/lib/coaster/version.rb

@@ -1,3 +1,3 @@
 module Coaster
-  VERSION = '1.3.4'
+  VERSION = '1.3.5'
 end

data/test/test_standard_error.rb

@@ -22,8 +22,8 @@ module Coaster
 
     def test_standard_messages
       e = StandardError.new('developer message')
-      assert_equal "standard error translation (developer message)", e.to_s
-      assert_equal "standard error translation (developer message)", e.message
+      assert_equal "developer message", e.to_s
+      assert_equal "developer message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'standard error translation', e._translate
@@ -41,8 +41,8 @@ module Coaster
 
     def test_no_translation_class
       e = UntitledError.new('developer message')
-      assert_equal "standard error translation (developer message)", e.to_s
-      assert_equal "standard error translation (developer message)", e.message
+      assert_equal "developer message", e.to_s
+      assert_equal "developer message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'standard error translation', e._translate
@@ -68,8 +68,8 @@ module Coaster
 
     def test_with_translation_class
       e = SampleError.new
-      assert_equal "Test sample error (Coaster::TestStandardError::SampleError)", e.to_s
-      assert_equal "Test sample error (Coaster::TestStandardError::SampleError)", e.message
+      assert_equal "Coaster::TestStandardError::SampleError", e.to_s
+      assert_equal "Coaster::TestStandardError::SampleError", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'Test sample error', e._translate
@@ -84,18 +84,26 @@ module Coaster
       assert_equal 'Test sample error (Coaster::TestStandardError::SampleError)', e.user_message
       assert_equal 'Test this title',  e.title
       e = SampleError.new('developer message')
-      assert_equal "Test sample error (developer message)", e.to_s
-      assert_equal "Test sample error (developer message)", e.message
+      assert_equal "developer message", e.to_s
+      assert_equal "developer message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'Test sample error', e._translate
       assert_equal 'Test sample error (developer message)', e.user_message
       assert_equal 'Test this title', e.title
-      e = SampleError.new(dm: 'developer message')
+      e = SampleError.new(desc: 'user message')
+      assert_equal "user message (Coaster::TestStandardError::SampleError)", e.to_s
+      assert_equal "user message (Coaster::TestStandardError::SampleError)", e.message
+      assert_equal 'user message', e.description
+      assert_equal 'user message', e.desc
+      assert_equal 'user message', e._translate
+      assert_equal 'user message', e.user_message
+      assert_equal 'Test this title', e.title
+      e = SampleError.new(m: 'developer message', desc: :translate)
       assert_equal "Test sample error (developer message)", e.to_s
       assert_equal "Test sample error (developer message)", e.message
-      assert_nil e.description
-      assert_nil e.desc
+      assert_equal "Test sample error", e.description
+      assert_equal "Test sample error", e.desc
       assert_equal 'Test sample error', e._translate
       assert_equal 'Test sample error', e.user_message
       assert_equal 'Test this title', e.title
@@ -140,7 +148,7 @@ module Coaster
       assert_equal 'Coaster::TestStandardError::ExampleError', e.to_hash['type']
       assert_equal 20, e.to_hash['status']
       assert_equal 500, e.to_hash['http_status']
-      assert_equal "Test example error (Test sample error (#{SampleError.name}))", e.to_hash['message']
+      assert_equal "Test example error (Coaster::TestStandardError::ExampleError) {Test sample error (Coaster::TestStandardError::SampleError)}", e.to_hash['message']
       assert_equal 'rams', e.to_hash['cause']['frog']
       assert_equal 'Coaster::TestStandardError::SampleError', e.to_hash['cause']['type']
       assert_equal 10, e.to_hash['cause']['status']
@@ -152,9 +160,10 @@ module Coaster
       begin
         raise SampleError, {frog: 'rams'}
       rescue => e
-        raise ExampleError, {wat: 'cha'}
+        raise ExampleError, {m: 'abc', wat: 'cha'}
       end
     rescue => e
+      assert_equal 'Test example error (abc) {Test sample error (Coaster::TestStandardError::SampleError)}', e.message
       assert_equal 'rams', e.cause.attr['frog']
       assert_equal 'rams', e.attr['frog']
       assert_equal 'cha', e.attr['wat']
@@ -169,7 +178,7 @@ module Coaster
     rescue => e
       detail = <<-LOG
 [Coaster::TestStandardError::ExampleError] status:20
-	MESSAGE: Test example error (Test sample error (Coaster::TestStandardError::SampleError))
+	MESSAGE: Test example error (Coaster::TestStandardError::ExampleError) {Test sample error (Coaster::TestStandardError::SampleError)}
 	@fingerprint: []
 	@tags: {}
 	@level: \"error\"
@@ -226,7 +235,7 @@ LOG
       begin
         root_cause_sample3
       rescue => e
-        assert_equal "standard error translation (a)", e.root_cause.message
+        assert_equal "a", e.root_cause.message
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coaster
 version: !ruby/object:Gem::Version
-  version: 1.3.4
+  version: 1.3.5
 platform: ruby
 authors:
 - buzz jung
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-14 00:00:00.000000000 Z
+date: 2020-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n