logster 1.3.4 → 1.4.0.pre

data/test/logster/test_redis_store.rb

@@ -198,28 +198,29 @@ class TestRedisStore < Minitest::Test
   end
 
   def test_search
-    @store.report(Logger::INFO, "test", "A")
-    @store.report(Logger::INFO, "test", "B")
+    @store.report(Logger::INFO, "test", "ABCDEFG")
+    @store.report(Logger::INFO, "test", "TUVWXYZ")
 
     messages = @store.latest
     assert_equal(2, messages.length)
 
-    latest = @store.latest(search: "B")
+    latest = @store.latest(search: "TUVWXYZ")
 
     assert_equal(1, latest.length)
+    assert_equal("TUVWXYZ", latest[0].message)
   end
 
   def test_search_exclude_results
-    @store.report(Logger::INFO, "test", "A")
-    @store.report(Logger::INFO, "test", "B")
+    @store.report(Logger::INFO, "test", "ABCDEFG")
+    @store.report(Logger::INFO, "test", "TUVWXYZ")
 
     messages = @store.latest
     assert_equal(2, messages.length)
 
-    latest = @store.latest(search: "-A")
+    latest = @store.latest(search: "-ABCD")
 
     assert_equal(1, latest.length)
-    assert_equal("B", latest[0].message)
+    assert_equal("TUVWXYZ", latest[0].message)
   end
 
   def test_regex_search
@@ -234,6 +235,128 @@ class TestRedisStore < Minitest::Test
     assert_equal(1, latest.length)
   end
 
+  def test_env_search
+    @store.report(Logger::INFO, "test", "message ABCD", env: { cluster: "business5" })
+    @store.report(Logger::INFO, "test", "message WXYZ", env: { cluster: "business7" })
+
+    messages = @store.latest
+    assert_equal(2, messages.length)
+
+    latest = @store.latest(search: "business5")
+
+    assert_equal(1, latest.length)
+    assert_equal("message ABCD", latest[0].message)
+
+    latest = @store.latest(search: "-business5")
+
+    assert_equal(1, latest.length)
+    assert_equal("message WXYZ", latest[0].message)
+
+    latest = @store.latest(search: /business/)
+
+    assert_equal(2, latest.length)
+    assert_equal(["message ABCD", "message WXYZ"], latest.map(&:message).sort)
+  end
+
+  def test_array_env_search_preserve_env
+    m1_original_env = [{ cluster: "business5" }, { cluster: "standard3" }]
+    m2_original_env = [{ cluster: "business2" }, { cluster: "standard7" }]
+
+    @store.report(Logger::INFO, "test", "message ABCD", env: m1_original_env, count: 2)
+    @store.report(Logger::INFO, "test", "message WXYZ", env: m2_original_env, count: 2)
+
+    messages = @store.latest
+    assert_equal(2, messages.length)
+
+    m1_key = messages[0].key
+    m2_key = messages[1].key
+
+    messages = @store.latest(search: "business")
+    assert_equal(2, messages.size)
+
+    # any hashes that don't match should be stripped from the env
+    # array but only temporarily until it's sent to the client
+    # env array should remain untouched in redis memory
+    assert_equal(["business5"], messages[0].env.map { |env| env["cluster"]})
+    assert_equal(1, messages[0].count)
+    assert_equal(["business2"], messages[1].env.map { |env| env["cluster"]})
+    assert_equal(1, messages[1].count)
+
+    m1 = @store.get(m1_key)
+    m2 = @store.get(m2_key)
+    # original env should preserved in redis memory
+    assert_equal(["business5", "standard3"], m1.env.map { |env| env["cluster"] })
+    assert_equal(["business2", "standard7"], m2.env.map { |env| env["cluster"] })
+  end
+
+  def test_both_env_and_title_match_search
+    @store.report(Logger::INFO, "test", "message", env: [{ cluster: "business15" }])
+    @store.report(Logger::INFO, "test", "message2", env: { cluster: "business15" })
+
+    messages = @store.latest
+    assert_equal(2, messages.size)
+
+    messages = @store.latest(search: "-business15")
+    assert_equal(0, messages.size)
+  end
+
+  def test_data_kept_intact_on_report_when_env_matches_an_ignore_pattern
+    begin
+      Logster.config.allow_grouping = true
+      backtrace = caller
+      message = @store.report(Logger::WARN, "", "my error", env: { whatever: "something", backtrace: backtrace })
+
+      @store.ignore = [
+          Logster::IgnorePattern.new("business")
+      ]
+      message2 = @store.report(Logger::WARN, "", "my error", env: { cluster: "business17", backtrace: backtrace })
+
+      message = @store.get(message.key)
+      assert(Array === message.env)
+      assert_equal(2, message.env.size)
+      # message2 shouldn't vanish even if
+      # its env matches an ignore pattern
+      # however it should be merged with message1
+      assert_equal("business17", message.env[1]["cluster"])
+    ensure
+      # reset so it doesn't affect other tests
+      @store.ignore = nil
+      Logster.config.allow_grouping = false
+    end
+  end
+
+  def test_array_env_negative_search
+    @store.report(Logger::INFO, "test", "message ABCD", env: [{ cluster: "business5" }, { cluster: "standard3" }], count: 2)
+    @store.report(Logger::INFO, "test", "message WXYZ", env: [{ cluster: "business2" }, { cluster: "standard7" }], count: 2)
+
+    messages = @store.latest
+    assert_equal(2, messages.length)
+
+    messages = @store.latest(search: "-business")
+    assert_equal(2, messages.size)
+
+    assert_equal(["standard3"], messages[0].env.map { |env| env["cluster"]})
+    assert_equal(1, messages[0].count)
+    assert_equal(["standard7"], messages[1].env.map { |env| env["cluster"]})
+    assert_equal(1, messages[1].count)
+  end
+
+  def test_negative_search_MUST_not_match_title_in_order_to_include_message
+    @store.report(Logger::INFO, "test", "message ABCD", env: [{ cluster: "business5" }, { cluster: "standard3" }], count: 2)
+
+    messages = @store.latest(search: "-ABCD")
+    assert_equal(0, messages.size) # cause title has ABCD
+  end
+
+  def test_positive_search_looks_at_title_OR_env
+    @store.report(Logger::INFO, "test", "message", env: [{ cluster: "business5 ABCDEFG" }, { cluster: "standard3" }], count: 2)
+
+    messages = @store.latest(search: "ABCDEFG")
+    assert_equal(1, messages.size)
+    assert_equal(1, messages[0].env.size)
+    assert_equal("business5 ABCDEFG", messages[0].env[0]["cluster"])
+  end
+
   def test_backtrace
     @store.report(Logger::INFO, "test", "pattern_1")
     message = @store.latest(limit: 1).first

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logster
 version: !ruby/object:Gem::Version
-  version: 1.3.4
+  version: 1.4.0.pre
 platform: ruby
 authors:
 - UI for viewing logs in Rack
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-12-30 00:00:00.000000000 Z
+date: 2019-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -162,6 +162,7 @@ files:
 - client-app/README.md
 - client-app/app/app.js
 - client-app/app/components/actions-menu.js
+- client-app/app/components/env-tab.js
 - client-app/app/components/message-info.js
 - client-app/app/components/message-row.js
 - client-app/app/components/panel-resizer.js
@@ -185,6 +186,7 @@ files:
 - client-app/app/styles/app.css
 - client-app/app/templates/application.hbs
 - client-app/app/templates/components/actions-menu.hbs
+- client-app/app/templates/components/env-tab.hbs
 - client-app/app/templates/components/message-info.hbs
 - client-app/app/templates/components/message-row.hbs
 - client-app/app/templates/components/panel-resizer.hbs
@@ -201,6 +203,7 @@ files:
 - client-app/testem.js
 - client-app/tests/index.html
 - client-app/tests/integration/components/actions-menu-test.js
+- client-app/tests/integration/components/env-tab-test.js
 - client-app/tests/integration/components/message-info-test.js
 - client-app/tests/integration/components/message-row-test.js
 - client-app/tests/integration/components/panel-resizer-test.js
@@ -258,9 +261,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.0.1
 signing_key: