Commits

Toshi MARUYAMA  committed 2ab8974

scm: bazaar: add non ASCII path test to unit model test (#11834)

  • Participants
  • Parent commits f0f8135

Comments (0)

Files changed (1)

File test/unit/repository_bazaar_test.rb

   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, tests cannot run independent locale.
+  #
+  # If you want to run Bazaar non ASCII path tests on *Ruby 1.9*,
+  # you need to set locale character set "ISO-8859-1".
+  # E.g. "LANG=en_US.ISO-8859-1".
+  # On 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 => 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
 
   def test_blank_path_to_repository_error_message
       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