diff test/unit/repository_bazaar_test.rb @ 1115:433d4f72a19b redmine-2.2

Update to Redmine SVN revision 11137 on 2.2-stable branch

--- a/test/unit/repository_bazaar_test.rb	Wed Jun 27 14:54:18 2012 +0100
+++ b/test/unit/repository_bazaar_test.rb	Mon Jan 07 12:01:42 2013 +0000
@@ -1,5 +1,5 @@
 # Redmine - project management software
-# Copyright (C) 2006-2011  Jean-Philippe Lang
+# Copyright (C) 2006-2012  Jean-Philippe Lang
 #
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
@@ -20,26 +20,82 @@
 class RepositoryBazaarTest < ActiveSupport::TestCase
   fixtures :projects
 
-  REPOSITORY_PATH = Rails.root.join('tmp/test/bazaar_repository/trunk').to_s
-  REPOSITORY_PATH.gsub!(/\/+/, '/')
+  include Redmine::I18n
+
+  REPOSITORY_PATH = Rails.root.join('tmp/test/bazaar_repository').to_s
+  REPOSITORY_PATH_TRUNK = File.join(REPOSITORY_PATH, "trunk")
   NUM_REV = 4
 
+  REPOSITORY_PATH_NON_ASCII = Rails.root.join(REPOSITORY_PATH + '/' + 'non_ascii').to_s
+
+  # Bazaar core does not support xml output such as Subversion and Mercurial.
+  # "bzr" command output and command line parameter depend on locale.
+  # So, non ASCII path tests cannot run independent locale.
+  #
+  # If you want to run Bazaar non ASCII path tests on Linux *Ruby 1.9*,
+  # you need to set locale character set "ISO-8859-1".
+  # E.g. "LANG=en_US.ISO-8859-1".
+  # On Linux other platforms (e.g. Ruby 1.8, JRuby),
+  # you need to set "RUN_LATIN1_OUTPUT_TEST = true" manually.
+  #
+  # On Windows, because it is too hard to change system locale,
+  # you cannot run Bazaar non ASCII path tests.
+  #
+  RUN_LATIN1_OUTPUT_TEST = (RUBY_PLATFORM != 'java' &&
+                             REPOSITORY_PATH.respond_to?(:force_encoding) &&
+                             Encoding.locale_charmap == "ISO-8859-1")
+
+  CHAR_1_UTF8_HEX   = "\xc3\x9c"
+  CHAR_1_LATIN1_HEX = "\xdc"
+
   def setup
     @project = Project.find(3)
     @repository = Repository::Bazaar.create(
-              :project => @project, :url => "file:///#{REPOSITORY_PATH}",
+              :project => @project, :url => REPOSITORY_PATH_TRUNK,
               :log_encoding => 'UTF-8')
     assert @repository
+    @char_1_utf8      = CHAR_1_UTF8_HEX.dup
+    @char_1_ascii8bit = CHAR_1_LATIN1_HEX.dup
+    if @char_1_utf8.respond_to?(:force_encoding)
+      @char_1_utf8.force_encoding('UTF-8')
+      @char_1_ascii8bit.force_encoding('ASCII-8BIT')
+    end
   end
 
-  if File.directory?(REPOSITORY_PATH)
+  def test_blank_path_to_repository_error_message
+    set_language_if_valid 'en'
+    repo = Repository::Bazaar.new(
+                          :project      => @project,
+                          :identifier   => 'test',
+                          :log_encoding => 'UTF-8'
+                        )
+    assert !repo.save
+    assert_include "Path to repository can't be blank",
+                   repo.errors.full_messages
+  end
+
+  def test_blank_path_to_repository_error_message_fr
+    set_language_if_valid 'fr'
+    str = "Chemin du d\xc3\xa9p\xc3\xb4t doit \xc3\xaatre renseign\xc3\xa9(e)"
+    str.force_encoding('UTF-8') if str.respond_to?(:force_encoding)
+    repo = Repository::Bazaar.new(
+                          :project      => @project,
+                          :url          => "",
+                          :identifier   => 'test',
+                          :log_encoding => 'UTF-8'
+                        )
+    assert !repo.save
+    assert_include str, repo.errors.full_messages
+  end
+
+  if File.directory?(REPOSITORY_PATH_TRUNK)
     def test_fetch_changesets_from_scratch
       assert_equal 0, @repository.changesets.count
       @repository.fetch_changesets
       @project.reload
 
       assert_equal NUM_REV, @repository.changesets.count
-      assert_equal 9, @repository.changes.count
+      assert_equal 9, @repository.filechanges.count
       assert_equal 'Initial import', @repository.changesets.find_by_revision('1').comments
     end
 
@@ -60,13 +116,16 @@
 
     def test_entries
       entries = @repository.entries
+      assert_kind_of Redmine::Scm::Adapters::Entries, entries
       assert_equal 2, entries.size
 
       assert_equal 'dir', entries[0].kind
       assert_equal 'directory', entries[0].name
+      assert_equal 'directory', entries[0].path
 
       assert_equal 'file', entries[1].kind
       assert_equal 'doc-mkdir.txt', entries[1].name
+      assert_equal 'doc-mkdir.txt', entries[1].path
     end
 
     def test_entries_in_subdirectory
@@ -75,6 +134,7 @@
 
       assert_equal 'file', entries.last.kind
       assert_equal 'edit.png', entries.last.name
+      assert_equal 'directory/edit.png', entries.last.path
     end
 
     def test_previous
@@ -112,6 +172,133 @@
       changeset = @repository.find_changeset_by_name('4')
       assert_nil changeset.next
     end
+
+    if File.directory?(REPOSITORY_PATH_NON_ASCII) && RUN_LATIN1_OUTPUT_TEST
+      def test_cat_latin1_path
+        latin1_repo = create_latin1_repo
+        buf = latin1_repo.cat(
+                 "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-2.txt", 2)
+        assert buf
+        lines = buf.split("\n")
+        assert_equal 2, lines.length
+        assert_equal 'It is written in Python.', lines[1]
+
+        buf = latin1_repo.cat(
+                 "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt", 2)
+        assert buf
+        lines = buf.split("\n")
+        assert_equal 1, lines.length
+        assert_equal "test-#{@char_1_ascii8bit}.txt", lines[0]
+      end
+
+      def test_annotate_latin1_path
+        latin1_repo = create_latin1_repo
+        ann1 = latin1_repo.annotate(
+                   "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-2.txt", 2)
+        assert_equal 2, ann1.lines.size
+        assert_equal '2', ann1.revisions[0].identifier
+        assert_equal 'test00@', ann1.revisions[0].author
+        assert_equal 'It is written in Python.', ann1.lines[1]
+        ann2 = latin1_repo.annotate(
+                   "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt", 2)
+        assert_equal 1, ann2.lines.size
+        assert_equal '2', ann2.revisions[0].identifier
+        assert_equal 'test00@', ann2.revisions[0].author
+        assert_equal "test-#{@char_1_ascii8bit}.txt", ann2.lines[0]
+      end
+
+      def test_diff_latin1_path
+        latin1_repo = create_latin1_repo
+        diff1 = latin1_repo.diff(
+                  "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt", 2, 1)
+        assert_equal 7, diff1.size
+        buf =  diff1[5].gsub(/\r\n|\r|\n/, "")
+        assert_equal "+test-#{@char_1_ascii8bit}.txt", buf
+      end
+
+      def test_entries_latin1_path
+        latin1_repo = create_latin1_repo
+        entries = latin1_repo.entries("test-#{@char_1_utf8}-dir", 2)
+        assert_kind_of Redmine::Scm::Adapters::Entries, entries
+        assert_equal 3, entries.size
+        assert_equal 'file', entries[1].kind
+        assert_equal "test-#{@char_1_utf8}-1.txt", entries[0].name
+        assert_equal "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt", entries[0].path
+      end
+
+      def test_entry_latin1_path
+        latin1_repo = create_latin1_repo
+        ["test-#{@char_1_utf8}-dir",
+          "/test-#{@char_1_utf8}-dir",
+          "/test-#{@char_1_utf8}-dir/"
+        ].each do |path|
+          entry = latin1_repo.entry(path, 2)
+          assert_equal "test-#{@char_1_utf8}-dir", entry.path
+          assert_equal "dir", entry.kind
+        end
+        ["test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt",
+          "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt"
+        ].each do |path|
+          entry = latin1_repo.entry(path, 2)
+          assert_equal "test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt",
+                       entry.path
+          assert_equal "file", entry.kind
+        end
+      end
+
+      def test_changeset_latin1_path
+        latin1_repo = create_latin1_repo
+        assert_equal 0, latin1_repo.changesets.count
+        latin1_repo.fetch_changesets
+        @project.reload
+        assert_equal 3, latin1_repo.changesets.count
+
+        cs2 = latin1_repo.changesets.find_by_revision('2')
+        assert_not_nil cs2
+        assert_equal "test-#{@char_1_utf8}", cs2.comments
+        c2  = cs2.filechanges.sort_by(&:path)
+        assert_equal 4, c2.size
+        assert_equal 'A', c2[0].action
+        assert_equal "/test-#{@char_1_utf8}-dir/", c2[0].path
+        assert_equal 'A', c2[1].action
+        assert_equal "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-1.txt", c2[1].path
+        assert_equal 'A', c2[2].action
+        assert_equal "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-2.txt", c2[2].path
+        assert_equal 'A', c2[3].action
+        assert_equal "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}.txt", c2[3].path
+
+        cs3 = latin1_repo.changesets.find_by_revision('3')
+        assert_not_nil cs3
+        assert_equal "modify, move and delete #{@char_1_utf8} files", cs3.comments
+        c3  = cs3.filechanges.sort_by(&:path)
+        assert_equal 3, c3.size
+        assert_equal 'M', c3[0].action
+        assert_equal "/test-#{@char_1_utf8}-1.txt", c3[0].path
+        assert_equal 'D', c3[1].action
+        assert_equal "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}-2.txt", c3[1].path
+        assert_equal 'M', c3[2].action
+        assert_equal "/test-#{@char_1_utf8}-dir/test-#{@char_1_utf8}.txt", c3[2].path
+      end
+    else
+      msg = "Bazaar non ASCII output test cannot run this environment." + "\n"
+      if msg.respond_to?(:force_encoding)
+        msg += "Encoding.locale_charmap: " + Encoding.locale_charmap + "\n"
+      end
+      puts msg
+    end
+
+    private
+
+    def create_latin1_repo
+      repo = Repository::Bazaar.create(
+                            :project      => @project,
+                            :identifier   => 'latin1',
+                            :url => REPOSITORY_PATH_NON_ASCII,
+                            :log_encoding => 'ISO-8859-1'
+                        )
+      assert repo
+      repo
+    end
   else
     puts "Bazaar test repository NOT FOUND. Skipping unit tests !!!"
     def test_fake; assert true end