Proof of concept #1026

Paul Martin

2016-04-16 eecaad8b8e2c447429c31a01d49260ddd6b4ee03

 src/test/java/com/gitblit/tests/GitServletTest.java

@@ -177,7 +177,7 @@
      model.authorizationControl = AuthorizationControl.NAMED;

      UserModel user = new UserModel("james");

      user.password = "james";

      gitblit().updateUserModel(user.username, user, true);

      gitblit().addUser(user);

      repositories().updateRepositoryModel(model.name, model, false);



      FileUtils.delete(ticgit2Folder, FileUtils.RECURSIVE);

@@ -380,27 +380,15 @@
   public void testCommitterVerification() throws Exception {

      UserModel user = getUser();



      // account only uses account name to verify

      testCommitterVerification(user, user.username, null, true);

      // committer email address is ignored because account does not specify email

      testCommitterVerification(user, user.username, "something", true);

      // completely different committer

      testCommitterVerification(user, "joe", null, false);

      testCommitterVerification(user, "joe", user.emailAddress, false);

      testCommitterVerification(user, user.username, null, false);

      testCommitterVerification(user, user.username, user.emailAddress, true);



      // test display name verification

      user.displayName = "James Moger";

      testCommitterVerification(user, user.displayName, null, true);

      testCommitterVerification(user, user.displayName, "something", true);

      testCommitterVerification(user, "joe", null, false);



      // test email address verification

      user.emailAddress = "something";

      testCommitterVerification(user, user.displayName, null, false);

      testCommitterVerification(user, user.displayName, "somethingelse", false);

      testCommitterVerification(user, user.displayName, "something", false);

      testCommitterVerification(user, user.displayName, user.emailAddress, true);



      // use same email address but with different committer

      testCommitterVerification(user, "joe", "somethingelse", false);

   }



   private void testCommitterVerification(UserModel user, String displayName, String emailAddress, boolean expectedSuccess) throws Exception {

@@ -431,7 +419,7 @@
      // grant user push permission

      user.setRepositoryPermission(model.name, AccessPermission.PUSH);



      gitblit().updateUserModel(user.username, user, true);

      gitblit().addUser(user);

      repositories().updateRepositoryModel(model.name, model, false);



      // clone temp bare repo to working copy

@@ -516,7 +504,7 @@
      // grant user push permission

      user.setRepositoryPermission(model.name, AccessPermission.PUSH);



      gitblit().updateUserModel(user.username, user, true);

      gitblit().addUser(user);

      repositories().updateRepositoryModel(model.name, model, false);



      // clone temp bare repo to working copy

@@ -669,7 +657,7 @@
      // grant user specified

      user.setRepositoryPermission(model.name, permission);



      gitblit().updateUserModel(user.username, user, true);

      gitblit().addUser(user);

      repositories().updateRepositoryModel(model.name, model, false);



      // clone temp bare repo to working copy

@@ -839,7 +827,7 @@
      user.canCreate = canCreate;

      user.canAdmin = canAdmin;



      gitblit().updateUserModel(user.username, user, true);

      gitblit().addUser(user);



      CredentialsProvider cp = new UsernamePasswordCredentialsProvider(user.username, user.password);