codespicuous 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7b264796715a1c06a98b0d5ffdee3492bde6852e
-  data.tar.gz: 2d7151e47dc07ca625c4d9c32789ca18e4444ff2
+  metadata.gz: 32369c66cbcbd810e9205237576757eca230612a
+  data.tar.gz: 1ba22aea1525014e6eefcba42caee9318d6645ed
 SHA512:
-  metadata.gz: 2f574226876b029b8c51c50d128f7dc99a96bd87592f6719723e303e658476e6f4ddeda712c97bb1def21043417952392ef1b92036ff49b709840427b9d47c44
-  data.tar.gz: a5121a0a45b4a15fd9d4336f543e4f51d4161ff7e8243e23e8e688bfad21333c88dbc76e791808e83795d0290d4632403c4a109dfa179ed86bd519be26e9743a
+  metadata.gz: e913239d16e84541c593216c0c31e44825b52422082b188e00143658488be1822ec35a57ddf7df965c1360a994763a069cd4e7c22346e46599482056be6b97dc
+  data.tar.gz: 139729759092fc037bb447161b651d915dbd0b5fca9306e44ad40b03459c9e5865457e982767386ad01812049362c6320e74ead0696e7a6fb24202e609ff9c88

data/lib/codespicuous/codespicuous.rb

@@ -44,7 +44,7 @@ class Codespicuous
 
   def generate_output
     generator = MetricsGenerator.new
-    generator.generate(@commit_history)
+    generator.generate(@config, @commit_history)
   end
 
   def list_committed_repositories

data/lib/codespicuous/codespicuous_config.rb

@@ -9,8 +9,11 @@ class CodespicuousConfig
     @offline = false
     @configuration_file_name = "codespicuous.yaml"
     @input_path = Pathname.new(".")
+    @output_path = Pathname.new(".")
     @list_repositories = false
     @svnlogdir = "svnlog"
+
+    @output_dir_csv_files = "csv_files"
   end
 
   def input_path=(path)
@@ -32,4 +35,12 @@ class CodespicuousConfig
   def path_to_cached_svn_log_dir
     (@input_path + Pathname.new(@svnlogdir)).to_s
   end
+
+  def path_to_output_dir_for_csv_files
+    (@output_path + Pathname.new(@output_dir_csv_files)).to_s
+  end
+
+  def path_to_csv_file(filename)
+    (Pathname.new(path_to_output_dir_for_csv_files) + Pathname.new(filename)).to_s
+  end
 end

data/lib/codespicuous/commithistory.rb

@@ -51,6 +51,10 @@ class CommitHistory
     @repositories.repository_names
   end
 
+  def amount_of_repositories
+    @repositories.amount
+  end
+
   def amount_of_comitters
     @committers.amount
   end
@@ -73,4 +77,36 @@ class CommitHistory
     @commits == commit_history.commits
   end
 
+  def copy_repositories_without_commits
+    repositories = Repositories.new
+    @repositories.each do |repository|
+      repositories.add(repository.clone_without_commits)
+    end
+    repositories
+  end
+
+  def prune_repositories_without_commit
+    @repositories.prune_repositories_without_commit
+  end
+
+  def restrict_to_teams
+    restricted_commit_history = CommitHistory.new
+
+    repositories = copy_repositories_without_commits
+    restricted_commit_history.configure(teams.clone_without_commits, repositories)
+
+    @commits.each do |commit|
+      if @teams.contains_committer?(commit.committer)
+        new_commit = commit.clone
+        new_commit.repository = repositories.repository(commit.repository.name)
+        restricted_commit_history.add_commit(new_commit)
+      end
+
+    end
+
+    restricted_commit_history.prune_repositories_without_commit
+
+    restricted_commit_history
+  end
+
 end

data/lib/codespicuous/commithistory_builder.rb

@@ -24,6 +24,11 @@ class CommitHistoryBuilder
     self
   end
 
+  def without_team
+    @team = nil
+    self
+  end
+
   def at(date)
     @commit_date = date
     add_commit_to_history
@@ -43,7 +48,7 @@ class CommitHistoryBuilder
     commit.date = DateTime.parse(@commit_date)
     commit.repository = @current_repository
     @commit_history.add_commit(commit)
-    @commit_history.add_team_member(@team, @author)
+    @commit_history.add_team_member(@team, @author) if @team
   end
 end
 

data/lib/codespicuous/committer.rb

@@ -9,6 +9,14 @@ class Committer
     @commits = Commits.new
   end
 
+  def clone_without_commits
+    cloned_committer = Committer.new(username)
+    cloned_committer.first_name = first_name
+    cloned_committer.last_name = last_name
+    cloned_committer.email = email
+    cloned_committer
+  end
+
   def self.create_committer(login, firstname, lastname, email)
     committer = Committer.new(login)
     committer.first_name = firstname
@@ -18,7 +26,7 @@ class Committer
   end
 
   def in_team_with_name?(team_name)
-    @team.name == team_name
+    !@team.nil? && @team.name == team_name
   end
 
   def add_commit commit

data/lib/codespicuous/metrics_generator.rb

@@ -2,21 +2,22 @@
 
 class MetricsGenerator
 
-  attr_accessor :commit_history
+  attr_accessor :commit_history, :config
 
-  def generate(commit_history)
+  def generate(config, commit_history)
     @commit_history = commit_history
+    @config = config
     generate_daniel
-    generate_csv
+    generate_csv_files
   end
 
   def generate_daniel
-    daniel = MetricsGeneratorDaniel.new(@commit_history)
+    daniel = MetricsGeneratorDaniel.new(@config, @commit_history)
     daniel.generate
   end
 
-  def generate_csv
-    csv = MetricsGeneratorCsv.new(@commit_history)
+  def generate_csv_files
+    csv = MetricsGeneratorCsvFiles.new(@config, @commit_history)
     csv.generate
   end
 end

data/lib/codespicuous/metrics_generator_csv_files.rb

@@ -0,0 +1,45 @@
+
+class MetricsGeneratorCsvFiles
+
+  attr_reader :all_csv_generator, :teams_csv_generator
+
+  def create_csv_files_output_directory
+    Dir.mkdir(@config.path_to_output_dir_for_csv_files) unless Dir.exists?(@config.path_to_output_dir_for_csv_files)
+  end
+
+  def initialize(config, commit_history)
+    @config = config
+    @all_csv_generator = MetricsGeneratorCsv.new(commit_history)
+    @restricted_history =  commit_history.restrict_to_teams
+    @teams_csv_generator = MetricsGeneratorCsv.new(@restricted_history)
+  end
+
+  def generate
+    create_csv_files_output_directory
+    create_commit_table_with_week_and_repository_info
+    create_commit_table_with_weeks_and_team_commits
+    create_commit_table_with_committers_and_repository_info
+    create_commit_table_with_weeks_and_committers
+  end
+
+  def create_commit_table_with_week_and_repository_info
+    File.write(@config.path_to_csv_file("all_week_and_repository_info.csv"), @all_csv_generator.create_commit_table_with_week_and_repository_info)
+    File.write(@config.path_to_csv_file("teams_week_and_repository_info.csv"), @teams_csv_generator.create_commit_table_with_week_and_repository_info)
+  end
+
+  def create_commit_table_with_weeks_and_team_commits
+    File.write(@config.path_to_csv_file("teams_week_and_team_commits.csv"), @teams_csv_generator.create_commit_table_with_weeks_and_team_commits)
+  end
+
+  def create_commit_table_with_committers_and_repository_info
+    File.write(@config.path_to_csv_file("teams_committer_and_repository_info.csv"), @teams_csv_generator.create_commit_table_with_committers_and_repository_info)
+  end
+
+  def create_commit_table_with_weeks_and_committers
+    File.write(@config.path_to_csv_file("teams_weeks_and_committers_info.csv"), @all_csv_generator.create_commit_table_with_weeks_and_committers)
+    @restricted_history.teams.each do | team |
+      File.write(@config.path_to_csv_file("teams_weeks_and_committers_info_#{team.name}.csv"), @all_csv_generator.create_commit_table_with_weeks_and_committers(team.name))
+    end
+
+  end
+end

data/lib/codespicuous/metrics_generator_daniel.rb

@@ -2,7 +2,7 @@
 
 class MetricsGeneratorDaniel
 
-  def initialize(commit_history)
+  def initialize(config, commit_history)
   end
 
   def generate

data/lib/codespicuous/repositories.rb

@@ -15,10 +15,18 @@ class Repository
     @commits = Commits.new
   end
 
+  def clone_without_commits
+    Repository.new(@name, @url)
+  end
+
   def add_commit commit
     @commits.add(commit)
   end
 
+  def amount_of_commits
+    @commits.amount
+  end
+
   def amount_of_commits_in_week(week_start)
     @commits.amount_of_commits_to_repository_in_week(name, week_start)
   end
@@ -61,6 +69,10 @@ class Repositories
     }
   end
 
+  def prune_repositories_without_commit
+    @repositories.delete_if { |key, value| value.amount_of_commits == 0 }
+  end
+
   def empty?
     @repositories.empty?
   end

data/lib/codespicuous/teams.rb

@@ -8,6 +8,14 @@ class Team
     @members = {}
   end
 
+  def clone_without_commits
+    cloned_team = Team.new(name)
+    members.each do | member |
+      cloned_team.add_member(member.clone_without_commits)
+    end
+    cloned_team
+  end
+
   def add_member(member)
     @members[member.username] = member
     member.team = self
@@ -62,6 +70,14 @@ class Teams
     @teams = {}
   end
 
+  def clone_without_commits
+    cloned_teams = Teams.new
+    teams.each do | name, team|
+      cloned_teams.add(team.clone_without_commits)
+    end
+    cloned_teams
+  end
+
   def find_by_name(name)
     @teams[name]
   end
@@ -76,6 +92,13 @@ class Teams
     }
   end
 
+  def contains_committer?(committer)
+    each_member { |team, member|
+      return true if member.username == committer.username
+    }
+    false
+  end
+
   def committers
     committers = Committers.new
     each_member { |team, committer| committers.add(committer) }

data/lib/codespicuous/version.rb

@@ -1,4 +1,4 @@
 
 module Codespicuous
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/lib/codespicuous.rb

@@ -31,6 +31,7 @@ require_relative 'codespicuous/codespicuous_config.rb'
 require_relative 'codespicuous/metrics_generator.rb'
 require_relative 'codespicuous/metrics_generator_daniel.rb'
 require_relative 'codespicuous/metrics_generator_csv.rb'
+require_relative 'codespicuous/metrics_generator_csv_files.rb'
 
 require_relative 'codespicuous/repository_lister.rb'
 # App

data/spec/codespicuous_spec.rb

@@ -63,7 +63,7 @@ describe "Codespicuous command line" do
   it "generates output" do
     generator = MetricsGenerator.new
     expect(MetricsGenerator).to receive(:new).and_return(generator)
-    expect(generator).to receive(:generate).with(subject.commit_history)
+    expect(generator).to receive(:generate).with(subject.config, subject.commit_history)
     subject.generate_output
   end
 

data/spec/commithistory_spec.rb

@@ -53,5 +53,49 @@ describe "Team commits per week table" do
     expect(@commit_history.amount_of_commits_for_team_in_week("Cheese", DateTime.new(2016,03,28))).to eq 1
   end
 
+  context "extracting a subset of the commit history" do
+
+    before (:each) do
+      @large_commit_history = CommitHistoryBuilder.new.
+        in_repository("one").
+          commits_of("basvodde").of_team("Wine").
+            at("2016-04-18").times(2).
+          commits_of("daniel").of_team("Cheese").
+            at("2016-03-28").
+          commits_of("janne").without_team.
+            at("2016-08-18").times(2).
+        in_repository("cpputest").
+          commits_of("janne").without_team.
+            at("2016-02-29").
+        build
+
+      @restricted_commit_history = @large_commit_history.restrict_to_teams
+    end
+
+    it "will only have two committers in the restricted commit history" do
+      expect(@large_commit_history.amount_of_comitters).to eq 3
+      expect(@restricted_commit_history.amount_of_comitters).to eq 2
+    end
+
+    it "will only have the repositories of the restricted committers" do
+      expect(@large_commit_history.amount_of_repositories).to eq 2
+      expect(@restricted_commit_history.amount_of_repositories).to eq 1
+    end
+
+    it "will also be able to get the right amount of commits via the reposities" do
+      expect(@large_commit_history.repository("one").amount_of_commits).to eq 5
+      expect(@restricted_commit_history.repository("one").amount_of_commits).to eq 3
+    end
+
+    it "will also be able to get the right amount of commits via the comitter" do
+      expect(@large_commit_history.committer("basvodde").amount_of_commits).to eq 2
+      expect(@restricted_commit_history.committer("basvodde").amount_of_commits).to eq 2
+    end
+
+    it "will also be able to get the right amount of team members in the teams" do
+      expect(@large_commit_history.team("Wine").amount_of_members).to eq 1
+      expect(@restricted_commit_history.team("Wine").amount_of_members).to eq 1
+    end
+  end
 end
 

data/spec/committers_spec.rb

@@ -34,6 +34,11 @@ describe "The committers and the teams that we're examining" do
     expect(@teams.amount).to eq 2
   end
 
+  it "is not in the team with name if th committer is not in a team" do
+    committer = Committer.new("lonely_guy")
+    expect(committer.in_team_with_name?("nice team")).to eq false
+  end
+
   it "has all the committer information" do
     bas = @committers.find_by_username('basvodde')
     expect(bas.first_name).to eq "Bas"

data/spec/metrics_generator_csv_files_spec.rb

@@ -0,0 +1,77 @@
+
+describe "Generate csv metric files" do
+
+  before (:each) do
+    @commit_history = CommitHistory.new
+    @generator =  MetricsGeneratorCsvFiles.new(CodespicuousConfig.new, @commit_history)
+  end
+
+  it "generates different type of csv file metrics" do
+
+    expect(@generator).to receive(:create_csv_files_output_directory)
+    expect(@generator).to receive(:create_commit_table_with_week_and_repository_info)
+    expect(@generator).to receive(:create_commit_table_with_weeks_and_team_commits)
+    expect(@generator).to receive(:create_commit_table_with_committers_and_repository_info)
+    expect(@generator).to receive(:create_commit_table_with_weeks_and_committers)
+
+    @generator.generate
+  end
+
+  it "Should create an output directory when it doesn't exists" do
+    expect(Dir).to receive(:exists?).with("csv_files").and_return(false)
+    expect(Dir).to receive(:mkdir).with("csv_files")
+    @generator.create_csv_files_output_directory
+  end
+
+  it "Should not create an output directory when it does exists" do
+    expect(Dir).to receive(:exists?).with("csv_files").and_return(true)
+    expect(Dir).not_to receive(:mkdir).with("csv_files")
+    @generator.create_csv_files_output_directory
+  end
+
+  it "can generate the week and repository files" do
+    expect(@generator.all_csv_generator).to receive(:create_commit_table_with_week_and_repository_info).and_return("all_csv_data")
+    expect(@generator.teams_csv_generator).to receive(:create_commit_table_with_week_and_repository_info).and_return("teams_csv_data")
+    expect(File).to receive(:write).with("csv_files/all_week_and_repository_info.csv", "all_csv_data")
+    expect(File).to receive(:write).with("csv_files/teams_week_and_repository_info.csv", "teams_csv_data")
+
+    @generator.create_commit_table_with_week_and_repository_info
+  end
+
+  it "can generate the week and team commit files" do
+    expect(@generator.teams_csv_generator).to receive(:create_commit_table_with_weeks_and_team_commits).and_return("csv_data")
+    expect(File).to receive(:write).with("csv_files/teams_week_and_team_commits.csv", "csv_data")
+
+    @generator.create_commit_table_with_weeks_and_team_commits
+  end
+
+  it "can generate the commit and repository info files" do
+    expect(@generator.teams_csv_generator).to receive(:create_commit_table_with_committers_and_repository_info).and_return("csv_data")
+    expect(File).to receive(:write).with("csv_files/teams_committer_and_repository_info.csv", "csv_data")
+
+    @generator.create_commit_table_with_committers_and_repository_info
+  end
+
+  it "can generate the weeks and committers info files" do
+    expect(@generator.all_csv_generator).to receive(:create_commit_table_with_weeks_and_committers).and_return("csv_data")
+    expect(File).to receive(:write).with("csv_files/teams_weeks_and_committers_info.csv", "csv_data")
+
+    @generator.create_commit_table_with_weeks_and_committers
+  end
+
+  it "can generate the weeks and committers per team info files" do
+    team = Team.new("teamx")
+    teams = Teams.new
+    teams.add(team)
+    @commit_history.configure(teams, @commit_history.repositories)
+    @generator =  MetricsGeneratorCsvFiles.new(CodespicuousConfig.new, @commit_history)
+
+    expect(@generator.all_csv_generator).to receive(:create_commit_table_with_weeks_and_committers).and_return("csv_data")
+    expect(@generator.all_csv_generator).to receive(:create_commit_table_with_weeks_and_committers).with("teamx").and_return("csv_data2")
+    expect(File).to receive(:write).with("csv_files/teams_weeks_and_committers_info.csv", "csv_data")
+    expect(File).to receive(:write).with("csv_files/teams_weeks_and_committers_info_teamx.csv", "csv_data2")
+
+    @generator.create_commit_table_with_weeks_and_committers
+  end
+
+end

data/spec/metrics_generator_daniel_spec.rb

@@ -3,7 +3,8 @@ describe "Daniel format metrics generator" do
 
   it "exists" do
     commit_history = CommitHistory.new
-    daniel = MetricsGeneratorDaniel.new(commit_history)
+    config = CodespicuousConfig.new
+    daniel = MetricsGeneratorDaniel.new(config, commit_history)
     daniel.generate
   end
 

data/spec/metrics_generator_spec.rb

@@ -3,21 +3,24 @@ describe "Generate different types of code metrics" do
 
   before (:each) do
     @commit_history = CommitHistory.new
+    @config = CodespicuousConfig.new
     @subject = MetricsGenerator.new
+
+    @subject.commit_history = @commit_history
+    @subject.config = @config
   end
 
   it "can generate different metrics" do
     expect(@subject).to receive(:generate_daniel)
-    expect(@subject).to receive(:generate_csv)
+    expect(@subject).to receive(:generate_csv_files)
 
-    @subject.generate(@commit_history)
+    @subject.generate(@config, @commit_history)
   end
 
   it "generate daniel format metrics" do
-    @subject.commit_history = @commit_history
 
-    daniel = MetricsGeneratorDaniel.new(@commit_history)
-    expect(MetricsGeneratorDaniel).to receive(:new).with(@commit_history).and_return(daniel)
+    daniel = MetricsGeneratorDaniel.new(@config, @commit_history)
+    expect(MetricsGeneratorDaniel).to receive(:new).with(@config, @commit_history).and_return(daniel)
     expect(daniel).to receive(:generate)
 
     @subject.generate_daniel
@@ -26,10 +29,10 @@ describe "Generate different types of code metrics" do
   it "generate csv format metrics" do
     @subject.commit_history = @commit_history
 
-    csv = MetricsGeneratorCsv.new(@commit_history)
-    expect(MetricsGeneratorCsv).to receive(:new).with(@commit_history).and_return(csv)
+    csv = MetricsGeneratorCsvFiles.new(@config, @commit_history)
+    expect(MetricsGeneratorCsvFiles).to receive(:new).with(@config, @commit_history).and_return(csv)
     expect(csv).to receive(:generate)
 
-    @subject.generate_csv
+    @subject.generate_csv_files
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codespicuous
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Bas Vodde
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-16 00:00:00.000000000 Z
+date: 2017-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -91,6 +91,7 @@ files:
 - lib/codespicuous/dateutil.rb
 - lib/codespicuous/metrics_generator.rb
 - lib/codespicuous/metrics_generator_csv.rb
+- lib/codespicuous/metrics_generator_csv_files.rb
 - lib/codespicuous/metrics_generator_daniel.rb
 - lib/codespicuous/participantsparser_from_csv.rb
 - lib/codespicuous/repositories.rb
@@ -108,6 +109,7 @@ files:
 - spec/committers_spec.rb
 - spec/danielparser_spec.rb
 - spec/integration_filezilla_spec.rb
+- spec/metrics_generator_csv_files_spec.rb
 - spec/metrics_generator_csv_spec.rb
 - spec/metrics_generator_daniel_spec.rb
 - spec/metrics_generator_spec.rb