fluent-plugin-condition-checker 0.1.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59770b074ad4850dbf87fae2f59788099ef308948a6d398c9857c56e0c5da8cc
-  data.tar.gz: 8d44f4e00886b2e41ac95d490f6c67f92fc7fb42b3f818f9cc5437851706de4c
+  metadata.gz: 14c83969fa6928655e996db271f6dab4483b32e7eb2c8282455a4664c96ab904
+  data.tar.gz: e437510a11585f4d31f69adca9bd9a714cf4d084eec45e89081565a44c9bfb47
 SHA512:
-  metadata.gz: dfa60bc60e4e048ddee2df596966ded0da52be0b0a58e5b30b908b1857e55ec5a63fdac0898c14fcf831cd71f358e01ca10df61ea273fa2cb151b57e153cce88
-  data.tar.gz: c7882cf182ad7c33c8aa2200a5ba489c2eb316949a31071800d9cf87b75a826b07874c64db01e27cfc944092e0607b5cf8ac623b9eb79467086d47eca4961b54
+  metadata.gz: eed247f918a990606d53ab539cc11d0c266bc263e3385a34b3db33caa13b60ef6d1f94958ace69debad63ac6355fa1785cf0515692a7029e4d5afb7d518a530a
+  data.tar.gz: 17d4ffaa4def66e7baf802da002ef14f70c4e1007d30aab3451b8e949814cdcdbbb5f365ed2c54cef3748ccda3998663fdd89e1af686cc3bc7d372da424c1449

data/.gitignore

@@ -0,0 +1 @@
+pkg/

data/.rubocop.yml

@@ -0,0 +1,18 @@
+Style/AsciiComments:
+  Enabled: false
+Metrics/LineLength:
+  Max: 140
+Naming/UncommunicativeMethodParamName:
+  Enabled: false
+Metrics/AbcSize:
+  Max: 30
+Metrics/MethodLength:
+  Max: 30
+Metrics/PerceivedComplexity:
+  Max: 10
+Metrics/CyclomaticComplexity:
+  Max: 10
+Metrics/BlockLength:
+  Max: 45
+Metrics/ClassLength:
+  Max: 120

data/Gemfile.lock

@@ -0,0 +1,48 @@
+PATH
+  remote: .
+  specs:
+    fluent-plugin-condition-checker (1.0.0)
+      fluentd (>= 0.14.10, < 2)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    concurrent-ruby (1.1.7)
+    cool.io (1.7.0)
+    fluentd (1.11.4)
+      cool.io (>= 1.4.5, < 2.0.0)
+      http_parser.rb (>= 0.5.1, < 0.7.0)
+      msgpack (>= 1.3.1, < 2.0.0)
+      serverengine (>= 2.0.4, < 3.0.0)
+      sigdump (~> 0.2.2)
+      strptime (>= 0.2.2, < 1.0.0)
+      tzinfo (>= 1.0, < 3.0)
+      tzinfo-data (~> 1.0)
+      yajl-ruby (~> 1.0)
+    http_parser.rb (0.6.0)
+    msgpack (1.3.3)
+    power_assert (1.2.0)
+    rake (12.3.3)
+    serverengine (2.2.1)
+      sigdump (~> 0.2.2)
+    sigdump (0.2.4)
+    strptime (0.2.5)
+    test-unit (3.3.6)
+      power_assert
+    tzinfo (2.0.2)
+      concurrent-ruby (~> 1.0)
+    tzinfo-data (1.2020.2)
+      tzinfo (>= 1.0.0)
+    yajl-ruby (1.4.1)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 2.0)
+  fluent-plugin-condition-checker!
+  rake (~> 12.0)
+  test-unit (~> 3.0)
+
+BUNDLED WITH
+   2.1.4

data/README.md

@@ -28,6 +28,26 @@ $ bundle
 
 ## Configuration
 
+```
+<match **>
+  @type condition-checker
+  <condition>
+    condition record["faceId"] != nil
+    tag exists_face
+    <data>
+      concept exists_face
+    </data>
+  </condition>
+  <condition>
+    condition record["faceId"] -= nil
+    tag no_face
+    <data>
+      concept no_face
+    </data>
+  </condition>
+</match>
+```
+
 You can generate configuration template:
 
 ```
@@ -36,8 +56,16 @@ $ fluent-plugin-config-format output condition-checker
 
 You can copy and paste generated documents here.
 
+## Development
+
+1. test!
+
+```
+bundle exec rake test
+```
+
 ## Copyright
 
-* Copyright(c) 2020- wally
-* License
-  * Apache License, Version 2.0
+- Copyright(c) 2020- wally
+- License
+  - Apache License, Version 2.0

data/example/fluent.conf

@@ -0,0 +1,31 @@
+<source>
+  @type forward
+  port 24224
+</source>
+
+<match **>
+  @type copy
+  <store>
+    @type stdout
+  </store>
+  <store>
+    @type condition-checker
+     tag "speak_condition"
+    <condition>
+      rule record["action"] == "speak"
+      condition "speak_action"
+      <data>
+        word record["word"] + "こんにちは"
+        aditiona_data 1
+      </data>
+    </condition>
+    <condition>
+      rule record["action"]  == "alert"
+      condition "alert_action"
+      <data>
+        word record["word"] + "はじめまして"
+      </data>
+    </condition>
+  </store>
+</match>
+

data/example/scenario.json

@@ -0,0 +1 @@
+{"label":"greeting","priority":2,"limit":30,"action":"greet"}

data/fluent-plugin-condition-checker.gemspec

@@ -3,7 +3,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name    = "fluent-plugin-condition-checker"
-  spec.version = "0.1.0"
+  spec.version = "1.0.0"
   spec.authors = ["wally"]
   spec.email   = ["u.str.gm@gmail.com"]
 
@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = test_files
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.14"
+  spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "rake", "~> 12.0"
   spec.add_development_dependency "test-unit", "~> 3.0"
   spec.add_runtime_dependency "fluentd", [">= 0.14.10", "< 2"]

data/lib/fluent/plugin/out_condition_checker.rb

@@ -23,362 +23,336 @@ module Fluent
       Fluent::Plugin.register_output("condition_checker", self)
 
       helpers :event_emitter
-  PATTERN_MAX_NUM = 20
-
-  config_param :remove_keys, :string, :default => nil,
-                     :desc => 'Specify record keys to be removed by a string separated by , (comma).'
-  config_param :keep_keys, :string, :default => nil,
-                     :desc => 'Specify record keys to be kept by a string separated by , (comma).'
-  config_param :renew_record, :bool, :default => false,
-                     :desc => 'Creates an output record newly without extending (merging) the input record fields.'
-  config_param :renew_time_key, :string, :default => nil,
-                     :desc => 'Overwrites the time of events with a value of the record field.'
-  # it should always be true
-  config_param :enable_ruby, :bool, :default => true, # true for lower version compatibility
-                     :desc => 'Enable to use ruby codes in placeholders.'
-  # it should always be true
-  config_param :auto_typecast, :bool, :default => true, # false for lower version compatibility
-                     :desc => 'Automatically cast the field types.'
-
-  # 調整中のconfig_param
-  # これで、何個もconditionを設定して、そして処理を決めることができる。
-  config_param :tag1, :string,
-               :desc => 'Specify the output tag name when the condition1 is true.'
-  config_param :condition1, :string,
-                     :desc => 'Specify the condition1 to evaluate'
-  (2..PATTERN_MAX_NUM).each do |i|
-    config_param ('tag' + i.to_s).to_sym, :string, default: nil,
-                 desc: 'Specify tag'+i.to_s+' (not necessary)'
-    config_param ('condition' + i.to_s).to_sym, :string, default: nil, # NAME REGEXP
-                 desc: 'Specify the condition'+i.to_s+' to evaluate (not necessary)'
-  end
+      PATTERN_MAX_NUM = 20
+
+      config_param :remove_keys, :string, :default => nil,
+                        :desc => 'Specify record keys to be removed by a string separated by , (comma).'
+      config_param :keep_keys, :string, :default => nil,
+                        :desc => 'Specify record keys to be kept by a string separated by , (comma).'
+      config_param :renew_record, :bool, :default => false,
+                        :desc => 'Creates an output record newly without extending (merging) the input record fields.'
+      config_param :renew_time_key, :string, :default => nil,
+                        :desc => 'Overwrites the time of events with a value of the record field.'
+      # it should always be true
+      config_param :enable_ruby, :bool, :default => true, # true for lower version compatibility
+                        :desc => 'Enable to use ruby codes in placeholders.'
+      # it should always be true
+      config_param :auto_typecast, :bool, :default => true, # false for lower version compatibility
+                        :desc => 'Automatically cast the field types.'
+      config_param :tag, :string, :default => 'checked_tag', :desc => "new record tag."
+      
+      config_param(
+        "condition".to_sym,
+        :hash,
+        default: nil,
+        desc: 'It is Conditions.'
+      )
 
-  config_param :tag_else, :string,
-               :desc => 'Specify the output tag name when the no conditions are true.'
+      BUILTIN_CONFIGURATIONS = %W(@id @type @label type output_tag remove_keys renew_record keep_keys enable_ruby renew_time_key auto_typecast tag_else record_else tag)
 
+      def configure(conf)
+        super
+        # ここで、BUILTIN_CONFIGURATIONS に入っていないものがあった場合はerrorをraise
+        conf.each_pair { |k, v|
+          next if BUILTIN_CONFIGURATIONS.include?(k) || k.match(/^condition\d\d?$/) || k.match(/^tag\d\d?$/)
 
-  BUILTIN_CONFIGURATIONS = %W(@id @type @label type output_tag remove_keys renew_record keep_keys enable_ruby renew_time_key auto_typecast tag_else record_else)
+          raise Fluent::ConfigError, 'out_condition_checker: some weird config is set {'+k.to_s+':'+v.to_s+'}'
+        }
 
-  def configure(conf)
-    super
-    # ここで、BUILTIN_CONFIGURATIONS に入っていないものがあった場合はerrorをraise
-    conf.each_pair { |k, v|
-      # print k.match('^/tag/\d\d?$')
-      next if BUILTIN_CONFIGURATIONS.include?(k) || k.match(/^condition\d\d?$/) || k.match(/^tag\d\d?$/)
+        # conditionsを読み込む
+        @conditions = []
+        conf.elements.select { |element| element.name.match(/^condition?$/) }
+            .each do |param|
+          condition = {}
+          data_record = {}
+
+          param.elements.select { |element| element.name.match(/^data?$/) }.each do |data|
+            data.each_pair do |key, value|
+              data_record.merge!(key => convert_value(value))
+            end
+          end
 
-      raise Fluent::ConfigError, 'out_condition_checker: some weird config is set {'+k.to_s+':'+v.to_s+'}'
-    }
+          param.each_pair do |key, value|
+            condition.merge!(key => convert_value(value))
+          end
 
-    #tagを読み込み
-    tags = []
-    (1..PATTERN_MAX_NUM).each do |i|
-      next unless conf["tag#{i}"]
-      tags.push(conf["tag#{i}"]) # tags[i+1] で、欲しいtagにアクセスできる
-    end
+          condition.merge!("data" => data_record)
+          @conditions.push(condition)
+        end
 
-    #conditionを読み込み
-    @conditions = []
-      (1..PATTERN_MAX_NUM).each do |i|
-        next unless conf["condition#{i}"]
-        @conditions.push(conf["condition#{i}"])
-    end
-    if tags.size != @conditions.size
-      raise Fluent::ConfigError, 'match the numbers of tags and conditions; number of tags: '+tags.size.to_s+', number of conditions: '+@conditions.size.to_s
-    end
 
+        if @remove_keys
+          @remove_keys = @remove_keys.split(',')
+        end
+        if @keep_keys
+          raise Fluent::ConfigError, 'out_condition_checker: `renew_record` must be true to use `keep_keys`' unless @renew_record
+          @keep_keys = @keep_keys.split(',')
+        end
 
-    # maps[i] で、欲しいconditionに対応するrecordにアクセスすることができる。
-    # FIXME: conf.elements.forEachでいい感じに回せそう
-    maps = []
-    (1..PATTERN_MAX_NUM).each do |i|
-      next unless conf["condition#{i}"] # 対応するcondition{i}が定義されているものだけ読み込む
-      conf.elements.select { |element| element.name == 'record'+i.to_s }.each { |element|
-        recordTmp = {}
-        element.each_pair { |k, v|
-          element.has_key?(k) # to suppress unread configuration warning
-          recordTmp.merge!({k => parse_value(v)})
-          # map_if_false[k] = parse_value(v)
+        placeholder_expander_params = {
+            :log           => log,
+            :auto_typecast => @auto_typecast, # It should always be true
         }
-        maps[i] = recordTmp
-      }
-    end
 
-    map_else ={}
-    conf.elements.select { |element| element.name == 'record_else' }.each { |element|
-      recordTmp = {}
-      element.each_pair { |k, v|
-      element.has_key?(k) # to suppress unread configuration warning
-        recordTmp.merge!({k => parse_value(v)})
-      }
-      map_else = recordTmp
-    }
-
-    if @remove_keys
-      @remove_keys = @remove_keys.split(',')
-    end
-    if @keep_keys
-      raise Fluent::ConfigError, 'out_condition_checker: `renew_record` must be true to use `keep_keys`' unless @renew_record
-      @keep_keys = @keep_keys.split(',')
-    end
+        @placeholder_expander =
+            if @enable_ruby
+              # require utilities which would be used in ruby placeholders
+              require 'pathname'
+              require 'uri'
+              require 'cgi'
+              RubyPlaceholderExpander.new(placeholder_expander_params)
+            else
+              p 'WARN!! Hey! You should enable ruby!!!'
+              PlaceholderExpander.new(placeholder_expander_params)
+            end
 
-    placeholder_expander_params = {
-        :log           => log,
-        :auto_typecast => @auto_typecast, # It should always be true
-    }
-
-    @placeholder_expander =
-        if @enable_ruby
-          # require utilities which would be used in ruby placeholders
-          require 'pathname'
-          require 'uri'
-          require 'cgi'
-          RubyPlaceholderExpander.new(placeholder_expander_params)
-        else
-          p 'WARN!! Hey! You should enable ruby!!!'
-          PlaceholderExpander.new(placeholder_expander_params)
-        end
 
+        @hostname = Socket.gethostname
+      end
 
-    @maps = @placeholder_expander.preprocess_map(maps)
-    @tags = @placeholder_expander.preprocess_map(tags)
-    @tag_else = @placeholder_expander.preprocess_map(conf['tag_else'])
-    @map_else = @placeholder_expander.preprocess_map(map_else)
+      def process(tag, es)
+        tag_parts = tag.split('.')
+        tag_prefix = tag_prefix(tag_parts)
+        tag_suffix = tag_suffix(tag_parts)
+        placeholder_values = {
+            'tag'        => tag,
+            'tags'       => tag_parts, # for old version compatibility
+            'tag_parts'  => tag_parts,
+            'tag_prefix' => tag_prefix,
+            'tag_suffix' => tag_suffix,
+            'hostname'   => @hostname,
+        }
+        es.each {|time, record|
+          placeholder_values.merge!({
+                                        'time'     => @placeholder_expander.time_value(time),
+                                        'record'   => record,
+                                    })
+                                    
+          # TODO: ここの処理よくないって evaluate
+          matched_conditions, aditional_data = evaluate_condition(@conditions, placeholder_values )
+
+          if matched_conditions.size > 0
+            new_record = reform(record, aditional_data, placeholder_values)
+            if @renew_time_key && new_record.has_key?(@renew_time_key)
+              time = new_record[@renew_time_key].to_i
+            end
+            @remove_keys.each {|k| new_record.delete(k) } if @remove_keys
+            router.emit(@tag, time, new_record)
+          end
+        }
+      rescue => e
+        log.warn "condition_checker: #{e.class} #{e.message} #{e.backtrace.first}"
+      end
 
-    @hostname = Socket.gethostname
-  end
+      private
 
-  def process(tag, es)
-    tag_parts = tag.split('.')
-    tag_prefix = tag_prefix(tag_parts)
-    tag_suffix = tag_suffix(tag_parts)
-    placeholder_values = {
-        'tag'        => tag,
-        'tags'       => tag_parts, # for old version compatibility
-        'tag_parts'  => tag_parts,
-        'tag_prefix' => tag_prefix,
-        'tag_suffix' => tag_suffix,
-        'hostname'   => @hostname,
-    }
-    es.each {|time, record|
-      placeholder_values.merge!({
-                                    'time'     => @placeholder_expander.time_value(time),
-                                    'record'   => record,
-                                })
-                                
-      # TODO: ここの処理よくないって evaluate
-      result, idx = evaluate_condition(@conditions, placeholder_values)
-      placeholder_values.merge!({ 'result'   => result })
-
-      if idx
-        new_tag, new_record = reform(@tags[idx], @maps[idx+1], record, placeholder_values)
-      else
-        # TODO: tag_elseは使えなくするoption作る"
-        new_tag, new_record = reform(@tag_else, @map_else, record, placeholder_values)
-        # return
+      def evaluate_condition(conditions, placeholders)
+        matched_conditions = []
+        aditional_data = {}
+        conditions.each_with_index{ |condition, idx|
+          result = expand_placeholders(condition["rule"], placeholders)
+          if result
+            matched_conditions.push(condition["condition"])
+             aditional_data.merge!(condition["data"])
+          end
+        }
+        return matched_conditions, aditional_data
       end
 
-      if new_tag
-        if @renew_time_key && new_record.has_key?(@renew_time_key)
-          time = new_record[@renew_time_key].to_i
+      def parse_value(value_str)
+        if value_str.start_with?('{', '[')
+          JSON.parse(value_str)
+        else
+          value_str
         end
-        @remove_keys.each {|k| new_record.delete(k) } if @remove_keys
-        router.emit(new_tag, time, new_record)
+      rescue => e
+        log.warn "failed to parse #{value_str} as json. Assuming #{value_str} is a string", :error_class => e.class, :error => e.message
+        value_str # emit as string
       end
-    }
-  rescue => e
-    log.warn "condition_checker: #{e.class} #{e.message} #{e.backtrace.first}"
-  end
 
-  private
-
-  def evaluate_condition(conditions, placeholders)
-    conditions.each_with_index{ |condition, idx|
-      result = expand_placeholders(condition, placeholders)
-      if result then return [result, idx] end
-    }
-    [@map_else, nil]
-  end
-
-  def parse_value(value_str)
-    if value_str.start_with?('{', '[')
-      JSON.parse(value_str)
-    else
-      value_str
-    end
-  rescue => e
-    log.warn "failed to parse #{value_str} as json. Assuming #{value_str} is a string", :error_class => e.class, :error => e.message
-    value_str # emit as string
-  end
+      def reform(record, aditional_data, placeholder_values)
+        new_record = @renew_record ? {} : record.dup
+        new_record.merge!(create_record(aditional_data, placeholder_values))
+        @keep_keys.each {|k| new_record[k] = record[k]} if @keep_keys and @renew_record
+        return new_record
+      end
 
-  def reform(tag, map, record, placeholder_values)
-    placeholders = @placeholder_expander.prepare_placeholders(placeholder_values)
-    new_tag = expand_placeholders(tag, placeholders)
-    new_record = @renew_record ? {} : record.dup
-    @keep_keys.each {|k| new_record[k] = record[k]} if @keep_keys and @renew_record
-    new_record.merge!(create_record(map, placeholders)) unless map.nil?
-    [new_tag, new_record]
-  end
+      def create_record(map, placeholders)
+        new_record = {}
+        map.each do |k, v|
+          value = @placeholder_expander.expand(v, placeholders, true)
+          if value.nil?
+            new_record.merge!({ k => convert_num(v) })
+          else
+            new_record.merge!({ k => convert_num(value) })
+          end
+        end
+        pp new_record
 
-  def create_record(map, placeholders)
-    new_record = {}
+        new_record
+      end
 
-    map.each_pair do |k, v|
-      new_key = @placeholder_expander.expand(k, placeholders, true)
+      def convert_num(value)
+        # Booleanがチェック
+        if value == "true"
+          return true
+        elsif value == "false"
+          return false
+        end
+        
+        if value.to_i.to_s == value.to_s
+          return value.to_i
+        else
+          return value
+        end
+      end
 
-      new_record.merge!({ new_key => convert_num(@placeholder_expander.expand(v, placeholders, true)) })
-    end
+      def convert_value(value)
+        # Booleanがチェック
+        return true if value == 'true'
 
-    new_record
-  end
+        return false if value == 'false'
 
-  def convert_num(value)
-    # Booleanがチェック
-    if value == "true"
-      return true
-    elsif value == "false"
-      return false
-    end
-    
-    if value.to_i.to_s == value.to_s
-      return value.to_i
-    else
-      return value
-    end
-  end
+        # 数値データなら数値で返す
+        return value.to_i if value.to_i.to_s == value.to_s
+        return value.to_f if value.to_f.to_s == value.to_s
 
-  def expand_placeholders(value, placeholders)
-    if value.is_a?(String)
-      new_value = @placeholder_expander.expand(value, placeholders)
-    elsif value.is_a?(Hash)
-      new_value = {}
-      value.each_pair do |k, v|
-        new_key = @placeholder_expander.expand(k, placeholders, true)
-        new_value[new_key] = expand_placeholders(v, placeholders)
+        value
       end
-    elsif value.is_a?(Array)
-      new_value = []
-      value.each_with_index do |v, i|
-        new_value[i] = expand_placeholders(v, placeholders)
+
+      def expand_placeholders(value, placeholders)
+        if value.is_a?(String)
+          new_value = @placeholder_expander.expand(value, placeholders)
+        elsif value.is_a?(Hash)
+          new_value = {}
+          value.each_pair do |k, v|
+            new_key = @placeholder_expander.expand(k, placeholders, true)
+            new_value[new_key] = expand_placeholders(v, placeholders)
+          end
+        elsif value.is_a?(Array)
+          new_value = []
+          value.each_with_index do |v, i|
+            new_value[i] = expand_placeholders(v, placeholders)
+          end
+        else
+          new_value = value
+        end
+        new_value
       end
-    else
-      new_value = value
-    end
-    new_value
-  end
 
-  def tag_prefix(tag_parts)
-    return [] if tag_parts.empty?
-    tag_prefix = [tag_parts.first]
-    1.upto(tag_parts.size-1).each do |i|
-      tag_prefix[i] = "#{tag_prefix[i-1]}.#{tag_parts[i]}"
-    end
-    tag_prefix
-  end
+      def tag_prefix(tag_parts)
+        return [] if tag_parts.empty?
+        tag_prefix = [tag_parts.first]
+        1.upto(tag_parts.size-1).each do |i|
+          tag_prefix[i] = "#{tag_prefix[i-1]}.#{tag_parts[i]}"
+        end
+        tag_prefix
+      end
 
-  def tag_suffix(tag_parts)
-    return [] if tag_parts.empty?
-    rev_tag_parts = tag_parts.reverse
-    rev_tag_suffix = [rev_tag_parts.first]
-    1.upto(tag_parts.size-1).each do |i|
-      rev_tag_suffix[i] = "#{rev_tag_parts[i]}.#{rev_tag_suffix[i-1]}"
-    end
-    rev_tag_suffix.reverse!
-  end
+      def tag_suffix(tag_parts)
+        return [] if tag_parts.empty?
+        rev_tag_parts = tag_parts.reverse
+        rev_tag_suffix = [rev_tag_parts.first]
+        1.upto(tag_parts.size-1).each do |i|
+          rev_tag_suffix[i] = "#{rev_tag_parts[i]}.#{rev_tag_suffix[i-1]}"
+        end
+        rev_tag_suffix.reverse!
+      end
 
-  # THIS CLASS MUST BE THREAD-SAFE
-  class RubyPlaceholderExpander
-    attr_reader :log
+      # THIS CLASS MUST BE THREAD-SAFE
+      class RubyPlaceholderExpander
+        attr_reader :log
 
-    def initialize(params)
-      @log = params[:log]
-      @auto_typecast = params[:auto_typecast]
-      @cleanroom_expander = CleanroomExpander.new
-    end
+        def initialize(params)
+          @log = params[:log]
+          @auto_typecast = params[:auto_typecast]
+          @cleanroom_expander = CleanroomExpander.new
+        end
 
-    def time_value(time)
-      Time.at(time)
-    end
+        def time_value(time)
+          Time.at(time)
+        end
 
-    # Preprocess record map to convert into ruby string expansion
-    #
-    # @param [Hash|String|Array] value record map config
-    # @param [Boolean] force_stringify the value must be string, used for hash key
-    def preprocess_map(value, force_stringify = false)
-      new_value = nil
-      if value.is_a?(String)
-        if @auto_typecast and !force_stringify
-          num_placeholders = value.scan('${').size
-          if num_placeholders == 1 and value.start_with?('${') && value.end_with?('}')
-            new_value = value[2..-2] # ${..} => ..
+        # Preprocess record map to convert into ruby string expansion
+        #
+        # @param [Hash|String|Array] value record map config
+        # @param [Boolean] force_stringify the value must be string, used for hash key
+        def preprocess_map(value, force_stringify = false)
+          new_value = nil
+          if value.is_a?(String)
+            if @auto_typecast and !force_stringify
+              num_placeholders = value.scan('${').size
+              if num_placeholders == 1 and value.start_with?('${') && value.end_with?('}')
+                new_value = value[2..-2] # ${..} => ..
+              end
+            end
+            unless new_value
+              new_value = "%Q[#{value.gsub('${', '#{')}]" # xx${..}xx => %Q[xx#{..}xx]
+            end
+          elsif value.is_a?(Hash)
+            new_value = {}
+            value.each_pair do |k, v|
+              new_value[preprocess_map(k, true)] = preprocess_map(v)
+            end
+          elsif value.is_a?(Array)
+            new_value = []
+            value.each_with_index do |v, i|
+              new_value[i] = preprocess_map(v)
+            end
+          else
+            new_value = value
           end
+          new_value
         end
-        unless new_value
-          new_value = "%Q[#{value.gsub('${', '#{')}]" # xx${..}xx => %Q[xx#{..}xx]
-        end
-      elsif value.is_a?(Hash)
-        new_value = {}
-        value.each_pair do |k, v|
-          new_value[preprocess_map(k, true)] = preprocess_map(v)
-        end
-      elsif value.is_a?(Array)
-        new_value = []
-        value.each_with_index do |v, i|
-          new_value[i] = preprocess_map(v)
+
+        # FIXME: 引数返すだけの関数があるので削除
+        def prepare_placeholders(placeholder_values)
+          placeholder_values
         end
-      else
-        new_value = value
-      end
-      new_value
-    end
 
-    # FIXME: 引数返すだけの関数があるので削除
-    def prepare_placeholders(placeholder_values)
-      placeholder_values
-    end
+        # Expand string with placeholders
+        #
+        # @param [String] str
+        def expand(str, placeholders, force_stringify = false)
+          # FIXME: tag情報は使用してなさそうなので, 不必要であれば
+          @cleanroom_expander.expand(
+              str,
+              placeholders['tag'],
+              placeholders['time'],
+              placeholders['record'],
+              placeholders['tag_parts'],
+              placeholders['tag_prefix'],
+              placeholders['tag_suffix'],
+              placeholders['hostname']
+          )
+        rescue => e
+          log.warn "record_reformer: failed to expand `#{str}`", :error_class => e.class, :error => e.message
+          log.warn_backtrace
+          nil
+        end
 
-    # Expand string with placeholders
-    #
-    # @param [String] str
-    def expand(str, placeholders, force_stringify = false)
-      # FIXME: tag情報は使用してなさそうなので, 不必要であれば
-      @cleanroom_expander.expand(
-          str,
-          placeholders['tag'],
-          placeholders['time'],
-          placeholders['record'],
-          placeholders['tag_parts'],
-          placeholders['tag_prefix'],
-          placeholders['tag_suffix'],
-          placeholders['hostname']
-      )
-    rescue => e
-      log.warn "record_reformer: failed to expand `#{str}`", :error_class => e.class, :error => e.message
-      log.warn_backtrace
-      nil
-    end
+        class CleanroomExpander
+          def expand(__str_to_eval__, tag, time, record, tag_parts, tag_prefix, tag_suffix, hostname, force_stringify = true)
+            Thread.current[:record_reformer_record] = record # for old version compatibility
+            instance_eval(__str_to_eval__)
+          end
 
-    class CleanroomExpander
-      def expand(__str_to_eval__, tag, time, record, tag_parts, tag_prefix, tag_suffix, hostname, force_stringify = true)
-        Thread.current[:record_reformer_record] = record # for old version compatibility
-        instance_eval(__str_to_eval__)
-      end
+          # for old version compatibility
+          def method_missing(name)
+            key = name.to_s
+            record = Thread.current[:record_reformer_record]
+            if record.has_key?(key)
+              record[key]
+            else
+              raise NameError, "undefined local variable or method `#{key}'"
+            end
+          end
 
-      # for old version compatibility
-      def method_missing(name)
-        key = name.to_s
-        record = Thread.current[:record_reformer_record]
-        if record.has_key?(key)
-          record[key]
-        else
-          raise NameError, "undefined local variable or method `#{key}'"
+          (Object.instance_methods).each do |m|
+            undef_method m unless m.to_s =~ /^__|respond_to_missing\?|object_id|public_methods|instance_eval|method_missing|define_singleton_method|respond_to\?|new_ostruct_member/
+          end
         end
       end
-
-      (Object.instance_methods).each do |m|
-        undef_method m unless m.to_s =~ /^__|respond_to_missing\?|object_id|public_methods|instance_eval|method_missing|define_singleton_method|respond_to\?|new_ostruct_member/
-      end
-    end
     end
   end
 end
-

data/test/plugin/test_out_condition_checker.rb

@@ -6,8 +6,65 @@ class ConditionCheckerOutputTest < Test::Unit::TestCase
     Fluent::Test.setup
   end
 
-  test "failure" do
-    flunk
+  def create_driver(conf)
+    Fluent::Test::Driver::Output.new(Fluent::Plugin::ConditionCheckerOutput).configure(conf)
+  end
+
+  test "simple_configure" do
+    conf = config_element(
+      'ROOT',
+      '',
+      {'tag' => "speak_chekc"},
+      [
+        config_element(
+          'condition',
+          '',
+          {
+            'rule' => 'record["action"] == "speak"',
+            'condition' => 'speaking'
+          },
+          [
+            config_element(
+              'data',
+              '',
+              {
+                'new_word' => 'record["word"] + "こんにちは"',
+                'aaa' => 1
+              },
+              []
+            )
+          ]
+        ),
+        config_element(
+          'condition',
+          '',
+          {
+            'rule' => 'record["action"] != "speak"',
+            'condition' => 'no_speak'
+          },
+          [
+            config_element(
+              'data',
+              '',
+              {
+                'new_word' => 'record["word"] + "はじめまして"'
+              },
+              []
+            )
+          ]
+        )
+      ]
+    )
+    d = create_driver(conf)
+    d.run(default_tag: 'test') do
+      record = {
+        'action' => "speak",
+        "word" => "aaa"
+      }
+      d.feed(record)
+      events = d.events
+      pp events
+    end
   end
 
   private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-condition-checker
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 1.0.0
 platform: ruby
 authors:
 - wally
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-16 00:00:00.000000000 Z
+date: 2020-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -79,10 +79,15 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".gitignore"
+- ".rubocop.yml"
 - Gemfile
+- Gemfile.lock
 - LICENSE
 - README.md
 - Rakefile
+- example/fluent.conf
+- example/scenario.json
 - fluent-plugin-condition-checker.gemspec
 - lib/fluent/plugin/out_condition_checker.rb
 - test/helper.rb