Merge branch 'master' of github.com:github/gollum

.gitignore

@@ -2,3 +2,4 @@ coverage
 pkg
 .DS_Store
 .bundle
+Gemfile.lock

Gemfile.lock

@@ -1,58 +0,0 @@
-PATH
-  remote: .
-  specs:
-    gollum (1.1.0)
-      albino (~> 1.0)
-      github-markup (>= 0.4.0, < 1.0.0)
-      grit (~> 2.3)
-      mustache (>= 0.11.2, < 1.0.0)
-      nokogiri (~> 1.4)
-      sanitize (~> 1.1)
-      sinatra (~> 1.0)
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    RedCloth (4.2.3)
-    albino (1.0)
-    diff-lcs (1.1.2)
-    github-markup (0.5.0)
-    grit (2.3.0)
-      diff-lcs (~> 1.1)
-      mime-types (~> 1.15)
-    mime-types (1.16)
-    mocha (0.9.9)
-      rake
-    mustache (0.11.2)
-    nokogiri (1.4.3.1)
-    org-ruby (0.5.3)
-      rubypants (>= 0.2.0)
-    rack (1.2.1)
-    rake (0.8.7)
-    rdiscount (1.6.5)
-    rubypants (0.2.0)
-    sanitize (1.2.1)
-      nokogiri (~> 1.4.1)
-    shoulda (2.11.3)
-    sinatra (1.1.0)
-      rack (~> 1.1)
-      tilt (~> 1.1)
-    tilt (1.1)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  RedCloth
-  albino (~> 1.0)
-  github-markup (>= 0.4.0, < 1.0.0)
-  gollum!
-  grit (~> 2.3)
-  mocha
-  mustache (>= 0.11.2, < 1.0.0)
-  nokogiri (~> 1.4)
-  org-ruby
-  rdiscount
-  sanitize (~> 1.1)
-  shoulda
-  sinatra (~> 1.0)

lib/gollum/blob_entry.rb

@@ -60,12 +60,14 @@ module Gollum
     #   normalize_dir("foo")   # => "/foo"
     #   normalize_dir("/foo/") # => "/foo"
     #   normalize_dir("/")     # => ""
+    #   normalize_dir("c:/")   # => ""
     #
     # dir - String directory name.
     #
     # Returns a normalized String directory name, or nil if no directory
     # is given.
     def self.normalize_dir(dir)
+      return '' if dir =~ /^.:\/$/
       if dir
         dir = ::File.expand_path(dir, '/')
         dir = '' if dir == '/'

lib/gollum/page.rb

@@ -4,15 +4,16 @@ module Gollum
 
     Wiki.page_class = self
 
-    VALID_PAGE_RE = /^(.+)\.(md|mkdn?|mdown|markdown|textile|rdoc|org|creole|re?st(\.txt)?|asciidoc|pod)$/i
+    VALID_PAGE_RE = /^(.+)\.(md|mkdn?|mdown|markdown|textile|rdoc|org|creole|re?st(\.txt)?|asciidoc|pod|(media)?wiki)$/i
-    FORMAT_NAMES = { :markdown => "Markdown",
+    FORMAT_NAMES = { :markdown  => "Markdown",
-                     :textile  => "Textile",
+                     :textile   => "Textile",
-                     :rdoc     => "RDoc",
+                     :rdoc      => "RDoc",
-                     :org      => "Org-mode",
+                     :org       => "Org-mode",
-                     :creole   => "Creole",
+                     :creole    => "Creole",
-                     :rest     => "reStructuredText",
+                     :rest      => "reStructuredText",
-                     :asciidoc => "AsciiDoc",
+                     :asciidoc  => "AsciiDoc",
-                     :pod      => "Pod" }
+                     :mediawiki => "MediaWiki",
+                     :pod       => "Pod" }
 
     # Sets a Boolean determing whether this page is a historical version.
     #
@@ -161,6 +162,8 @@ module Gollum
           :pod
         when /\.(\d)$/i
           :roff
+        when /\.(media)?wiki$/i
+          :mediawiki
         else
           nil
       end
@@ -244,14 +247,15 @@ module Gollum
     # Returns the String extension (no leading period).
     def self.format_to_ext(format)
       case format
-        when :markdown then 'md'
+        when :markdown  then 'md'
-        when :textile  then 'textile'
+        when :textile   then 'textile'
-        when :rdoc     then 'rdoc'
+        when :rdoc      then 'rdoc'
-        when :org      then 'org'
+        when :org       then 'org'
-        when :creole   then 'creole'
+        when :creole    then 'creole'
-        when :rest     then 'rest'
+        when :rest      then 'rest'
-        when :asciidoc then 'asciidoc'
+        when :asciidoc  then 'asciidoc'
-        when :pod      then 'pod'
+        when :pod       then 'pod'
+        when :mediawiki then 'mediawiki'
       end
     end
 

test/helper.rb

@@ -22,11 +22,12 @@ def testpath(path)
 end
 
 def cloned_testpath(path)
-  path   = testpath(path)
+  repo   = File.expand_path(testpath(path))
-  cloned = path.chomp('.git')
+  path   = File.dirname(repo)
-  FileUtils.rm_rf cloned
+  cloned = File.join(path, self.class.name)
-  Dir.chdir(File.expand_path(File.dirname(path))) do
+  FileUtils.rm_rf(cloned)
-    %x{git clone #{File.basename(path)}}
+  Dir.chdir(path) do
+    %x{git clone #{File.basename(repo)} #{self.class.name}}
   end
   cloned
 end

test/test_page.rb

@@ -134,4 +134,13 @@ context "Page" do
     assert_equal '', Gollum::Page.cname(nil)
     assert_equal '', Gollum::Page.cname(3)
   end
+
+  test "normalize_dir" do
+    assert_equal "", Gollum::BlobEntry.normalize_dir("")
+    assert_equal "", Gollum::BlobEntry.normalize_dir(".")
+    assert_equal "", Gollum::BlobEntry.normalize_dir("/")
+    assert_equal "", Gollum::BlobEntry.normalize_dir("c:/")
+    assert_equal "/foo", Gollum::BlobEntry.normalize_dir("foo")
+    assert_equal "/foo", Gollum::BlobEntry.normalize_dir("/foo")
+  end
 end

test/test_page_revert.rb

@@ -8,7 +8,7 @@ context "Page Reverting" do
   end
 
   teardown do
-    FileUtils.rm_rf @path
+    FileUtils.rm_rf(@path)
   end
 
   test "reverts single commit" do