coaster 1.2.2 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b275d9ddd61f4631acadb1fee4ebfd3fb1d0bb0856621b732b431721940b460
-  data.tar.gz: f10dfe4a4583ee7ca7594ee514f384d428b3795ff421b420a86b70a89319389e
+  metadata.gz: aeeb91379c4231e8622891bfbfee5c93c202e96706bfa7d569b4bcea3e19d696
+  data.tar.gz: 124f01130e4fadf67e9e28c134e409986f8d1186e324f1b93e35fdf040a09652
 SHA512:
-  metadata.gz: 57169b9c338d9499983fbe438d5739d2b82539652f448ed3e703e9c94570fb6f719f16aa628ea7b8928f1db475ed1e3d490acad9b8246ba43792a563cc9fa3a9
-  data.tar.gz: 7dcb832e9d9e855973a6743ec05c12a25fcd78d58806d5b7b2e1c8f85faf6896f397ac82ed008a600dc8d273597593c3ff651cc7f13c0b3637a0c43a07c52056
+  metadata.gz: fe9c0490709aa11a3c96816294702dbebd982b7304af0633f319612035f4be627f030a928749506d454fd2806c485b6e440c37c0895dae8ec2633a780a66f4dd
+  data.tar.gz: 5a7de756347f74c4ece3dad9b36a987d44715b29cca55ded06d2a1bdfb0b38f4e16c654a4d9c2db3284c4cf76ab629afcbfa3bc33f373eeb21fc52db8008d6e0

data/lib/coaster/core_ext/standard_error.rb

@@ -62,19 +62,18 @@ class StandardError
       @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 = ''
+      msg = nil
     else
       msg = message
     end
 
     @fingerprint = [] unless @fingerprint.is_a?(Array)
     @tags = {} unless @tags.is_a?(Hash)
-    msg ||= self.class._translate
+    msg = cause.message if msg.blank? && cause
     super(msg)
   end
 
@@ -143,6 +142,15 @@ 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}\n#{to_origin_s}"
+  end
+
   def to_hash
     hash = attributes.merge(
       type: self.class.name, status: status,

data/lib/coaster/core_ext/standard_error/raven.rb

@@ -66,17 +66,4 @@ class StandardError
     capture(options)
     just_logging(options)
   end
-
-  # https://github.com/getsentry/sentry-ruby/blob/fbbc7a51ed10684d0e8b7bd9d2a1b65a7351c9ef/lib/raven/event.rb#L162
-  # sentry use `to_s` as a message
-  alias to_origin_s to_s
-  def message
-    to_origin_s
-  end
-
-  def to_s
-    str = super # https://ruby-doc.org/core-2.5.1/Exception.html#method-i-to_s
-    str = "#{user_message} / #{str}" if user_message.present?
-    str
-  end
 end

data/lib/coaster/version.rb

@@ -1,3 +1,3 @@
 module Coaster
-  VERSION = '1.2.2'
+  VERSION = '1.3.0'
 end

data/test/test_standard_error.rb

@@ -21,74 +21,90 @@ module Coaster
     end
 
     def test_standard_messages
-      e = StandardError.new('asdf')
-      assert_equal 'asdf', e.message
+      e = StandardError.new('developer message')
+      assert_equal "standard error translation\ndeveloper message", e.to_s
+      assert_equal "standard error translation\ndeveloper message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'standard error translation', e._translate
       assert_equal 'standard error translation', e.user_message
-      assert_equal 'standard error translation / asdf', e.to_s
       assert_equal 'standard error title', e.title
-      e = StandardError.new(m: 'foo', desc: 'bar')
-      assert_equal 'foo', e.message
-      assert_equal 'bar', e.description
-      assert_equal 'bar', e.desc
-      assert_equal 'bar', e._translate
-      assert_equal 'bar', e.user_message
+      e = StandardError.new(m: 'developer message', desc: 'user message')
+      assert_equal "user message\ndeveloper message", e.to_s
+      assert_equal "user message\ndeveloper message", 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 'standard error title', e.title
-      assert_equal 'bar / foo', e.to_s
     end
 
     def test_no_translation_class
-      e = UntitledError.new('asdf')
-      assert_equal 'asdf', e.message
+      e = UntitledError.new('developer message')
+      assert_equal "standard error translation\ndeveloper message", e.to_s
+      assert_equal "standard error translation\ndeveloper message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'standard error translation', e._translate
       assert_equal 'standard error translation', e.user_message
-      assert_equal 'standard error translation / asdf', e.to_s
       assert_equal 'standard error title', e.title
-      e = UntitledError.new(m: 'foo', desc: 'bar')
-      assert_equal 'foo', e.message
-      assert_equal 'bar', e.description
-      assert_equal 'bar', e.desc
-      assert_equal 'bar', e._translate
-      assert_equal 'bar', e.user_message
-      assert_equal 'bar / foo', e.to_s
+      e = UntitledError.new(m: 'developer message', desc: 'user message')
+      assert_equal "user message\ndeveloper message", e.to_s
+      assert_equal "user message\ndeveloper message", 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 'standard error title', e.title
     end
 
     def test_with_translation_class
-      e = SampleError.new('asdf')
-      assert_equal 'asdf', e.message
+      e = SampleError.new
+      assert_equal "Test sample error\nCoaster::TestStandardError::SampleError", e.to_s
+      assert_equal "Test sample error\nCoaster::TestStandardError::SampleError", e.message
+      assert_nil e.description
+      assert_nil e.desc
+      assert_equal 'Test sample error', e._translate
+      assert_equal 'Test sample error', e.user_message
+      assert_equal 'Test this title',  e.title
+      e = SampleError.new(beet: 'apple')
+      assert_equal "Test sample error\nCoaster::TestStandardError::SampleError", e.to_s
+      assert_equal "Test sample error\nCoaster::TestStandardError::SampleError", e.message
+      assert_nil e.description
+      assert_nil e.desc
+      assert_equal 'Test sample error', e._translate
+      assert_equal 'Test sample error', e.user_message
+      assert_equal 'Test this title',  e.title
+      e = SampleError.new('developer message')
+      assert_equal "Test sample error\ndeveloper message", e.to_s
+      assert_equal "Test sample error\ndeveloper message", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'Test sample error', e._translate
       assert_equal 'Test sample error', e.user_message
-      assert_equal 'Test sample error / asdf', e.to_s
       assert_equal 'Test this title', e.title
-      e = SampleError.new(m: 'foo', desc: 'bar')
-      assert_equal 'foo', e.message
-      assert_equal 'bar', e.description
-      assert_equal 'bar', e.desc
-      assert_equal 'bar', e._translate
-      assert_equal 'bar', e.user_message
-      assert_equal 'bar / foo', e.to_s
+      e = SampleError.new(m: 'developer message', desc: 'user message')
+      assert_equal "user message\ndeveloper message", e.to_s
+      assert_equal "user message\ndeveloper message", 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(tkey: 'sample.interpolation', value: 'blah')
-      assert_equal 'Test sample error', e.message
+      assert_equal "Sample Interpolation blah\nCoaster::TestStandardError::SampleError", e.to_s
+      assert_equal "Sample Interpolation blah\nCoaster::TestStandardError::SampleError", e.message
       assert_nil e.description
       assert_nil e.desc
       assert_equal 'Sample Interpolation blah', e._translate
       assert_equal 'Sample Interpolation blah', e.user_message
-      assert_equal 'Sample Interpolation blah / Test sample error', e.to_s
       assert_equal 'Test this title', e.title
     end
 
     def test_message
       raise SampleError, {m: 'beer is proof'}
     rescue => e
-      assert_equal 'beer is proof', e.message
+      assert_equal "Test sample error\nbeer is proof", e.message
     end
 
     def test_attributes
@@ -108,12 +124,12 @@ 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 sample error', e.to_hash['message']
+      assert_equal "Test example error\nTest sample error\n#{SampleError.name}", 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']
       assert_equal 500, e.to_hash['cause']['http_status']
-      assert_equal 'Test sample error', e.to_hash['cause']['message']
+      assert_equal "Test sample error\nCoaster::TestStandardError::SampleError", e.to_hash['cause']['message']
     end
 
     def test_cause_attributes
@@ -137,7 +153,9 @@ module Coaster
     rescue => e
       detail = <<-LOG
 [Coaster::TestStandardError::ExampleError] status:20
-	MESSAGE: Test sample error
+	MESSAGE: Test example error
+		Test sample error
+		Coaster::TestStandardError::SampleError
 	@fingerprint: []
 	@tags: {}
 	@level: \"error\"
@@ -146,6 +164,7 @@ module Coaster
 	@raven: {}
 	CAUSE: [Coaster::TestStandardError::SampleError] status:10
 		MESSAGE: Test sample error
+			Coaster::TestStandardError::SampleError
 		@fingerprint: []
 		@tags: {}
 		@level: \"error\"
@@ -194,7 +213,7 @@ LOG
       begin
         root_cause_sample3
       rescue => e
-        assert_equal 'a', e.root_cause.message
+        assert_equal "standard error translation\na", e.root_cause.message
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coaster
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.3.0
 platform: ruby
 authors:
 - buzz jung
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-13 00:00:00.000000000 Z
+date: 2020-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n
@@ -80,62 +80,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: pry
-  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: pry-stack_explorer
-  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: pry-byebug
-  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: pry-doc
-  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: minitest
   requirement: !ruby/object:Gem::Requirement