[Groonga-commit] droonga/fluent-plugin-droonga at 4e91be1 [master] Raise error for missing column correctly

アーカイブの一覧に戻る

YUKI Hiroshi null+****@clear*****
Thu Dec 26 17:41:29 JST 2013


YUKI Hiroshi	2013-12-26 17:41:29 +0900 (Thu, 26 Dec 2013)

  New Revision: 4e91be1e0ec4da7176a8529733070e8b09d104cb
  https://github.com/droonga/fluent-plugin-droonga/commit/4e91be1e0ec4da7176a8529733070e8b09d104cb

  Message:
    Raise error for missing column correctly

  Modified files:
    lib/droonga/plugin/handler/add.rb
    test/command/suite/add/unknown-column.expected

  Modified: lib/droonga/plugin/handler/add.rb (+1 -1)
===================================================================
--- lib/droonga/plugin/handler/add.rb    2013-12-26 17:37:42 +0900 (704eba9)
+++ lib/droonga/plugin/handler/add.rb    2013-12-26 17:41:29 +0900 (b0eaa16)
@@ -98,7 +98,7 @@ module Droonga
           raise InvalidValue.new(column, value, request)
         rescue Groonga::NoSuchColumn => error
           record.delete if record.added?
-          raise UnknownColumn.new(column, table, request)
+          raise UnknownColumn.new(column, request["table"], request)
         end
       end
     end

  Modified: test/command/suite/add/unknown-column.expected (+1 -1)
===================================================================
--- test/command/suite/add/unknown-column.expected    2013-12-26 17:37:42 +0900 (81c5a38)
+++ test/command/suite/add/unknown-column.expected    2013-12-26 17:41:29 +0900 (a3579c3)
@@ -7,7 +7,7 @@
     "type": "add.result",
     "body": {
       "name": "UnknownColumn",
-      "message": "The column \"unknown\" does not exist in the table \"User\"",
+      "message": "The column \"unknown\" does not exist in the table \"User\".",
       "detail": {
         "table": "User",
         "key": "key",
-------------- next part --------------
HTML����������������������������...
ダウンロード 



More information about the Groonga-commit mailing list
アーカイブの一覧に戻る