| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 1 | // Copyright 2012 Google Inc. All Rights Reserved. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
| 15 | package com.google.gitiles; |
| 16 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 17 | import static com.google.common.truth.Truth.assertThat; |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 18 | import static org.eclipse.jgit.lib.Constants.OBJ_BLOB; |
| 19 | import static org.eclipse.jgit.lib.Constants.OBJ_COMMIT; |
| 20 | import static org.eclipse.jgit.lib.Constants.OBJ_TAG; |
| Dave Borowitz | 48ca670 | 2013-04-09 11:52:41 -0700 | [diff] [blame] | 21 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 22 | import com.google.common.cache.CacheBuilder; |
| 23 | import com.google.gitiles.RevisionParser.Result; |
| Shawn Pearce | b43b2d5 | 2013-03-18 10:55:15 -0700 | [diff] [blame] | 24 | import org.eclipse.jgit.internal.storage.dfs.DfsRepository; |
| 25 | import org.eclipse.jgit.internal.storage.dfs.DfsRepositoryDescription; |
| 26 | import org.eclipse.jgit.internal.storage.dfs.InMemoryRepository; |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 27 | import org.eclipse.jgit.junit.TestRepository; |
| 28 | import org.eclipse.jgit.revwalk.RevBlob; |
| 29 | import org.eclipse.jgit.revwalk.RevCommit; |
| 30 | import org.eclipse.jgit.revwalk.RevTag; |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 31 | import org.junit.Before; |
| 32 | import org.junit.Test; |
| Dave Borowitz | 3dc854f | 2014-11-04 16:19:37 -0800 | [diff] [blame] | 33 | import org.junit.runner.RunWith; |
| 34 | import org.junit.runners.JUnit4; |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 35 | |
| 36 | /** Tests for the revision parser. */ |
| Dave Borowitz | 3dc854f | 2014-11-04 16:19:37 -0800 | [diff] [blame] | 37 | @RunWith(JUnit4.class) |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 38 | public class RevisionParserTest { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 39 | private TestRepository<DfsRepository> repo; |
| 40 | private RevisionParser parser; |
| 41 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 42 | @Before |
| 43 | public void setUp() throws Exception { |
| David Pursehouse | ccaa85d | 2017-05-30 10:47:27 +0900 | [diff] [blame] | 44 | repo = new TestRepository<>(new InMemoryRepository(new DfsRepositoryDescription("test"))); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 45 | parser = |
| 46 | new RevisionParser( |
| 47 | repo.getRepository(), |
| 48 | new TestGitilesAccess(repo.getRepository()).forRequest(null), |
| Ronald Bhuleskar | 999a71d | 2021-11-19 15:24:51 -0800 | [diff] [blame] | 49 | new VisibilityCache(CacheBuilder.newBuilder().maximumSize(0)), |
| 50 | new BranchRedirect()); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 51 | } |
| 52 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 53 | @Test |
| 54 | public void parseRef() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 55 | RevCommit master = repo.branch("refs/heads/master").commit().create(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 56 | assertThat(parser.parse("master")).isEqualTo(new Result(Revision.peeled("master", master))); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 57 | assertThat(parser.parse("refs/heads/master")) |
| 58 | .isEqualTo(new Result(Revision.peeled("refs/heads/master", master))); |
| 59 | assertThat(parser.parse("refs//heads//master")).isNull(); |
| 60 | assertThat(parser.parse("refs heads master")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 61 | } |
| 62 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 63 | @Test |
| 64 | public void parseRefParentExpression() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 65 | RevCommit root = repo.commit().create(); |
| 66 | RevCommit parent1 = repo.commit().parent(root).create(); |
| 67 | RevCommit parent2 = repo.commit().parent(root).create(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 68 | RevCommit merge = repo.branch("master").commit().parent(parent1).parent(parent2).create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 69 | assertThat(parser.parse("master")).isEqualTo(new Result(Revision.peeled("master", merge))); |
| 70 | assertThat(parser.parse("master^")).isEqualTo(new Result(Revision.peeled("master^", parent1))); |
| 71 | assertThat(parser.parse("master~1")) |
| 72 | .isEqualTo(new Result(Revision.peeled("master~1", parent1))); |
| 73 | assertThat(parser.parse("master^2")) |
| 74 | .isEqualTo(new Result(Revision.peeled("master^2", parent2))); |
| 75 | assertThat(parser.parse("master^3")).isNull(); |
| 76 | assertThat(parser.parse("master~2")).isEqualTo(new Result(Revision.peeled("master~2", root))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 77 | } |
| 78 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 79 | @Test |
| 80 | public void parseCommitShaVisibleFromHead() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 81 | RevCommit parent = repo.commit().create(); |
| 82 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 83 | assertThat(parser.parse(commit.name())) |
| 84 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 85 | assertThat(parser.parse(parent.name())) |
| 86 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 87 | |
| 88 | String abbrev = commit.name().substring(0, 6); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 89 | assertThat(parser.parse(abbrev)).isEqualTo(new Result(Revision.peeled(abbrev, commit))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 90 | } |
| 91 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 92 | @Test |
| 93 | public void parseCommitShaVisibleFromTag() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 94 | RevCommit parent = repo.commit().create(); |
| 95 | RevCommit commit = repo.commit().parent(parent).create(); |
| 96 | repo.branch("master").commit().create(); |
| 97 | repo.update("refs/tags/tag", repo.tag("tag", commit)); |
| 98 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 99 | assertThat(parser.parse(commit.name())) |
| 100 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 101 | assertThat(parser.parse(parent.name())) |
| 102 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 103 | } |
| 104 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 105 | @Test |
| 106 | public void parseCommitShaVisibleFromOther() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 107 | RevCommit parent = repo.commit().create(); |
| 108 | RevCommit commit = repo.commit().parent(parent).create(); |
| 109 | repo.branch("master").commit().create(); |
| 110 | repo.update("refs/tags/tag", repo.tag("tag", repo.commit().create())); |
| 111 | repo.update("refs/meta/config", commit); |
| 112 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 113 | assertThat(parser.parse(commit.name())) |
| 114 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 115 | assertThat(parser.parse(parent.name())) |
| 116 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 117 | } |
| 118 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 119 | @Test |
| 120 | public void parseCommitShaVisibleFromChange() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 121 | RevCommit parent = repo.commit().create(); |
| 122 | RevCommit commit = repo.commit().parent(parent).create(); |
| 123 | repo.branch("master").commit().create(); |
| 124 | repo.update("refs/changes/01/0001", commit); |
| 125 | |
| 126 | // Matches exactly. |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 127 | assertThat(parser.parse(commit.name())) |
| 128 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 129 | // refs/changes/* is excluded from ancestry search. |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 130 | assertThat(parser.parse(parent.name())).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 131 | } |
| 132 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 133 | @Test |
| 134 | public void parseNonVisibleCommitSha() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 135 | RevCommit other = repo.commit().create(); |
| Dave Borowitz | a12b147 | 2012-11-12 14:46:28 -0800 | [diff] [blame] | 136 | repo.branch("master").commit().create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 137 | assertThat(parser.parse(other.name())).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 138 | |
| 139 | repo.branch("other").update(other); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 140 | assertThat(parser.parse(other.name())) |
| 141 | .isEqualTo(new Result(Revision.peeled(other.name(), other))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 142 | } |
| 143 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 144 | @Test |
| 145 | public void parseDiffRevisions() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 146 | RevCommit parent = repo.commit().create(); |
| 147 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| 148 | RevCommit other = repo.branch("other").commit().create(); |
| 149 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 150 | assertThat(parser.parse("master^..master")) |
| 151 | .isEqualTo( |
| 152 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "")); |
| 153 | assertThat(parser.parse("master^..master/")) |
| 154 | .isEqualTo( |
| 155 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "/")); |
| 156 | assertThat(parser.parse("master^..master/path/to/a/file")) |
| 157 | .isEqualTo( |
| 158 | new Result( |
| 159 | Revision.peeled("master", commit), |
| 160 | Revision.peeled("master^", parent), |
| 161 | "/path/to/a/file")); |
| 162 | assertThat(parser.parse("master^..master/path/to/a/..file")) |
| 163 | .isEqualTo( |
| 164 | new Result( |
| 165 | Revision.peeled("master", commit), |
| 166 | Revision.peeled("master^", parent), |
| 167 | "/path/to/a/..file")); |
| 168 | assertThat(parser.parse("refs/heads/master^..refs/heads/master")) |
| 169 | .isEqualTo( |
| 170 | new Result( |
| 171 | Revision.peeled("refs/heads/master", commit), |
| 172 | Revision.peeled("refs/heads/master^", parent), |
| 173 | "")); |
| 174 | assertThat(parser.parse("master~1..master")) |
| 175 | .isEqualTo( |
| 176 | new Result(Revision.peeled("master", commit), Revision.peeled("master~1", parent), "")); |
| David Pursehouse | 2bf4a70 | 2018-08-09 22:33:50 +0100 | [diff] [blame] | 177 | assertThat(parser.parse("master~2..master")).isNull(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 178 | assertThat(parser.parse("other..master")) |
| 179 | .isEqualTo( |
| 180 | new Result(Revision.peeled("master", commit), Revision.peeled("other", other), "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 181 | } |
| 182 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 183 | @Test |
| 184 | public void parseFirstParentExpression() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 185 | RevCommit parent = repo.commit().create(); |
| 186 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| 187 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 188 | assertThat(parser.parse("master^!")) |
| 189 | .isEqualTo( |
| 190 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "")); |
| 191 | assertThat(parser.parse("master^^!")) |
| 192 | .isEqualTo(new Result(Revision.peeled("master^", parent), Revision.NULL, "")); |
| 193 | assertThat(parser.parse(parent.name() + "^!")) |
| 194 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent), Revision.NULL, "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 195 | |
| Dave Borowitz | a12b147 | 2012-11-12 14:46:28 -0800 | [diff] [blame] | 196 | repo.update("refs/tags/tag", repo.tag("tag", commit)); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 197 | assertThat(parser.parse("tag^!")) |
| 198 | .isEqualTo(new Result(Revision.peeled("tag", commit), Revision.peeled("tag^", parent), "")); |
| 199 | assertThat(parser.parse("tag^^!")) |
| 200 | .isEqualTo(new Result(Revision.peeled("tag^", parent), Revision.NULL, "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 201 | } |
| 202 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 203 | @Test |
| 204 | public void nonVisibleDiffShas() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 205 | RevCommit other = repo.commit().create(); |
| 206 | RevCommit master = repo.branch("master").commit().create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 207 | assertThat(parser.parse("other..master")).isNull(); |
| 208 | assertThat(parser.parse("master..other")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 209 | |
| 210 | repo.branch("other").update(other); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 211 | assertThat(parser.parse("other..master")) |
| 212 | .isEqualTo( |
| 213 | new Result(Revision.peeled("master", master), Revision.peeled("other", other), "")); |
| 214 | assertThat(parser.parse("master..other")) |
| 215 | .isEqualTo( |
| 216 | new Result(Revision.peeled("other", other), Revision.peeled("master", master), "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 217 | } |
| 218 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 219 | @Test |
| 220 | public void parseTag() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 221 | RevCommit master = repo.branch("master").commit().create(); |
| 222 | RevTag masterTag = repo.update("refs/tags/master-tag", repo.tag("master-tag", master)); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 223 | RevTag masterTagTag = |
| 224 | repo.update("refs/tags/master-tag-tag", repo.tag("master-tag-tag", master)); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 225 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 226 | assertThat(parser.parse("master-tag")) |
| 227 | .isEqualTo(new Result(new Revision("master-tag", masterTag, OBJ_TAG, master, OBJ_COMMIT))); |
| 228 | assertThat(parser.parse("master-tag-tag")) |
| 229 | .isEqualTo( |
| 230 | new Result(new Revision("master-tag-tag", masterTagTag, OBJ_TAG, master, OBJ_COMMIT))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 231 | |
| 232 | RevBlob blob = repo.update("refs/tags/blob", repo.blob("blob")); |
| 233 | RevTag blobTag = repo.update("refs/tags/blob-tag", repo.tag("blob-tag", blob)); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 234 | assertThat(parser.parse("blob")).isEqualTo(new Result(Revision.peeled("blob", blob))); |
| 235 | assertThat(parser.parse("blob-tag")) |
| 236 | .isEqualTo(new Result(new Revision("blob-tag", blobTag, OBJ_TAG, blob, OBJ_BLOB))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 237 | } |
| 238 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 239 | @Test |
| 240 | public void parseUnsupportedRevisionExpressions() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 241 | RevBlob blob = repo.blob("blob contents"); |
| 242 | RevCommit master = repo.branch("master").commit().add("blob", blob).create(); |
| 243 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 244 | assertThat(repo.getRepository().resolve("master^{}")).isEqualTo(master); |
| 245 | assertThat(parser.parse("master^{}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 246 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 247 | assertThat(repo.getRepository().resolve("master^{commit}")).isEqualTo(master); |
| 248 | assertThat(parser.parse("master^{commit}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 249 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 250 | assertThat(repo.getRepository().resolve("master:blob")).isEqualTo(blob); |
| 251 | assertThat(parser.parse("master:blob")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 252 | |
| Dave Borowitz | 63de61a | 2018-08-15 14:05:55 -0700 | [diff] [blame] | 253 | // InMemoryRepository doesn't implement a reflog, so we can't test reflog entries. |
| 254 | // https://bugs.eclipse.org/bugs/show_bug.cgi?id=537972 |
| David Pursehouse | 0138abf | 2018-07-25 10:00:17 +0100 | [diff] [blame] | 255 | // assertThat(repo.getRepository().resolve("master@{0}")).isEqualTo(null); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 256 | assertThat(parser.parse("master@{0}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 257 | } |
| Dave Borowitz | 48ca670 | 2013-04-09 11:52:41 -0700 | [diff] [blame] | 258 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 259 | @Test |
| Jonathan Nieder | 4a7dac0 | 2019-07-01 16:15:03 -0700 | [diff] [blame] | 260 | public void parseEmailInRevision() throws Exception { |
| 261 | RevCommit c = repo.commit().create(); |
| David Pursehouse | 12fda93 | 2019-07-10 10:11:21 +0900 | [diff] [blame] | 262 | repo.update("refs/experimental/[email protected]/foo", c); |
| Jonathan Nieder | 4a7dac0 | 2019-07-01 16:15:03 -0700 | [diff] [blame] | 263 | assertThat(parser.parse("refs/experimental/[email protected]/foo")) |
| 264 | .isEqualTo(new Result(Revision.peeled("refs/experimental/[email protected]/foo", c))); |
| 265 | } |
| 266 | |
| 267 | @Test |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 268 | public void parseMissingSha() throws Exception { |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 269 | assertThat(parser.parse("deadbeef")).isNull(); |
| 270 | assertThat(parser.parse("deadbeefdeadbeefdeadbeefdeadbeefdeadbeef")).isNull(); |
| Dave Borowitz | 48ca670 | 2013-04-09 11:52:41 -0700 | [diff] [blame] | 271 | } |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 272 | } |