JENKINS-15128: even if updatedFiles are empty, we should consider current commit as checked in in order to remove it from commitsQueue

Author: fcamblor

src/main/java/hudson/plugins/scm_sync_configuration/SCMManipulator.java

@@ -9,6 +9,7 @@
 import org.apache.maven.scm.command.checkin.CheckInScmResult;
 import org.apache.maven.scm.command.checkout.CheckOutScmResult;
 import org.apache.maven.scm.command.remove.RemoveScmResult;
+import org.apache.maven.scm.command.update.UpdateScmResult;
 import org.apache.maven.scm.manager.NoSuchScmProviderException;
 import org.apache.maven.scm.manager.ScmManager;
 import org.apache.maven.scm.repository.ScmRepository;
@@ -74,8 +75,8 @@ private boolean expectScmRepositoryInitiated(){
 		return scmRepositoryInitiated;
 	}
 
-	public void update(File root) throws ScmException {
-		this.scmManager.update(scmRepository, new ScmFileSet(root));
+	public UpdateScmResult update(File root) throws ScmException {
+		return this.scmManager.update(scmRepository, new ScmFileSet(root));
 	}
 	public boolean checkout(File checkoutDirectory){
 		boolean checkoutOk = false;

src/main/java/hudson/plugins/scm_sync_configuration/ScmSyncConfigurationBusiness.java

@@ -202,6 +202,7 @@ private void processCommitsQueue() {
 
                 if(updatedFiles.isEmpty()){
                     LOGGER.finest("Empty changeset to commit (no changes found on files) => commit skipped !");
+                    checkedInCommits.add(commit);
                 } else {
                     // Commiting files...
                     boolean result = scmManipulator.checkinFiles(scmRoot, commit.getMessage());