Raise error when attempting to make changes to a file modified in the worktree

Gemfile

@@ -1,6 +1,7 @@
 source 'https://rubygems.org'
 gemspec :name => 'gollum-lib'
 gem 'irb'
+gem 'gollum-rugged_adapter', git: 'https://github.com/dometto/rugged_adapter', branch: 'clean_workdir'
 
 if RUBY_PLATFORM == 'java' then
   group :development do

lib/gollum-lib.rb

@@ -54,5 +54,6 @@ def initialize(attempted, message = nil)
   class InvalidGitRepositoryError < StandardError; end
   class NoSuchPathError < StandardError; end
   class IllegalDirectoryPath < StandardError; end
+  class WorkdirModifiedError < StandardError; end
 
 end

lib/gollum-lib/committer.rb

@@ -79,12 +79,15 @@ def parents
     # Raises Gollum::DuplicatePageError if a matching filename already exists, unless force_overwrite is explicitly enabled.
     # This way, pages are not inadvertently overwritten.
     #
+    # Even if force_overwrite is enabled, this throws a WorkdirModifiedError if the workdir path contains modifications that are not in the repository.
+    #
     # Returns nothing (modifies the Index in place).
     def add_to_index(path, data, options = {}, force_overwrite = false)
       if tree = index.current_tree
         unless page_path_scheduled_for_deletion?(index.tree, path) || force_overwrite
           raise DuplicatePageError.new(path) if tree / path
         end
+        raise WorkdirModifiedError.new(path) if index.workdir_path_modified?(path)
       end
 
       unless options[:normalize] == false

lib/gollum-lib/wiki.rb

@@ -355,7 +355,7 @@ def update_page(page, name, format, data, commit = {})
       committer    = multi_commit ? commit[:committer] : Committer.new(self, commit)
 
       if !rename
-        committer.add(page.path, normalize(data))
+        committer.add_to_index(page.path, data, {normalize: true}, true)
       else
         committer.delete(page.path)
         committer.add_to_index(new_path, data)

test/test_committer.rb

@@ -148,6 +148,20 @@
     assert_equal @wiki.repo.head.commit.note, 'My notes'
   end
 
+  test "raise WorkdirModifiedError when the workdir contains changes" do
+    committer = Gollum::Committer.new(@wiki)
+    path = 'Bilbo-Baggins.md'
+    filepath = File.join(@wiki.path, path)
+
+    f = File.open(filepath, 'w') { |f| f.puts "Workdir no longer clean" }
+    assert_raises Gollum::WorkdirModifiedError do
+      committer.add_to_index(path, 'content', {}, true)
+    end
+
+    File.delete(filepath)
+    assert_equal 'content', committer.add_to_index(path, 'content', {}, true)
+  end
+
   teardown do
     FileUtils.rm_rf(@path)
   end