require 'set'

module Overcommit::HookContext
  # Simulates a pre-commit context pretending that all files have been changed.
  #
  # This results in pre-commit hooks running against the entire repository,
  # which is useful for automated CI scripts.
  class RunAll < Base
    EMPTY_SET = Set.new

    def modified_files
      all_files
    end

    # Return an empty set since in this context the user didn't actually touch
    # any lines.
    def modified_lines_in_file(_file)
      EMPTY_SET
    end

    def hook_class_name
      'PreCommit'
    end

    def hook_type_name
      'pre_commit'
    end

    def hook_script_name
      'pre-commit'
    end

    private

    def all_files
      @all_files ||= Overcommit::GitRepo.all_files
    end
  end
end

module Overcommit::HookContext
  # Simulates a pre-commit context pretending that all files have been changed.
  #
  # This results in pre-commit hooks running against the entire repository,
  # which is useful for automated CI scripts.
  class RunAll < Base
    def modified_files
      all_files
    end

    def hook_class_name
      'PreCommit'
    end

    def hook_type_name
      'pre_commit'
    end

    def hook_script_name
      'pre-commit'
    end

    private

    def all_files
      @all_files ||= Overcommit::GitRepo.all_files
    end
  end
end

--- 
+++ 
@@ -1,11 +1,21 @@
+require 'set'
+
 module Overcommit::HookContext
   # Simulates a pre-commit context pretending that all files have been changed.
   #
   # This results in pre-commit hooks running against the entire repository,
   # which is useful for automated CI scripts.
   class RunAll < Base
+    EMPTY_SET = Set.new
+
     def modified_files
       all_files
+    end
+
+    # Return an empty set since in this context the user didn't actually touch
+    # any lines.
+    def modified_lines_in_file(_file)
+      EMPTY_SET
     end
 
     def hook_class_name
Fix RunAll context to define modified_lines_in_file

Change-Id: Ic3553043bff52a5a2893b41813f866e88ca75752
Reviewed-on: http://gerrit.causes.com/44906
Tested-by: jenkins <d95b56ce41a2e1ac4cecdd398defd7414407cc08@brigade.com>
Reviewed-by: Shane da Silva <67fb7bfe39f841b16a1f86d3928acc568d12c716@brigade.com>
