lazylead 0.6.2 → 0.7.0

data/lib/lazylead/task/accuracy/accuracy.rb

@@ -114,9 +114,7 @@ module Lazylead
     end
 
     def color
-      if colors.nil? || !defined?(@score) || !@score.is_a?(Numeric)
-        return "#061306"
-      end
+      return "#061306" if colors.nil? || !defined?(@score) || !@score.is_a?(Numeric)
       colors.reverse_each do |color|
         return color.last if @accuracy >= color.first
       end
@@ -125,12 +123,12 @@ module Lazylead
 
     def colors
       @colors ||= begin
-                    JSON.parse(@opts["colors"])
-                        .to_h
-                        .to_a
-                        .each { |e| e[0] = e[0].to_i }
-                        .sort_by { |e| e[0] }
-                  end
+        JSON.parse(@opts["colors"])
+            .to_h
+            .to_a
+            .each { |e| e[0] = e[0].to_i }
+            .sort_by { |e| e[0] }
+      end
     end
 
     # Calculate grade for accuracy

data/lib/lazylead/task/accuracy/attachment.rb

@@ -27,10 +27,6 @@ require_relative "requirement"
 module Lazylead
   # Check that ticket has an attachment.
   class Attachment < Lazylead::Requirement
-    def initialize(desc, score, field)
-      super(desc, score, field)
-    end
-
     def passed(issue)
       issue.attachments.any?(&method(:matching))
     end

data/lib/lazylead/task/accuracy/logs.rb

@@ -27,18 +27,18 @@ require_relative "attachment"
 module Lazylead
   # Check that ticket has log file(s) in attachment.
   class Logs < Lazylead::Attachment
-    def initialize
+    def initialize(files = %w[log.zip logs.zip log.gz logs.gz log.tar.gz
+                              logs.tar.gz log.7z logs.7z log.tar logs.tar])
       super("Log files", 2, "Attachments")
+      @files = files
     end
 
     # Ensure that ticket has a '*.log' file more '5KB'
     def matching(attachment)
       name = attachment.attrs["filename"].downcase
       return false unless attachment.attrs["size"].to_i > 5120
-      return true if File.extname(name).start_with?(".log", ".txt")
-      %w[.log.zip .log.gz .log.tar.gz logs.7z log.7z].any? do |l|
-        name.end_with? l
-      end
+      return true if File.extname(name).start_with? ".log", ".txt", ".out"
+      @files.any? { |l| name.end_with? l }
     end
   end
 end

data/lib/lazylead/task/accuracy/onlyll.rb

@@ -0,0 +1,148 @@
+# frozen_string_literal: true
+
+# The MIT License
+#
+# Copyright (c) 2019-2020 Yurii Dubinka
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom
+# the Software is  furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included
+# in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+# ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE
+# OR OTHER DEALINGS IN THE SOFTWARE.
+
+require_relative "../../log"
+require_relative "../../opts"
+
+module Lazylead
+  module Task
+    #
+    # Ensure that ticket accuracy evaluation labels are set only by LL, not by
+    #  some person.
+    #
+    # The task supports the following features:
+    #  - fetch issues from remote ticketing system by query
+    #  - check the history of labels modification
+    #  - remove evaluation labels if they set not by LL
+    class OnlyLL
+      def initialize(log = Log.new)
+        @log = log
+      end
+
+      def run(sys, postman, opts)
+        found = sys.issues(opts["jql"],
+                           opts.jira_defaults.merge(expand: "changelog"))
+                   .map { |i| Labels.new(i, opts) }
+                   .select(&:exists?)
+                   .reject(&:valid?)
+                   .each(&:remove)
+        postman.send opts.merge(tickets: found) unless found.empty?
+      end
+    end
+  end
+
+  # The ticket with grid labels
+  class Labels
+    attr_reader :issue
+
+    def initialize(issue, opts)
+      @issue = issue
+      @opts = opts
+    end
+
+    # Ensure that issue has evaluation labels for accuracy rules
+    def exists?
+      return false if @issue.labels.nil? || @issue.labels.empty?
+      return false unless @issue.labels.is_a? Array
+      grid.any? { |g| @issue.labels.any? { |l| g.eql? l } }
+    end
+
+    # Compare the score evaluated by LL and current ticket score.
+    # @return true if current score equal to LL evaluation
+    def valid?
+      score.eql?(@issue.labels.sort.find { |l| grid.any? { |g| l.eql? g } })
+    end
+
+    # Find expected ticket score evaluated by LL.
+    # If LL evaluated the same ticket several times (no matter why),
+    #  then the last score would be returned.
+    def score
+      to_l(
+        @issue.history
+              .select { |h| h["author"]["key"].eql? @opts["author"] }
+              .select { |h| to_l(h) }
+              .reverse
+              .first
+      ).fetch("toString", "").split.find { |l| grid.any? { |g| l.eql? g } }
+    end
+
+    # Find history record with labels changes.
+    def to_l(history)
+      return {} if history.nil? || history.empty?
+      history["items"].find { |f| f["field"].eql? "labels" }
+    end
+
+    # Detect the percentage grid for tickets, by default its 0%, 10%, 20%, etc.
+    def grid
+      @grid ||= begin
+        if @opts.key? "grid"
+          @opts.slice("grid", ",")
+        else
+          %w[0% 10% 20% 30% 40% 50% 60% 70% 80% 90% 100%]
+        end
+      end
+    end
+
+    # Remove score labels from the ticket.
+    def remove
+      @issue.labels!(@issue.labels - grid)
+    end
+
+    # Detect the violators with their changes.
+    #   .violators  => ["Tom Hhhh set 40%", "Bob Mmmm set 50%"]
+    def violators
+      @issue.history
+            .reject { |h| h["author"]["key"].eql? @opts["author"] }
+            .select { |h| grid?(h) }
+            .group_by { |h| h["author"]["key"] }
+            .map do |a|
+        "#{a.last.first['author']['displayName']} set #{hacked(a.last)}"
+      end
+    end
+
+    # Ensure that history record has label change related to LL grid labels.
+    # @return true if LL grid labels added
+    def grid?(record)
+      diff(record).any? { |d| d.find { |l| grid.any? { |g| l.eql? g } } }
+    end
+
+    # Detect label diff in single history record from ticket's history.
+    def diff(record)
+      record["items"].select { |f| f["field"].eql? "labels" }
+                     .reject { |f| f["toString"].nil? || f["toString"].blank? }
+                     .map do |f|
+        from = []
+        from = f["fromString"].split unless f["fromString"].nil?
+        f["toString"].split - from
+      end
+    end
+
+    # Hacked score by violator in ticket's history.
+    def hacked(record)
+      diff(record.first)
+        .first
+        .find { |l| grid.any? { |g| l.eql? g } }
+    end
+  end
+end

data/lib/lazylead/task/accuracy/servers.rb

@@ -38,13 +38,22 @@ module Lazylead
 
     def passed(issue)
       return true if @envs.empty?
-      lines = issue["environment"].to_s + "\n" + issue.description
-      lines.split("\n")
-           .reject(&:blank?)
-           .map(&:strip)
-           .flat_map { |l| l.split(" ").map(&:strip) }
-           .select { |w| w.include?("http://") || w.include?("https://") }
-           .any? { |u| @envs.any? { |e| u.match? e } }
+      "#{issue['environment']}\n#{issue.description}".split("\n")
+                                                     .reject(&:blank?)
+                                                     .map(&:strip)
+                                                     .flat_map { |l| l.split.map(&:strip) }
+                                                     .select(&method(:url?))
+                                                     .any?(&method(:match?))
+    end
+
+    # Ensure that particular text contains web url
+    def url?(text)
+      text.include?("http://") || text.include?("https://")
+    end
+
+    # Ensure that particular url matches expected servers urls
+    def match?(url)
+      @envs.any? { |e| url.match? e }
     end
   end
 end

data/lib/lazylead/task/accuracy/stacktrace.rb

@@ -60,7 +60,7 @@ module Lazylead
       words = desc.gsub(/{(c|C)(o|O)(d|D)(e|E)/, " {code")
                   .gsub("}", "} ")
                   .gsub("Caused by:", "Caused_by:")
-                  .split(" ")
+                  .split
                   .map(&:strip)
                   .reject(&:blank?)
       pairs(words, "{code").map { |s| words[s.first..s.last].join("\n") }

data/lib/lazylead/task/propagate_down.rb

@@ -106,7 +106,7 @@ module Lazylead
           v = a.last
           d[k] = expected[k] if v.nil? || v.blank?
           next if v.nil?
-          d[k] = v + "," + expected[k] unless v.to_s.include? expected[k]
+          d[k] = "#{v},#{expected[k]}" unless v.to_s.include? expected[k]
         end
       end
 

data/lib/lazylead/task/svn/grep.rb

@@ -47,21 +47,28 @@ module Lazylead
 
         # Return all svn commits for particular date range in repo
         def svn_log(opts)
-          now = if opts.key? "now"
-                  DateTime.parse(opts["now"])
-                else
-                  DateTime.now
-                end
-          start = (now.to_time - opts["period"].to_i).to_datetime
           cmd = [
             "svn log --diff --no-auth-cache",
             "--username #{opts.decrypt('svn_user', 'svn_salt')}",
             "--password #{opts.decrypt('svn_password', 'svn_salt')}",
-            "-r {#{start}}:{#{now}} #{opts['svn_url']}"
+            "-r {#{from(opts)}}:{#{now(opts)}} #{opts['svn_url']}"
           ]
           stdout = `#{cmd.join(" ")}`
-          stdout.split("-" * 72).reject(&:blank?).reverse
-                .map { |e| Entry.new(e) }
+          stdout.split("-" * 72).reject(&:blank?).reverse.map { |e| Entry.new(e) }
+        end
+
+        # The start date & time for search range
+        def from(opts)
+          (now(opts).to_time - opts["period"].to_i).to_datetime
+        end
+
+        # The current date & time for search range
+        def now(opts)
+          if opts.key? "now"
+            DateTime.parse(opts["now"])
+          else
+            DateTime.now
+          end
         end
       end
     end
@@ -96,8 +103,8 @@ module Lazylead
     # The modified lines contains expected text
     def includes?(text)
       text = [text] unless text.respond_to? :each
-      lines[4..-1].select { |l| l.start_with? "+" }
-                  .any? { |l| text.any? { |t| l.include? t } }
+      lines[4..].select { |l| l.start_with? "+" }
+                .any? { |l| text.any? { |t| l.include? t } }
     end
 
     def lines
@@ -111,12 +118,12 @@ module Lazylead
     # Detect SVN diff lines with particular text
     def diff(text)
       @diff ||= begin
-                  files = affected(text).uniq
-                  @commit.split("Index: ")
-                         .select { |i| files.any? { |f| i.start_with? f } }
-                         .map { |i| i.split "\n" }
-                         .flatten
-                end
+        files = affected(text).uniq
+        @commit.split("Index: ")
+               .select { |i| files.any? { |f| i.start_with? f } }
+               .map { |i| i.split "\n" }
+               .flatten
+      end
     end
 
     # Detect affected files with particular text
@@ -125,7 +132,7 @@ module Lazylead
         lines[i].start_with?("+") && text.any? { |t| lines[i].include? t }
       end
       occurrences.map do |occ|
-        lines[2..occ].reverse.find { |l| l.start_with? "Index: " }[7..-1]
+        lines[2..occ].reverse.find { |l| l.start_with? "Index: " }[7..]
       end
     end
   end

data/lib/lazylead/task/svn/touch.rb

@@ -52,9 +52,7 @@ module Lazylead
           svn_log(opts).xpath("//logentry[paths/path[#{xpath}]]")
                        .map(&method(:to_entry))
                        .each do |e|
-            if e.paths.path.respond_to? :delete_if
-              e.paths.path.delete_if { |p| files.none? { |f| p.include? f } }
-            end
+            e.paths.path.delete_if { |p| files.none? { |f| p.include? f } } if e.paths.path.respond_to? :delete_if
           end
         end
 

data/lib/lazylead/version.rb

@@ -23,5 +23,5 @@
 # OR OTHER DEALINGS IN THE SOFTWARE.
 
 module Lazylead
-  VERSION = "0.6.2"
+  VERSION = "0.7.0"
 end

data/lib/messages/only_ll.erb

@@ -0,0 +1,107 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+  <style> /* CSS styles taken from https://github.com/yegor256/tacit */
+  th {
+    font-weight: 600
+  }
+
+  table tr {
+    border-bottom-width: 2.16px
+  }
+
+  table tr th {
+    border-bottom-width: 2.16px
+  }
+
+  table tr td, table tr th {
+    overflow: hidden;
+    padding: 5.4px 3.6px;
+    line-height: 14px;
+  }
+
+  #summary {
+    text-align: left;
+  }
+
+  .auto {
+    min-width: auto;
+    white-space: nowrap;
+  }
+
+  a {
+    color: #275a90;
+    text-decoration: none
+  }
+
+  a:hover {
+    text-decoration: underline
+  }
+
+  * {
+    border: 0;
+    border-collapse: separate;
+    border-spacing: 0;
+    box-sizing: border-box;
+    margin: 0;
+    max-width: 100%;
+    padding: 0;
+    vertical-align: baseline;
+    font-family: system-ui, "Helvetica Neue", Helvetica, Arial, sans-serif;
+    font-size: 13px;
+    font-stretch: normal;
+    font-style: normal;
+    font-weight: 400;
+    line-height: 29.7px
+  }
+
+  html, body {
+    width: 100%
+  }
+
+  html {
+    height: 100%
+  }
+
+  body {
+    background: #fff;
+    color: #1a1919;
+    padding: 36px
+  }
+  </style>
+  <title>Only LL</title>
+</head>
+<body>
+<p>Hi,</p>
+<p>The LL accuracy labels are cleaned from the following tickets:</p>
+<table summary="table with tickets triage score">
+  <tr>
+    <th id="key">Key</th>
+    <th id="priority">Priority</th>
+    <th id="reporter">Reporter</th>
+    <th id="violators">Violators</th>
+    <th id="summary">Summary</th>
+    <th id="labels">Labels</th>
+  </tr>
+  <% tickets.each do |t| %>
+    <tr>
+      <td>
+        <div class="auto">
+          <a href="<%= t.issue.url %>"><%= t.issue.key %></a>
+        </div>
+      </td>
+      <td><%= t.issue.priority %></td>
+      <td>
+        <div class="auto"><%= t.issue.reporter.name %></div>
+      </td>
+      <td><%= t.violators.join(', ') %></td>
+      <td><%= t.issue.summary %></td>
+      <td><%= t.issue.labels.join(', ') %></td>
+    </tr>
+  <% end %>
+</table>
+<p>Posted by
+  <a href="https://github.com/dgroup/lazylead">lazylead v<%= version %></a>.
+</p>
+</body>
+</html>