Redmine

Hi!

Thanks for your patch! But I think there still an issue... I test Redmine Git Hosting plugin with Travis (as you do) and one the test is still failing : https://travis-ci.org/jbox-web/redmine_git_hosting.

As you can see I test Redmine Git Hosting with Redmine 2.6.2/3.0.0 (from tarball) and 2.6-stable/3.0-stable (from SVN) and both stable branch are broken. (from plugin point of view).

It's about identifier uniqueness.

Here the test :

before do
  @repository_1 = create_git_repository(project: @project_child, is_default: true) # Create a repo in database with no identifier
end

describe "Test uniqueness" do
  context "when blank identifier is already taken by a repository" do
    it { expect(build_git_repository(project: @project_child, identifier: '')).to be_invalid } # This one fails
  end

  context "when set as default and blank identifier is already taken by a repository" do
    it { expect(build_git_repository(project: @project_child, identifier: '', is_default: true)).to be_invalid } # This one pass
  end

....

It seems that depending on the "default status" of a repository you can still create 2 repositories with blank identifier.