fluentd-ui 1.0.1 → 1.1.0

data/test/models/fluentd/setting/config_test.rb

@@ -0,0 +1,98 @@
+require "test_helper"
+
+class FluentdConfigTest < ActiveSupport::TestCase
+  def create_config(fixture_name)
+    ::Fluentd::Setting::Config.new(fixture_path(fixture_name))
+  end
+
+  sub_test_case "#delete_element" do
+    test "delete a source" do
+      config = create_config("config/simple.conf")
+      element = config.elements(name: "source").first
+      config.delete_element("source", nil, element)
+      assert_equal(config.formatted.strip, <<CONFIG.chomp)
+<filter dummy>
+  @type stdout
+</filter>
+
+<match dummy>
+  @type stdout
+</match>
+CONFIG
+    end
+
+    test "delete a filter" do
+      config = create_config("config/simple.conf")
+      element = config.elements(name: "filter").first
+      config.delete_element("filter", nil, element)
+      assert_equal(config.formatted.strip, <<CONFIG.chomp)
+<source>
+  @type dummy
+  tag dummy
+</source>
+
+<match dummy>
+  @type stdout
+</match>
+CONFIG
+    end
+
+    test "delete a match" do
+      config = create_config("config/simple.conf")
+      element = config.elements(name: "match").first
+      config.delete_element("match", nil, element)
+      assert_equal(config.formatted.strip, <<CONFIG.chomp)
+<source>
+  @type dummy
+  tag dummy
+</source>
+
+<filter dummy>
+  @type stdout
+</filter>
+CONFIG
+    end
+
+    test "delete all elements under the label" do
+      config = create_config("config/multi-label.conf")
+      input = config.elements(name: "label", arg: "@INPUT").first
+      input.elements.each do |element|
+        config.delete_element("label", "@INPUT", element)
+      end
+      assert_equal(config.formatted.strip, <<CONFIG.chomp)
+<source>
+  @type dummy
+  tag dummy1
+  @label @INPUT
+</source>
+
+<source>
+  @type dummy
+  tag dummy2
+  @label @INPUT
+</source>
+
+<label @MAIN>
+  <filter dummy1>
+    @type stdout
+  </filter>
+  <filter dummy2>
+    @type stdout
+  </filter>
+  <match dummy1>
+    @type stdout
+  </match>
+  <match dummy2>
+    @type stdout
+  </match>
+</label>
+CONFIG
+    end
+
+    test "cannot delete specified element" do
+      config = create_config("config/simple.conf")
+      element = ::Fluent::Config::Element.new("source", nil, { :@type => "dummy", :tag => "dummy" }, [])
+      assert_nil(config.delete_element("source", nil, element))
+    end
+  end
+end

data/test/support/configurable_daemon_settings.rb

@@ -15,7 +15,7 @@ module ConfigurableDaemonSettings
       end
       visit(__send__("daemon_setting_#{@type}_path"))
       within("form") do
-        fill_in(@form_name.capitalize, with: @form_value)
+        fill_in(@form_name.humanize, with: @form_value)
       end
       click_button(I18n.t("fluentd.common.finish"))
       assert do

data/test/system/fluentd/setting/filter_grep_test.rb

@@ -0,0 +1,96 @@
+require "application_system_test_case"
+
+class FilterGrepTest < ApplicationSystemTestCase
+  setup do
+    login_with(FactoryBot.build(:user))
+    @daemon = stub_daemon
+  end
+
+  test "show form" do
+    visit(daemon_setting_filter_grep_path)
+    assert do
+      page.has_css?("input[name=\"setting[label]\"]")
+    end
+    assert do
+      page.has_css?("input#setting_and_0_regexp_0__key")
+    end
+    assert do
+      page.has_css?("input#setting_and_0_regexp_0__pattern")
+    end
+    assert do
+      page.has_css?("input#setting_or_0_regexp_0__key")
+    end
+    assert do
+      page.has_css?("input#setting_or_0_regexp_0__pattern")
+    end
+  end
+
+  test "append and" do
+    visit(daemon_setting_filter_grep_path)
+    first(".card-header .btn .fa-plus").click
+    assert do
+      page.has_css?("input#setting_and_1_regexp_0__key")
+    end
+    assert do
+      page.has_css?("input#setting_and_1_regexp_0__pattern")
+    end
+    first(".card-header .btn .fa-minus").click
+    assert do
+      !page.has_css?("input#setting_and_1_regexp_0__key")
+    end
+    assert do
+      !page.has_css?("input#setting_and_1_regexp_0__pattern")
+    end
+  end
+
+  test "append regexp" do
+    visit(daemon_setting_filter_grep_path)
+    first(".card-body .btn .fa-plus").click
+    assert do
+      page.has_css?("input#setting_and_0_regexp_1__key")
+    end
+    assert do
+      page.has_css?("input#setting_and_0_regexp_1__pattern")
+    end
+    first(".card-body .btn .fa-minus").click
+    assert do
+      !page.has_css?("input#setting_and_0_regexp_1__key")
+    end
+    assert do
+      !page.has_css?("input#setting_and_0_regexp_1__pattern")
+    end
+  end
+
+  test "toggle regexp/exclude" do
+    visit(daemon_setting_filter_grep_path)
+    assert_equal(first(".card-body label").text, "Regexp")
+    first('input[name="setting[and[0]][grep_type]"][value="exclude"]').click()
+    assert_equal(first(".card-body label").text, "Exclude")
+    assert do
+      !page.has_css?("input#setting_and_0_regexp_0__key")
+    end
+    assert do
+      !page.has_css?("input#setting_and_0_regexp_0__pattern")
+    end
+    assert do
+      page.has_css?("input#setting_and_0_exclude_0__key")
+    end
+    assert do
+      page.has_css?("input#setting_and_0_exclude_0__pattern")
+    end
+  end
+
+  test "update config" do
+    visit(daemon_setting_filter_grep_path)
+    within("form") do
+      fill_in("setting_label", with: "@INPUT")
+      fill_in("setting_pattern", with: "pattern")
+      fill_in("setting_and_0_regexp_0__key", with: "message")
+      fill_in("setting_and_0_regexp_0__pattern", with: "pattern")
+    end
+    click_button(I18n.t("fluentd.common.finish"))
+    assert do
+      @daemon.agent.config.include?("@INPUT")
+    end
+  end
+end

data/test/system/fluentd/setting/filter_parser_test.rb

@@ -0,0 +1,13 @@
+require "application_system_test_case"
+
+class FilterParserTest < ApplicationSystemTestCase
+  include ConfigurableDaemonSettings
+
+  setup do
+    login_with(FactoryBot.build(:user))
+    @type = "filter_parser"
+    @form_name = "key_name"
+    @form_value = "message"
+    @daemon = stub_daemon
+  end
+end

data/test/system/fluentd/setting/filter_record_transformer_test.rb

@@ -0,0 +1,30 @@
+require "application_system_test_case"
+
+class FilterRecordTransformerTest < ApplicationSystemTestCase
+  setup do
+    login_with(FactoryBot.build(:user))
+    @daemon = stub_daemon
+  end
+
+  test "show form" do
+    visit(daemon_setting_filter_record_transformer_path)
+    assert do
+      page.has_css?("textarea[name=\"setting[record]\"]")
+    end
+  end
+
+  test "update config" do
+    value = "key value"
+    assert do
+      !@daemon.agent.config.include?(value)
+    end
+    visit(daemon_setting_filter_record_transformer_path)
+    within("form") do
+      fill_in("Record", with: value)
+    end
+    click_button(I18n.t("fluentd.common.finish"))
+    assert do
+      @daemon.agent.config.include?(value)
+    end
+  end
+end

data/test/system/fluentd/setting/filter_stdout_test.rb

@@ -0,0 +1,13 @@
+require "application_system_test_case"
+
+class FilterStdoutTest < ApplicationSystemTestCase
+  include ConfigurableDaemonSettings
+
+  setup do
+    login_with(FactoryBot.build(:user))
+    @type = "filter_stdout"
+    @form_name = "pattern"
+    @form_value = "stdout.**"
+    @daemon = stub_daemon
+  end
+end

data/test/system/source_and_output_test.rb

@@ -50,14 +50,21 @@ class SourceAndOutputTest < ApplicationSystemTestCase
     end
 
     test "elements" do
-      assert do
-        !page.has_content?(I18n.t("fluentd.settings.source_and_output.setting_empty"))
+      within(".input .card") do
+        assert do
+          !has_content?(I18n.t("fluentd.settings.source_and_output.setting_empty"))
+        end
       end
-     assert do
-        page.has_css?('.input .card .card-header')
+      assert do
+        has_css?(".input .card .card-header")
       end
       assert do
-        page.has_css?('.output .card .card-header')
+        has_css?(".output .card .card-header")
+      end
+      within(".filter .empty") do
+        assert do
+          has_content?(I18n.t("fluentd.settings.source_and_output.setting_empty"))
+        end
       end
     end
 
@@ -79,20 +86,9 @@ class SourceAndOutputTest < ApplicationSystemTestCase
     end
 
     test "display plugin name" do
-      within ".input" do
-        assert do
-          page.has_content?("forward")
-        end
-      end
-
-      within ".output" do
-        assert do
-          page.has_content?("stdout")
-        end
-        assert do
-          page.has_content?("s3")
-        end
-      end
+      assert_equal(first(".input .card .card-header").text, "forward")
+      assert_equal(all(".output .card .card-header").map(&:text),
+                   ["stdout (debug.*)", "s3 (s3.*)"])
     end
 
     sub_test_case "edit, update, delete" do
@@ -181,4 +177,157 @@ class SourceAndOutputTest < ApplicationSystemTestCase
       end
     end
   end
+
+  sub_test_case "filter" do
+    setup do
+      config = <<-CONFIG.strip_heredoc
+        <source>
+           @type dummy
+           tag debug.*
+        </source>
+
+        <filter debug.*>
+          @type stdout
+        </filter>
+
+        <match debug.*>
+          # http://docs.fluentd.org/articles/out_stdout
+          type stdout
+        </match>
+      CONFIG
+      @daemon.agent.config_write(config)
+      visit(source_and_output_daemon_setting_path)
+    end
+
+    test "elements" do
+      assert_equal(first(".filter .card .card-header").text, "stdout (debug.*)")
+    end
+  end
+
+  sub_test_case "label" do
+    setup do
+      @config = <<-CONFIG.strip_heredoc
+        <source>
+           @type dummy
+           tag debug.*
+           @label @INPUT
+        </source>
+
+        <label @INPUT>
+          <filter debug.*>
+            @type grep
+            <regexp>
+              key message
+              pattern /debug.+/
+            </regexp>
+          </filter>
+
+          <match debug.*>
+            @type stdout
+          </match>
+        </label>
+      CONFIG
+      @daemon.agent.config_write(@config)
+      visit(source_and_output_daemon_setting_path)
+    end
+
+    test "elements under @INPUT" do
+      assert do
+        all(".input h5").map(&:text).include?("@INPUT")
+      end
+      assert_equal(first(".input .card .card-header").text, "dummy")
+      assert do
+        all(".filter h5").map(&:text).include?("@INPUT")
+      end
+      assert_equal(first(".filter .card .card-header").text, "grep (debug.*)")
+      assert do
+        all(".output h5").map(&:text).include?("@INPUT")
+      end
+      assert_equal(first(".output .card .card-header").text, "stdout (debug.*)")
+    end
+
+    test "click delete button" do
+      assert do
+        find(".filter .card-header").click
+      end
+      page.accept_confirm do
+        find(".filter .btn", text: I18n.t("terms.destroy")).click
+      end
+      within(".filter .empty") do
+        assert do
+          has_content?(I18n.t("fluentd.settings.source_and_output.setting_empty"))
+        end
+      end
+    end
+
+    test "click edit and cancel" do
+      config = <<-CONFIG.strip_heredoc
+        <filter debug.*>
+          @type grep
+          <regexp>
+            key message
+            pattern /debug.+/
+          </regexp>
+        </filter>
+      CONFIG
+      new_config = <<-CONFIG.strip_heredoc
+        <filter debug.*>
+          @type grep
+          <regexp>
+            key message
+            pattern /debug2.+/
+          </regexp>
+        </filter>
+      CONFIG
+      assert do
+        find(".filter .card-header").click
+      end
+      find(".filter .btn", text: I18n.t("terms.edit")).click
+      original_contents = page.evaluate_script(%Q!document.querySelector(".CodeMirror").CodeMirror.getValue()!)
+      assert_equal(original_contents, config)
+      page.execute_script(<<-JS)
+        var cm = document.querySelector('.CodeMirror').CodeMirror;
+        cm.setValue(#{new_config.to_json});
+      JS
+      find(".filter .btn", text: I18n.t("terms.cancel")).click
+      contents = page.evaluate_script("document.querySelector('.filter pre').textContent")
+      assert_equal(contents, config)
+      assert_equal(@daemon.agent.config.strip, @config.strip)
+    end
+
+    test "click edit and save" do
+      config = <<-CONFIG.strip_heredoc
+        <filter debug.*>
+          @type grep
+          <regexp>
+            key message
+            pattern /debug.+/
+          </regexp>
+        </filter>
+      CONFIG
+      new_config = <<-CONFIG.strip_heredoc
+        <filter debug.*>
+          @type grep
+          <regexp>
+            key message
+            pattern /debug2.+/
+          </regexp>
+        </filter>
+      CONFIG
+      assert do
+        find(".filter .card-header").click
+      end
+      find(".filter .btn", text: I18n.t("terms.edit")).click
+      original_contents = page.evaluate_script(%Q!document.querySelector(".CodeMirror").CodeMirror.getValue()!)
+      assert_equal(original_contents, config)
+      page.execute_script(<<-JS)
+        var cm = document.querySelector('.CodeMirror').CodeMirror;
+        cm.setValue(#{new_config.to_json});
+      JS
+      find(".filter .btn", text: I18n.t("terms.save")).click
+      sleep(1)
+      contents = page.evaluate_script("document.querySelector('.filter pre').textContent")
+      assert_equal(new_config, contents)
+    end
+  end
 end