| // Copyright (C) 2013 The Android Open Source Project |
| // |
| // Licensed under the Apache License, Version 2.0 (the "License"); |
| // you may not use this file except in compliance with the License. |
| // You may obtain a copy of the License at |
| // |
| // http://www.apache.org/licenses/LICENSE-2.0 |
| // |
| // Unless required by applicable law or agreed to in writing, software |
| // distributed under the License is distributed on an "AS IS" BASIS, |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| // See the License for the specific language governing permissions and |
| // limitations under the License. |
| |
| package com.google.gerrit.acceptance.server.change; |
| |
| import static com.google.gerrit.acceptance.git.GitUtil.add; |
| import static com.google.gerrit.acceptance.git.GitUtil.cloneProject; |
| import static com.google.gerrit.acceptance.git.GitUtil.createCommit; |
| import static com.google.gerrit.acceptance.git.GitUtil.createProject; |
| import static com.google.gerrit.acceptance.git.GitUtil.initSsh; |
| import static com.google.gerrit.acceptance.git.GitUtil.pushHead; |
| import static org.junit.Assert.assertEquals; |
| |
| import com.google.common.collect.ImmutableList; |
| import com.google.common.collect.Iterables; |
| import com.google.gerrit.acceptance.AbstractDaemonTest; |
| import com.google.gerrit.acceptance.AccountCreator; |
| import com.google.gerrit.acceptance.RestSession; |
| import com.google.gerrit.acceptance.SshSession; |
| import com.google.gerrit.acceptance.TestAccount; |
| import com.google.gerrit.acceptance.git.PushOneCommit; |
| import com.google.gerrit.acceptance.git.GitUtil.Commit; |
| import com.google.gerrit.reviewdb.client.Change; |
| import com.google.gerrit.reviewdb.client.PatchSet; |
| import com.google.gerrit.reviewdb.client.Project; |
| import com.google.gerrit.reviewdb.server.ReviewDb; |
| import com.google.gerrit.server.OutputFormat; |
| import com.google.gson.reflect.TypeToken; |
| import com.google.gwtorm.server.OrmException; |
| import com.google.gwtorm.server.SchemaFactory; |
| import com.google.inject.Inject; |
| |
| import org.eclipse.jgit.api.Git; |
| import org.eclipse.jgit.api.ResetCommand.ResetType; |
| import org.eclipse.jgit.api.errors.GitAPIException; |
| import org.junit.After; |
| import org.junit.Before; |
| import org.junit.Test; |
| |
| import java.io.IOException; |
| import java.util.List; |
| |
| public class GetRelatedIT extends AbstractDaemonTest { |
| |
| @Inject |
| private AccountCreator accounts; |
| |
| @Inject |
| private SchemaFactory<ReviewDb> reviewDbProvider; |
| |
| private TestAccount admin; |
| private RestSession session; |
| private Git git; |
| private ReviewDb db; |
| |
| @Before |
| public void setUp() throws Exception { |
| admin = accounts.admin(); |
| session = new RestSession(server, admin); |
| |
| initSsh(admin); |
| Project.NameKey project = new Project.NameKey("p"); |
| SshSession sshSession = new SshSession(server, admin); |
| createProject(sshSession, project.get()); |
| git = cloneProject(sshSession.getUrl() + "/" + project.get()); |
| sshSession.close(); |
| db = reviewDbProvider.open(); |
| } |
| |
| @After |
| public void cleanup() { |
| db.close(); |
| } |
| |
| @Test |
| public void getRelatedNoResult() throws GitAPIException, |
| IOException, Exception { |
| PushOneCommit push = new PushOneCommit(db, admin.getIdent()); |
| PatchSet.Id ps = push.to(git, "refs/for/master").getPatchSetId(); |
| List<ChangeAndCommit> related = getRelated(ps); |
| assertEquals(0, related.size()); |
| } |
| |
| @Test |
| public void getRelatedLinear() throws GitAPIException, |
| IOException, Exception { |
| add(git, "a.txt", "1"); |
| Commit c1 = createCommit(git, admin.getIdent(), "subject: 1"); |
| add(git, "b.txt", "2"); |
| Commit c2 = createCommit(git, admin.getIdent(), "subject: 2"); |
| pushHead(git, "refs/for/master", false); |
| |
| for (Commit c : ImmutableList.of(c2, c1)) { |
| List<ChangeAndCommit> related = getRelated(getPatchSetId(c)); |
| assertEquals(2, related.size()); |
| assertEquals("related to " + c.getChangeId(), c2.getChangeId(), related.get(0).changeId); |
| assertEquals("related to " + c.getChangeId(), c1.getChangeId(), related.get(1).changeId); |
| } |
| } |
| |
| @Test |
| public void getRelatedReorder() throws GitAPIException, |
| IOException, Exception { |
| // Create two commits and push. |
| add(git, "a.txt", "1"); |
| Commit c1 = createCommit(git, admin.getIdent(), "subject: 1"); |
| add(git, "b.txt", "2"); |
| Commit c2 = createCommit(git, admin.getIdent(), "subject: 2"); |
| pushHead(git, "refs/for/master", false); |
| PatchSet.Id c1ps1 = getPatchSetId(c1); |
| PatchSet.Id c2ps1 = getPatchSetId(c2); |
| |
| // Swap the order of commits and push again. |
| git.reset().setMode(ResetType.HARD).setRef("HEAD^^").call(); |
| git.cherryPick().include(c2.getCommit()).include(c1.getCommit()).call(); |
| pushHead(git, "refs/for/master", false); |
| PatchSet.Id c1ps2 = getPatchSetId(c1); |
| PatchSet.Id c2ps2 = getPatchSetId(c2); |
| |
| for (PatchSet.Id ps : ImmutableList.of(c2ps2, c1ps2)) { |
| List<ChangeAndCommit> related = getRelated(ps); |
| assertEquals(2, related.size()); |
| assertEquals("related to " + ps, c1.getChangeId(), related.get(0).changeId); |
| assertEquals("related to " + ps, c2.getChangeId(), related.get(1).changeId); |
| } |
| |
| for (PatchSet.Id ps : ImmutableList.of(c2ps1, c1ps1)) { |
| List<ChangeAndCommit> related = getRelated(ps); |
| assertEquals(2, related.size()); |
| assertEquals("related to " + ps, c2.getChangeId(), related.get(0).changeId); |
| assertEquals("related to " + ps, c1.getChangeId(), related.get(1).changeId); |
| } |
| } |
| |
| @Test |
| public void getRelatedReorderAndExtend() throws GitAPIException, |
| IOException, Exception { |
| // Create two commits and push. |
| add(git, "a.txt", "1"); |
| Commit c1 = createCommit(git, admin.getIdent(), "subject: 1"); |
| add(git, "b.txt", "2"); |
| Commit c2 = createCommit(git, admin.getIdent(), "subject: 2"); |
| pushHead(git, "refs/for/master", false); |
| PatchSet.Id c1ps1 = getPatchSetId(c1); |
| PatchSet.Id c2ps1 = getPatchSetId(c2); |
| |
| // Swap the order of commits, create a new commit on top, and push again. |
| git.reset().setMode(ResetType.HARD).setRef("HEAD^^").call(); |
| git.cherryPick().include(c2.getCommit()).include(c1.getCommit()).call(); |
| add(git, "c.txt", "3"); |
| Commit c3 = createCommit(git, admin.getIdent(), "subject: 3"); |
| pushHead(git, "refs/for/master", false); |
| PatchSet.Id c1ps2 = getPatchSetId(c1); |
| PatchSet.Id c2ps2 = getPatchSetId(c2); |
| PatchSet.Id c3ps1 = getPatchSetId(c3); |
| |
| |
| for (PatchSet.Id ps : ImmutableList.of(c3ps1, c2ps2, c1ps2)) { |
| List<ChangeAndCommit> related = getRelated(ps); |
| assertEquals(3, related.size()); |
| assertEquals("related to " + ps, c3.getChangeId(), related.get(0).changeId); |
| assertEquals("related to " + ps, c1.getChangeId(), related.get(1).changeId); |
| assertEquals("related to " + ps, c2.getChangeId(), related.get(2).changeId); |
| } |
| |
| for (PatchSet.Id ps : ImmutableList.of(c2ps1, c1ps1)) { |
| List<ChangeAndCommit> related = getRelated(ps); |
| assertEquals(3, related.size()); |
| assertEquals("related to " + ps, c3.getChangeId(), related.get(0).changeId); |
| assertEquals("related to " + ps, c2.getChangeId(), related.get(1).changeId); |
| assertEquals("related to " + ps, c1.getChangeId(), related.get(2).changeId); |
| } |
| } |
| |
| private List<ChangeAndCommit> getRelated(PatchSet.Id ps) throws IOException { |
| String url = String.format("/changes/%d/revisions/%d/related", |
| ps.getParentKey().get(), ps.get()); |
| RelatedInfo related = OutputFormat.JSON_COMPACT.newGson().fromJson( |
| session.get(url).getReader(), |
| new TypeToken<RelatedInfo>() {}.getType()); |
| return related.changes; |
| } |
| |
| private PatchSet.Id getPatchSetId(Commit c) throws OrmException { |
| return Iterables.getOnlyElement( |
| db.changes().byKey(new Change.Key(c.getChangeId()))).currentPatchSetId(); |
| } |
| } |