lazylead 0.9.1 → 0.10.1

data/lib/lazylead/task/loading.rb

@@ -72,8 +72,9 @@ module Lazylead
         "#{id} has #{total} tasks"
       end
 
-      def sprints
-        @tasks.group_by(&:sprint).sort
+      def sprints(label)
+        return @tasks.group_by(&:sprint).sort if label.nil? || label.blank?
+        @tasks.group_by { |t| t.sprint(label) }.sort
       end
     end
 

data/lib/lazylead/task/micromanager.rb

@@ -43,20 +43,39 @@ module Lazylead
       def run(sys, postman, opts)
         allowed = opts.slice "allowed", ","
         dues = sys.issues(opts["jql"], opts.jira_defaults.merge(expand: "changelog"))
-                  .map { |i| Due.new(i, allowed) }
+                  .map { |i| Due.new(i, allowed, since(opts)) }
                   .select(&:illegal?)
+                  .reject(&:obsolete?)
         return if dues.empty?
         postman.send opts.merge(dues: dues)
       end
+
+      # Detect history period where search should start.
+      #
+      # opts["period"] The default period for past is 1 day (86400 seconds).
+      #                So, if now 2017-04-06 15:50:58.674+0000
+      #                it returns 2017-04-05 15:50:58 +0000
+      #
+      # opts["now"]    The current time for unit tests.
+      #                If absent the "Time.now" is used.
+      #
+      def since(opts)
+        @since ||= if opts.key? "now"
+                     Time.parse(opts["now"])
+                   else
+                     Time.now - opts.fetch("period", "86400").to_i
+                   end
+      end
     end
 
     # Instance of "Due" history item for the particular ticket.
     class Due
       attr_reader :issue, :when
 
-      def initialize(issue, allowed)
+      def initialize(issue, allowed, since)
         @issue = issue
         @allowed = allowed
+        @since = since
       end
 
       # Gives true when last change of "Due Date" field was done
@@ -66,6 +85,11 @@ module Lazylead
         @allowed.none? { |a| a.eql? last.id }
       end
 
+      # Give true when "Due Date" changes happens in past and its alert already sent.
+      def obsolete?
+        @when < @since
+      end
+
       # Detect details about last change of "Due Date" to non-null value
       def last
         @last ||= begin
@@ -76,7 +100,7 @@ module Lazylead
             @when = @issue["created"]
             dd = @issue.reporter
           else
-            @when = dd["created"].to_date
+            @when = dd["created"]
             dd = Lazylead::User.new(dd["author"])
           end
           dd

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

@@ -27,8 +27,10 @@ require "tempfile"
 require "nokogiri"
 require "backtrace"
 require "active_support/core_ext/hash/conversions"
+require_relative "../../os"
 require_relative "../../salt"
 require_relative "../../opts"
+require_relative "svn"
 
 module Lazylead
   module Task
@@ -43,15 +45,13 @@ module Lazylead
         end
 
         def run(_, postman, opts)
-          cmd = [
-            "svn log --diff --no-auth-cache",
-            "--username #{opts.decrypt('svn_user', 'svn_salt')}",
-            "--password #{opts.decrypt('svn_password', 'svn_salt')}",
-            "-r#{opts['since_rev']}:HEAD #{opts['svn_url']}"
-          ]
-          stdout = `#{cmd.join(" ")}`
-          stdout.scrub!
-          send_email postman, opts.merge(stdout: stdout) unless stdout.blank?
+          stdout = OS.new.run "svn log --diff --no-auth-cache",
+                              "--username #{opts.decrypt('svn_user', 'svn_salt')}",
+                              "--password #{opts.decrypt('svn_password', 'svn_salt')}",
+                              "-r#{opts['since_rev']}:HEAD #{opts['svn_url']}"
+          return if stdout.blank?
+          commits = Lazylead::Svn::Commits.new(stdout)
+          send_email postman, opts.merge(commits: commits) unless commits.empty?
         end
 
         # Send email with svn log as an attachment.

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

@@ -25,8 +25,10 @@
 require "tmpdir"
 require "nokogiri"
 require "active_support/core_ext/hash/conversions"
+require_relative "../../os"
 require_relative "../../salt"
 require_relative "../../opts"
+require_relative "svn"
 
 module Lazylead
   module Task
@@ -47,14 +49,12 @@ module Lazylead
 
         # Return all svn commits for particular date range in repo
         def svn_log(opts)
-          cmd = [
-            "svn log --diff --no-auth-cache",
-            "--username #{opts.decrypt('svn_user', 'svn_salt')}",
-            "--password #{opts.decrypt('svn_password', 'svn_salt')}",
-            "-r {#{from(opts)}}:{#{now(opts)}} #{opts['svn_url']}"
-          ]
-          stdout = `#{cmd.join(" ")}`
-          stdout.split("-" * 72).reject(&:blank?).reverse.map { |e| Entry.new(e) }
+          stdout = OS.new.run "svn log --diff --no-auth-cache",
+                              "--username #{opts.decrypt('svn_user', 'svn_salt')}",
+                              "--password #{opts.decrypt('svn_password', 'svn_salt')}",
+                              "-r {#{from(opts)}}:{#{now(opts)}} #{opts['svn_url']}"
+          return [] if stdout.blank?
+          Lazylead::Svn::Commits.new(stdout)
         end
 
         # The start date & time for search range
@@ -73,67 +73,4 @@ module Lazylead
       end
     end
   end
-
-  # Single SVN commit details
-  class Entry
-    def initialize(commit)
-      @commit = commit
-    end
-
-    def to_s
-      "#{rev} #{msg}"
-    end
-
-    def rev
-      header.first[1..]
-    end
-
-    def author
-      header[1]
-    end
-
-    def time
-      header[2]
-    end
-
-    def msg
-      lines[1]
-    end
-
-    # The modified lines contains expected text
-    def includes?(text)
-      text = [text] unless text.respond_to? :each
-      lines[4..].select { |l| l.start_with? "+" }
-                .any? { |l| text.any? { |t| l.include? t } }
-    end
-
-    def lines
-      @lines ||= @commit.split("\n").reject(&:blank?)
-    end
-
-    def header
-      @header ||= lines.first.split(" | ").reject(&:blank?)
-    end
-
-    # 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
-    end
-
-    # Detect affected files with particular text
-    def affected(text)
-      occurrences = lines.each_index.select do |i|
-        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..]
-      end
-    end
-  end
 end

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

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+# The MIT License
+#
+# Copyright (c) 2019-2021 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 "forwardable"
+
+module Lazylead
+  module Svn
+    #
+    # SVN commit built from command-line stdout (OS#run).
+    #
+    class Commit
+      def initialize(raw)
+        @raw = raw
+      end
+
+      def to_s
+        "#{rev} #{msg}"
+      end
+
+      def rev
+        header.first[1..]
+      end
+
+      def author
+        header[1]
+      end
+
+      def time
+        DateTime.parse(header[2]).strftime("%d-%m-%Y %H:%M:%S")
+      end
+
+      def msg
+        lines[1]
+      end
+
+      def lines
+        @lines ||= @raw.split("\n").reject(&:blank?)
+      end
+
+      def header
+        @header ||= lines.first.split(" | ").reject(&:blank?)
+      end
+
+      # The modified lines contains expected text
+      def includes?(text)
+        text = [text] unless text.respond_to? :each
+        lines[4..].select { |l| l.start_with? "+" }.any? { |l| text.any? { |t| l.include? t } }
+      end
+
+      # Detect SVN diff lines with particular text
+      def diff(text)
+        @diff ||= begin
+          files = affected(text).uniq
+          @raw.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
+      def affected(text)
+        occurrences = lines.each_index.select do |i|
+          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..]
+        end
+      end
+    end
+
+    #
+    # SVN commits built from command-line stdout (OS#run).
+    #
+    class Commits
+      extend Forwardable
+      def_delegators :@all, :each, :map, :select, :empty?
+
+      # @todo #/DEV Find a way how to avoid @all initialization directly in constructor.
+      #  There should be a way how to make lazy initialization of @all with 'forwardable'.
+      def initialize(stdout)
+        @all = stdout.split("-" * 72).reject(&:blank?).reverse.map { |e| Commit.new(e) }
+      end
+    end
+  end
+end

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

@@ -25,6 +25,7 @@
 require "tmpdir"
 require "nokogiri"
 require "active_support/core_ext/hash/conversions"
+require_relative "../../os"
 require_relative "../../salt"
 require_relative "../../opts"
 
@@ -64,13 +65,10 @@ module Lazylead
                   DateTime.now
                 end
           start = (now.to_time - opts["period"].to_i).to_datetime
-          cmd = [
-            "svn log --no-auth-cache",
-            "--username #{opts.decrypt('svn_user', 'svn_salt')}",
-            "--password #{opts.decrypt('svn_password', 'svn_salt')}",
-            "--xml -v -r {#{start}}:{#{now}} #{opts['svn_url']}"
-          ]
-          raw = `#{cmd.join(" ")}`
+          raw = OS.new.run "svn log --no-auth-cache",
+                           "--username #{opts.decrypt('svn_user', 'svn_salt')}",
+                           "--password #{opts.decrypt('svn_password', 'svn_salt')}",
+                           "--xml -v -r {#{start}}:{#{now}} #{opts['svn_url']}"
           Nokogiri.XML(raw, nil, "UTF-8")
         end
 

data/lib/lazylead/version.rb

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

data/lib/messages/illegal_assignee_change.erb

@@ -102,8 +102,7 @@
     <tr>
       <td><a href='<%= a.issue.url %>'><%= a.issue.key %></a></td>
       <td><%= a.issue.priority %></td>
-      <td><%= DateTime.parse(a.last["created"])
-                      .strftime('%d-%b-%Y %I:%M:%S %p') %></td>
+      <td><%= DateTime.parse(a.last["created"]).strftime('%d-%b') %></td>
       <td><span style='color: red'><%= a.last["author"]["displayName"] %></span>
         (<%= a.last["author"]["name"] %>)
       </td>

data/lib/messages/illegal_duedate_change.erb

@@ -95,18 +95,20 @@
     <th id="when">When</th>
     <th id="who">Who</th>
     <th id="to">To</th>
-    <th id="summary">Summary</th>
+    <th id="assignee">Assignee</th>
     <th id="reporter">Reporter</th>
+    <th id="summary">Summary</th>
   </tr>
   <% dues.each do |d| %>
     <tr>
       <td><a href='<%= d.issue.url %>'><%= d.issue.key %></a></td>
       <td><%= d.issue.priority %></td>
-      <td><%= d.when %></td>
+      <td><%= d.when.to_date %></td>
       <td><span style='color: red'><%= d.last.name %></span> (<%= d.last.id %>)</td>
       <td><span style='color: red'><%= d.issue.duedate %></span></td>
-      <td><%= d.issue.summary %></td>
+      <td><%= d.issue.assignee.name %></td>
       <td><%= d.issue.reporter.name %></td>
+      <td><%= d.issue.summary %></td>
     </tr>
   <% end %>
 </table>

data/lib/messages/loading.erb

@@ -73,7 +73,7 @@
     <th id="total">
       <a href="<%= search_link %><%= CGI.escape(jql) %>">Assigned From</a>
     </th>
-    <th id="duedate">Next Due date</th>
+    <th id="duedate">Next due date</th>
   </tr>
   <% assignments.each do |teammate, assignment| %>
     <% if assignment.free? %>
@@ -89,7 +89,7 @@
         <% if assignment.free? %>
           <span style="color: red">0</span>
         <% else %>
-          <% assignment.sprints.each do |s| %>
+          <% assignment.sprints(defined?(sprint) ? sprint : "customfield_10480").each do |s| %>
             <a href="<%= search_link %><%= CGI.escape("#{jql} and assignee=#{teammate}") %>">
               <%= s.first.blank? ? "No sprint" : s.first %>: <%= s.last.size %>
             </a>

data/lib/messages/svn_diff.erb

@@ -64,20 +64,13 @@
       padding: 36px
     }
 
-    .commit {
-      min-width: 100%;
-      border-radius: 3.5px;
-      overflow: hidden;
-      display: inline-block;
-      line-height: 15px;
-      font-family: Menlo, Monaco, Consolas, "Courier New", monospace;
-      border: 1px solid #BCC6CC;
+    th, td {
+      padding-left: 3px;
+      padding-right: 3px;
     }
 
-    .commit * {
-      padding-left: 4px;
-      font-size: 8px;
-      line-height: 12px;
+    th {
+      font-weight: bold;
     }
   </style>
   <title>SVN log</title>
@@ -86,23 +79,30 @@
 <p>Hi,</p>
 <p>The following file(s) changed since <code><%= since_rev %></code> rev in
   <a href="<%= svn_url %>"><%= svn_url %></a>:</p>
-<% stdout.split("------------------------------------------------------------------------").reject(&:blank?).reverse.each do |commit| %>
-  <div class="commit">
-    <% commit.split("\n").reject(&:blank?).each_with_index do |line, index| %>
-      <p style="background: gainsboro;">
-        <% if index.zero? %>
-          <% details = line.split("|").map(&:strip).reject(&:blank?) %>
-          <a href="<%= commit_url %><%= details[0][1..-1] %>"><%= details[0] %></a>
-          by <a href="<%= user %><%= details[1] %>"><%= details[1] %></a>
-          at <span style="color: #275a90;"><%= details[2] %></span>
-        <% end %>
-        <% if index == 1 %>
-          <span style="padding-left: 4px"><%= line %></span>
-        <% end %>
-      </p>
-    <% end %>
-  </div>
-<% end %>
+<table summary="diff">
+  <tr>
+    <th id="rev">rev</th>
+    <th id="author">Author</th>
+    <th id="time">When</th>
+    <th id="msg">Message</th>
+  </tr>
+  <% commits.each do |commit| %>
+    <tr>
+      <td>
+        <div class="auto">
+          <a href="<%= commit_url %><%= commit.rev %>"><%= commit.rev %></a>
+        </div>
+      </td>
+      <td>
+        <div class="auto">
+          <a href="<%= user %><%= commit.author %>"><%= commit.author %></a>
+        </div>
+      </td>
+      <td><%= commit.time %></td>
+      <td><%= commit.lines[1] %></td>
+    </tr>
+  <% end %>
+</table>
 <p>Posted by
   <a href="https://github.com/dgroup/lazylead">lazylead v<%= version %></a>.
 </p>