git-recommend 0.4.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b73d571a58194483a14486ea7254d14473701d08
-  data.tar.gz: fbac66cc87250b307dc314e83a218d5818086afe
+  metadata.gz: 7396b42cb2f6d31e427214554481e4425f20e5b7
+  data.tar.gz: 4ef7cbba5c84eaef068e1c5c2d120e7cc09bab19
 SHA512:
-  metadata.gz: 44f08a73639231b50ca8d20f1f871be6fced27f33ae392efcaa25ae2813bd1862f5438bcb8c612e2a1cd0338c7486314f2c6db687af2e443da119fb546262813
-  data.tar.gz: 2edcb0a49a7b2df8aaf0ff1cca792a62f40bba9a369ee0f4d98f1eeacfba7a60517306ab465f91b08d204b073b43fa91c9e255178a87c15f28773a770373060b
+  metadata.gz: e277ce21a284fc256370aebcb744f4d47dc5516930ce75817763439b4dd6dff8498214c7446d282902ce7945b4cee1aac5d9679b203792a59925ee35593c9ae7
+  data.tar.gz: df30d9cf8bb9d86899fd2c73d0de86613789b444ddc6394eb24caa9754890dfd651aa4406cb24dd05886cb54cd67363e51f29ae58fcd46704993855d41c15ebd

data/bin/git-test

@@ -1,4 +1,4 @@
-#!/usr/bin/env ruby
+#impacted_items!/usr/bin/env ruby
 require_relative '../lib/helper.rb'
 
 GitTest.start(ARGV)

data/lib/git_recommend.rb

@@ -81,6 +81,7 @@ class GitRecommend < Thor
         end
       end
       s.print
+      exit 0
     else
       $stderr.puts "No changed files detected. Change some files or run with --help for more options."
     end

data/lib/git_test.rb

@@ -2,6 +2,9 @@ require 'helper'
 
 class GitTest < Thor
 
+  LHS = 'changed_items'
+  RHS = 'impacted_items'
+
   method_option :change_recommendation, aliases: '-c', type: :string, required: true, desc: "Path to test augmented change recommendation"
   desc "plan", "Generates a prioritized test plan given a test augmented change recommendation (output of 'git-test join')"
   def plan
@@ -12,18 +15,19 @@ class GitTest < Thor
       exit 1
     end
 
+
     test_plan = Hash.new
-    coverage = {"lhs" => Set.new, "rhs" => Set.new}
+    coverage = {LHS => Set.new, RHS => Set.new}
 
     change_rec.each do |row|
       support = row["support"].to_r.to_f
       # find relevant tests
-      ["lhs","rhs"].each do |side|
+      [LHS,RHS].each do |side|
         tests = (row["#{side}_tested_by"].nil? ? [] : row["#{side}_tested_by"].split(","))
         tests.each do |test|
           coverage[side] << row[side]
           if test_plan[test].nil?
-            test_plan[test] = {'cg' => 0, 'lhs' => 0, 'rhs' => 0}
+            test_plan[test] = {'cg' => 0, LHS => 0, RHS => 0}
           end
           test_plan[test]['cg'] += support
           test_plan[test][side] += 1
@@ -35,27 +39,26 @@ class GitTest < Thor
     test_plan.each {|test,v| test_plan[test]['cg'] = v['cg']/max_cg}
 
     # print warnings to stderr if any
-    ["lhs","rhs"].each do |side|
+    [LHS,RHS].each do |side|
       items_in_plan = change_rec.map {|row| row[side]}.to_set
       if coverage[side].size < items_in_plan.size
         uncovered = items_in_plan - coverage[side]
-        type_of_impact = ((side == "lhs") ? "changed" : "impacted")
-        $stderr.puts "#{uncovered.size} #{type_of_impact} items were not covered by at least 1 test. The first 10 were: \n #{uncovered.take(10).join("\n")}"
+        $stderr.puts "#{uncovered.size} #{side.gsub('_',' ')} were not covered by at least 1 test. The first 10 were: \n #{uncovered.take(10).join("\n")}"
       end
     end
     # print testplan to stdout
-    $stdout.puts "test,impact,changed_items_covered,impacted_items_covered"
-    test_plan.sort_by {|k,v| [v['lhs'],v['rhs']].max}.each do |test,v|
-      $stdout.puts "#{test},#{v['cg']},#{v['lhs']},#{v['rhs']}"
+    $stdout.puts "test,impact,#{LHS}_covered,#{RHS}_covered"
+    test_plan.sort_by {|k,v| [v[LHS],v[RHS]].max}.each do |test,v|
+      $stdout.puts "#{test},#{v['cg']},#{v[LHS]},#{v[RHS]}"
     end
   end
 
   method_option :mapping, aliases: '-m', type: :string, required: true, desc: "Path to test<->code mapping file"
   method_option :change_recommendation, aliases: '-c', type: :string, required: true ,desc: "Path to change recommendation"
-  desc "join", "Adds a column \"tested_by\", which for each potentially impacted item lists the tests which test this item."
+  desc "join", "Adds columns which specify which tests are covering each item"
   def join
     begin
-      mapping = CSV.read(options[:mapping], headers: true)
+      mapping = CSV.read(options[:mapping], headers: true, skip_lines: /#/)
     rescue CSV::MalformedCSVError, ArgumentError => e
       $stderr.puts "An error occured when parsing the mapping file '#{options[:mapping]}:\n #{e}"
       exit 1
@@ -70,10 +73,10 @@ class GitTest < Thor
 
 
     # print header
-    CSV {|r| r << %w(lhs rhs support lhs_tested_by rhs_tested_by)}
+    CSV {|r| r << %w(#{LHS} #{RHS} support #{LHS}_tested_by #{RHS}_tested_by)}
     change_rec.each do |row|
-      lhs = row["lhs"]
-      rhs = row["rhs"]
+      lhs = row['lhs']
+      rhs = row['rhs']
       support = row["support"].to_r.to_f
       # find relevant tests
       lhs_tests = []

data/lib/version.rb

@@ -1,5 +1,5 @@
 module Git
   module Recommend
-    VERSION = "0.4.1"
+    VERSION = "0.5.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git-recommend
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Thomas Rolfsnes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-16 00:00:00.000000000 Z
+date: 2017-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor