td-logger 0.3.3 → 0.3.4

data/ChangeLog

@@ -1,4 +1,9 @@
 
+== 2011-11-10 version 0.3.4
+
+* Raise error database name or table name is invalid
+
+
 == 2011-11-10 version 0.3.3
 
 * Validate database name and table name before queuing events

data/lib/td/logger/td_logger.rb

@@ -159,14 +159,14 @@ class TreasureDataLogger < Fluent::Logger::LoggerBase
     begin
       TreasureData::API.validate_database_name(db)
     rescue
-      @logger.error("TreasureDataLogger: Invalid database name #{db.inspect}: #{$!}")
-      return false
+      @logger.error "TreasureDataLogger: Invalid database name #{db.inspect}: #{$!}"
+      raise "Invalid database name #{db.inspect}: #{$!}"
     end
     begin
       TreasureData::API.validate_table_name(table)
-      @logger.error("TreasureDataLogger: Invalid table name #{table.inspect}: #{$!}")
     rescue
-      return false
+      @logger.error "TreasureDataLogger: Invalid table name #{table.inspect}: #{$!}"
+      raise "Invalid table name #{table.inspect}: #{$!}"
     end
 
     begin

data/lib/td/logger/version.rb

@@ -1,7 +1,7 @@
 module TreasureData
 module Logger
 
-VERSION = '0.3.3'
+VERSION = '0.3.4'
 
 end
 end

data/spec/td_logger_spec.rb

@@ -26,14 +26,22 @@ describe TreasureData::Logger::TreasureDataLogger do
   context 'validate' do
     it 'validate table name' do
       td = TreasureData::Logger::TreasureDataLogger.new('db1', :apikey=>'test')
-      td.post('invalid-name', {}).should == false
-      td.post('', {}).should == false
-      td.post('9', {}).should == false
+      proc {
+        td.post('invalid-name', {})
+      }.should raise_error(RuntimeError)
+      proc {
+        td.post('', {})
+      }.should raise_error(RuntimeError)
+      proc {
+        td.post('9', {})
+      }.should raise_error(RuntimeError)
     end
 
     it 'validate database name' do
       td = TreasureData::Logger::TreasureDataLogger.new('invalid-db-name', :apikey=>'test')
-      td.post('table', {}).should == false
+      proc {
+        td.post('table', {})
+      }.should raise_error(RuntimeError)
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: td-logger
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-11-10 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: msgpack
-  requirement: &70314747460680 !ruby/object:Gem::Requirement
+  requirement: &70130806269940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.4.4
   type: :runtime
   prerelease: false
-  version_requirements: *70314747460680
+  version_requirements: *70130806269940
 - !ruby/object:Gem::Dependency
   name: td-client
-  requirement: &70314747460200 !ruby/object:Gem::Requirement
+  requirement: &70130806269460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 0.8.0
   type: :runtime
   prerelease: false
-  version_requirements: *70314747460200
+  version_requirements: *70130806269460
 - !ruby/object:Gem::Dependency
   name: fluent-logger
-  requirement: &70314747459720 !ruby/object:Gem::Requirement
+  requirement: &70130806268980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,7 +43,7 @@ dependencies:
         version: 0.4.1
   type: :runtime
   prerelease: false
-  version_requirements: *70314747459720
+  version_requirements: *70130806268980
 description: 
 email: 
 executables: []