diff_parser 1.0.0 → 1.1.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZjkxM2Q3NjAzODMyYmEwYzI2MGVhZTBjZmY1MTc1YTViN2E0ZGE0Yw==
+    NDM3MjZhYmUyMmZkZTQxNDc0N2U4NWNkNGIzM2NjNWQwYTg4MTVlMg==
   data.tar.gz: !binary |-
-    N2VhNzliYzNkYmFkZThlNGM4ZDQ2OWE1ODdlNjZmZTliNGIxMDI5NA==
+    YzVlNDFlNzEzNDM5MDE5Yjk0OTcyMDliNTYwZWVhM2M1ZTEyYzVhNA==
 SHA512:
   metadata.gz: !binary |-
-    NWFkMjQzNTU1NTJkMTEwYjE2MjU4OGZlNzk2ZWNiMjk0MjdkODVlMzBkMTFh
-    YzE2YmFlNGU3ZDY3MWMxMDZiNzRlZDk0Y2I0YTdkNTlhN2E0NDY4Y2I3ODNi
-    MjhiNmU3NzAzNTRjZjUwNmU0ZmZjZmMwNWMyYmIwODkzMmE2ZTU=
+    YTdmZDQ5ODRmYmMzYzBiOWYwMmVjNzEwMjQzZGY4MjBkNWVlYzk2Y2RlYzU0
+    MDNlYzAwZmIzNjVhYTc5YTQ0NmFlNjBiOGUzMmZlNGVhODIwYjg1YTdiMjAw
+    NDI1YjM4Yzg2YzkwYzAyNjE1YWQzYmM2YWQ0MDY2NzlhOGYwMjY=
   data.tar.gz: !binary |-
-    MmIxMTdmNDRiMmZkMzgwNTQ2NTA0NTllZDRlMmM0NDc3MzMzYzRkYzNlZDQ4
-    ODM5M2Y5N2NmYjNiYjQzYjI5YTc0NjFhNTU4MzM3NTdjNzhkYzU0YmJhYmM3
-    NzhhOTI5ZWYwMzVhMjVhMTAwOWUzYTJmODIzMWMyYTkxZTllNjY=
+    MTRhOGVjYjYxMjkxZTkwNWJmOTVkYzU5MTNlZGVmMDEyNzdkNGU4NDNkYmY3
+    MDdjMzVhN2UxZDdmY2IyOGI1YzYyMjhhNTlmMWI4YmZlZWU1YzUxY2IzOGY3
+    NGFiNjI5YzdkMzAwMjFjMDgzOGNjNDc1OGZiNzkxMDk4NGJiMGU=

data/lib/diff_parser.rb

@@ -5,3 +5,18 @@ require 'diff_parser/diff_files_collection'
 require 'diff_parser/raw_line'
 require 'diff_parser/diff_line'
 require 'diff_parser/inline_diff'
+
+module DiffParser
+  mattr_accessor :max_changes_to_render
+  @@max_changes_to_render = 500
+
+  mattr_accessor :reduced_max_changes_to_render
+  @@reduced_max_changes_to_render = 10
+
+  mattr_accessor :changes_reduction_threshold
+  @@changes_reduction_threshold = 20
+
+  def self.setup
+    yield self
+  end
+end

data/lib/diff_parser/commit.rb

@@ -1,3 +1,6 @@
+# http://developer.github.com/v3/repos/commits/#get-a-single-commit
+# http://developer.github.com/v3/media/#commits-commit-comparison-and-pull-requests
+
 module DiffParser
   class Commit
     attr_reader :github, :repo, :sha
@@ -9,17 +12,7 @@ module DiffParser
     end
 
     def diff
-      DiffParser::Diff.new(raw, info)
-    end
-
-    private
-
-    def info
-      @info ||= github.commit(repo, sha)
-    end
-
-    def raw
-      @raw ||= github.commit(repo, sha, accept: 'application/vnd.github.VERSION.diff')
+      DiffParser::Diff.new(self)
     end
   end
 end

data/lib/diff_parser/diff.rb

@@ -1,14 +1,16 @@
 module DiffParser
   class Diff
-    attr_reader :raw_diff, :diff_info
+    attr_reader :raw, :info, :commit
+    delegate :github, :repo, :sha, to: :commit
 
-    def initialize(raw_diff, diff_info)
-      @raw_diff = raw_diff
-      @diff_info = diff_info
+    def initialize(commit)
+      @commit = commit
+      @info = github.commit(commit.repo, sha)
+      @raw = github.commit(repo, sha, accept: 'application/vnd.github.VERSION.diff')
     end
 
     def files
-      DiffParser::DiffFilesCollection.new(raw_diff, diff_info)
+      DiffParser::DiffFilesCollection.new(self)
     end
   end
 end

data/lib/diff_parser/diff_files_collection.rb

@@ -2,15 +2,14 @@ module DiffParser
   class DiffFilesCollection
     include Enumerable
 
-    attr_reader :raw_diff, :diff_info
+    attr_reader :diff
 
-    def initialize(raw_diff, diff_info)
-      @raw_diff = raw_diff
-      @diff_info = diff_info
+    def initialize(diff)
+      @diff = diff
     end
 
     def raw_lines
-      preprocessed_lines = DiffParser::InlineDiff.processing(raw_diff.lines)
+      preprocessed_lines = DiffParser::InlineDiff.processing(diff.raw.lines)
       preprocessed_lines.map { |line| RawLine.new(line) }
     end
 
@@ -22,8 +21,8 @@ module DiffParser
 
         if raw_line.info?
           yield current_file if current_file
-          current_file = DiffParser::File.new(raw_line.full_line, diff_info)
-        else
+          current_file = DiffParser::File.new(raw_line.full_line, diff)
+        elsif !current_file.folded?
           current_file.push_line(raw_line, idx)
         end
       end

data/lib/diff_parser/file.rb

@@ -1,16 +1,16 @@
 module DiffParser
   class File
     attr_accessor :line_old, :line_new
-    attr_reader :lines, :sha, :path, :file_info
+    attr_reader :lines, :path, :file_info, :diff
     delegate :additions, :deletions, to: :file_info
 
-    def initialize(path, diff_info)
+    def initialize(path, diff)
       @path = preprocess_path(path)
+      @diff = diff
       @lines = []
       @line_old = 1
       @line_new = 1
-      @sha = diff_info.sha
-      @file_info = retrieve_file_info(diff_info)
+      @file_info = retrieve_file_info(diff)
     end
 
     def push_line(raw_line, idx)
@@ -41,14 +41,47 @@ module DiffParser
       end
     end
 
+    def sha
+      diff.sha
+    end
+
+    def folded?
+      @folded ||= if file_max_changes_exceeded?
+                  lines << DiffLine.new('Changes not shown', 'info', nil, nil, 0, nil)
+                  true
+                else
+                  false
+                end
+    end
+
     private
 
+    def max_changes_to_render
+      if diff.info.files.count > DiffParser.changes_reduction_threshold
+        DiffParser.reduced_max_changes_to_render
+      else
+        DiffParser.max_changes_to_render
+      end
+    end
+
+    def file_max_changes_exceeded?
+      (file_info.additions > max_changes_to_render && added?) || (file_info.deletions > max_changes_to_render && removed?)
+    end
+
+    def added?
+      file_info.status == 'added'
+    end
+
+    def removed?
+      file_info.status == 'removed'
+    end
+
     def preprocess_path(path)
       path.split(' ').last[2..-1]
     end
 
-    def retrieve_file_info(diff_info)
-      diff_info.files.find { |f| f.filename == path }
+    def retrieve_file_info(diff)
+      diff.info.files.find { |f| f.filename == path }
     end
   end
 end

data/lib/diff_parser/version.rb

@@ -1,3 +1,3 @@
 module DiffParser
-  VERSION = "1.0.0"
+  VERSION = "1.1.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: diff_parser
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - stevo