| 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), |
| Ivan Frade | 9ca395d | 2019-11-25 13:29:49 -0800 | [diff] [blame] | 49 | new VisibilityCache(CacheBuilder.newBuilder().maximumSize(0))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 50 | } |
| 51 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 52 | @Test |
| 53 | public void parseRef() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 54 | RevCommit master = repo.branch("refs/heads/master").commit().create(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 55 | assertThat(parser.parse("master")).isEqualTo(new Result(Revision.peeled("master", master))); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 56 | assertThat(parser.parse("refs/heads/master")) |
| 57 | .isEqualTo(new Result(Revision.peeled("refs/heads/master", master))); |
| 58 | assertThat(parser.parse("refs//heads//master")).isNull(); |
| 59 | assertThat(parser.parse("refs heads master")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 60 | } |
| 61 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 62 | @Test |
| 63 | public void parseRefParentExpression() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 64 | RevCommit root = repo.commit().create(); |
| 65 | RevCommit parent1 = repo.commit().parent(root).create(); |
| 66 | RevCommit parent2 = repo.commit().parent(root).create(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 67 | RevCommit merge = repo.branch("master").commit().parent(parent1).parent(parent2).create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 68 | assertThat(parser.parse("master")).isEqualTo(new Result(Revision.peeled("master", merge))); |
| 69 | assertThat(parser.parse("master^")).isEqualTo(new Result(Revision.peeled("master^", parent1))); |
| 70 | assertThat(parser.parse("master~1")) |
| 71 | .isEqualTo(new Result(Revision.peeled("master~1", parent1))); |
| 72 | assertThat(parser.parse("master^2")) |
| 73 | .isEqualTo(new Result(Revision.peeled("master^2", parent2))); |
| 74 | assertThat(parser.parse("master^3")).isNull(); |
| 75 | 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] | 76 | } |
| 77 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 78 | @Test |
| 79 | public void parseCommitShaVisibleFromHead() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 80 | RevCommit parent = repo.commit().create(); |
| 81 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 82 | assertThat(parser.parse(commit.name())) |
| 83 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 84 | assertThat(parser.parse(parent.name())) |
| 85 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 86 | |
| 87 | String abbrev = commit.name().substring(0, 6); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 88 | assertThat(parser.parse(abbrev)).isEqualTo(new Result(Revision.peeled(abbrev, commit))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 89 | } |
| 90 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 91 | @Test |
| 92 | public void parseCommitShaVisibleFromTag() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 93 | RevCommit parent = repo.commit().create(); |
| 94 | RevCommit commit = repo.commit().parent(parent).create(); |
| 95 | repo.branch("master").commit().create(); |
| 96 | repo.update("refs/tags/tag", repo.tag("tag", commit)); |
| 97 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 98 | assertThat(parser.parse(commit.name())) |
| 99 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 100 | assertThat(parser.parse(parent.name())) |
| 101 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 102 | } |
| 103 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 104 | @Test |
| 105 | public void parseCommitShaVisibleFromOther() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 106 | RevCommit parent = repo.commit().create(); |
| 107 | RevCommit commit = repo.commit().parent(parent).create(); |
| 108 | repo.branch("master").commit().create(); |
| 109 | repo.update("refs/tags/tag", repo.tag("tag", repo.commit().create())); |
| 110 | repo.update("refs/meta/config", commit); |
| 111 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 112 | assertThat(parser.parse(commit.name())) |
| 113 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| 114 | assertThat(parser.parse(parent.name())) |
| 115 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 116 | } |
| 117 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 118 | @Test |
| 119 | public void parseCommitShaVisibleFromChange() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 120 | RevCommit parent = repo.commit().create(); |
| 121 | RevCommit commit = repo.commit().parent(parent).create(); |
| 122 | repo.branch("master").commit().create(); |
| 123 | repo.update("refs/changes/01/0001", commit); |
| 124 | |
| 125 | // Matches exactly. |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 126 | assertThat(parser.parse(commit.name())) |
| 127 | .isEqualTo(new Result(Revision.peeled(commit.name(), commit))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 128 | // refs/changes/* is excluded from ancestry search. |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 129 | assertThat(parser.parse(parent.name())).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 130 | } |
| 131 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 132 | @Test |
| 133 | public void parseNonVisibleCommitSha() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 134 | RevCommit other = repo.commit().create(); |
| Dave Borowitz | a12b147 | 2012-11-12 14:46:28 -0800 | [diff] [blame] | 135 | repo.branch("master").commit().create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 136 | assertThat(parser.parse(other.name())).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 137 | |
| 138 | repo.branch("other").update(other); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 139 | assertThat(parser.parse(other.name())) |
| 140 | .isEqualTo(new Result(Revision.peeled(other.name(), other))); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 141 | } |
| 142 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 143 | @Test |
| 144 | public void parseDiffRevisions() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 145 | RevCommit parent = repo.commit().create(); |
| 146 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| 147 | RevCommit other = repo.branch("other").commit().create(); |
| 148 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 149 | assertThat(parser.parse("master^..master")) |
| 150 | .isEqualTo( |
| 151 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "")); |
| 152 | assertThat(parser.parse("master^..master/")) |
| 153 | .isEqualTo( |
| 154 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "/")); |
| 155 | assertThat(parser.parse("master^..master/path/to/a/file")) |
| 156 | .isEqualTo( |
| 157 | new Result( |
| 158 | Revision.peeled("master", commit), |
| 159 | Revision.peeled("master^", parent), |
| 160 | "/path/to/a/file")); |
| 161 | assertThat(parser.parse("master^..master/path/to/a/..file")) |
| 162 | .isEqualTo( |
| 163 | new Result( |
| 164 | Revision.peeled("master", commit), |
| 165 | Revision.peeled("master^", parent), |
| 166 | "/path/to/a/..file")); |
| 167 | assertThat(parser.parse("refs/heads/master^..refs/heads/master")) |
| 168 | .isEqualTo( |
| 169 | new Result( |
| 170 | Revision.peeled("refs/heads/master", commit), |
| 171 | Revision.peeled("refs/heads/master^", parent), |
| 172 | "")); |
| 173 | assertThat(parser.parse("master~1..master")) |
| 174 | .isEqualTo( |
| 175 | new Result(Revision.peeled("master", commit), Revision.peeled("master~1", parent), "")); |
| David Pursehouse | 2bf4a70 | 2018-08-09 22:33:50 +0100 | [diff] [blame] | 176 | assertThat(parser.parse("master~2..master")).isNull(); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 177 | assertThat(parser.parse("other..master")) |
| 178 | .isEqualTo( |
| 179 | new Result(Revision.peeled("master", commit), Revision.peeled("other", other), "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 180 | } |
| 181 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 182 | @Test |
| 183 | public void parseFirstParentExpression() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 184 | RevCommit parent = repo.commit().create(); |
| 185 | RevCommit commit = repo.branch("master").commit().parent(parent).create(); |
| 186 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 187 | assertThat(parser.parse("master^!")) |
| 188 | .isEqualTo( |
| 189 | new Result(Revision.peeled("master", commit), Revision.peeled("master^", parent), "")); |
| 190 | assertThat(parser.parse("master^^!")) |
| 191 | .isEqualTo(new Result(Revision.peeled("master^", parent), Revision.NULL, "")); |
| 192 | assertThat(parser.parse(parent.name() + "^!")) |
| 193 | .isEqualTo(new Result(Revision.peeled(parent.name(), parent), Revision.NULL, "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 194 | |
| Dave Borowitz | a12b147 | 2012-11-12 14:46:28 -0800 | [diff] [blame] | 195 | repo.update("refs/tags/tag", repo.tag("tag", commit)); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 196 | assertThat(parser.parse("tag^!")) |
| 197 | .isEqualTo(new Result(Revision.peeled("tag", commit), Revision.peeled("tag^", parent), "")); |
| 198 | assertThat(parser.parse("tag^^!")) |
| 199 | .isEqualTo(new Result(Revision.peeled("tag^", parent), Revision.NULL, "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 200 | } |
| 201 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 202 | @Test |
| 203 | public void nonVisibleDiffShas() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 204 | RevCommit other = repo.commit().create(); |
| 205 | RevCommit master = repo.branch("master").commit().create(); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 206 | assertThat(parser.parse("other..master")).isNull(); |
| 207 | assertThat(parser.parse("master..other")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 208 | |
| 209 | repo.branch("other").update(other); |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 210 | assertThat(parser.parse("other..master")) |
| 211 | .isEqualTo( |
| 212 | new Result(Revision.peeled("master", master), Revision.peeled("other", other), "")); |
| 213 | assertThat(parser.parse("master..other")) |
| 214 | .isEqualTo( |
| 215 | new Result(Revision.peeled("other", other), Revision.peeled("master", master), "")); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 216 | } |
| 217 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 218 | @Test |
| 219 | public void parseTag() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 220 | RevCommit master = repo.branch("master").commit().create(); |
| 221 | 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] | 222 | RevTag masterTagTag = |
| 223 | 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] | 224 | |
| Dave Borowitz | cf38c03 | 2016-05-02 11:06:23 -0400 | [diff] [blame] | 225 | assertThat(parser.parse("master-tag")) |
| 226 | .isEqualTo(new Result(new Revision("master-tag", masterTag, OBJ_TAG, master, OBJ_COMMIT))); |
| 227 | assertThat(parser.parse("master-tag-tag")) |
| 228 | .isEqualTo( |
| 229 | 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] | 230 | |
| 231 | RevBlob blob = repo.update("refs/tags/blob", repo.blob("blob")); |
| 232 | 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] | 233 | assertThat(parser.parse("blob")).isEqualTo(new Result(Revision.peeled("blob", blob))); |
| 234 | assertThat(parser.parse("blob-tag")) |
| 235 | .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] | 236 | } |
| 237 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 238 | @Test |
| 239 | public void parseUnsupportedRevisionExpressions() throws Exception { |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 240 | RevBlob blob = repo.blob("blob contents"); |
| 241 | RevCommit master = repo.branch("master").commit().add("blob", blob).create(); |
| 242 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 243 | assertThat(repo.getRepository().resolve("master^{}")).isEqualTo(master); |
| 244 | assertThat(parser.parse("master^{}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 245 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 246 | assertThat(repo.getRepository().resolve("master^{commit}")).isEqualTo(master); |
| 247 | assertThat(parser.parse("master^{commit}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 248 | |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 249 | assertThat(repo.getRepository().resolve("master:blob")).isEqualTo(blob); |
| 250 | assertThat(parser.parse("master:blob")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 251 | |
| Dave Borowitz | 63de61a | 2018-08-15 14:05:55 -0700 | [diff] [blame] | 252 | // InMemoryRepository doesn't implement a reflog, so we can't test reflog entries. |
| 253 | // https://bugs.eclipse.org/bugs/show_bug.cgi?id=537972 |
| David Pursehouse | 0138abf | 2018-07-25 10:00:17 +0100 | [diff] [blame] | 254 | // assertThat(repo.getRepository().resolve("master@{0}")).isEqualTo(null); |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 255 | assertThat(parser.parse("master@{0}")).isNull(); |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 256 | } |
| Dave Borowitz | 48ca670 | 2013-04-09 11:52:41 -0700 | [diff] [blame] | 257 | |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 258 | @Test |
| Jonathan Nieder | 4a7dac0 | 2019-07-01 16:15:03 -0700 | [diff] [blame] | 259 | public void parseEmailInRevision() throws Exception { |
| 260 | RevCommit c = repo.commit().create(); |
| David Pursehouse | 12fda93 | 2019-07-10 10:11:21 +0900 | [diff] [blame] | 261 | repo.update("refs/experimental/[email protected]/foo", c); |
| Jonathan Nieder | 4a7dac0 | 2019-07-01 16:15:03 -0700 | [diff] [blame] | 262 | assertThat(parser.parse("refs/experimental/[email protected]/foo")) |
| 263 | .isEqualTo(new Result(Revision.peeled("refs/experimental/[email protected]/foo", c))); |
| 264 | } |
| 265 | |
| 266 | @Test |
| Dave Borowitz | d40bdf1 | 2014-04-19 19:33:56 -0700 | [diff] [blame] | 267 | public void parseMissingSha() throws Exception { |
| Dave Borowitz | fde41fd | 2015-09-16 15:14:38 -0400 | [diff] [blame] | 268 | assertThat(parser.parse("deadbeef")).isNull(); |
| 269 | assertThat(parser.parse("deadbeefdeadbeefdeadbeefdeadbeefdeadbeef")).isNull(); |
| Dave Borowitz | 48ca670 | 2013-04-09 11:52:41 -0700 | [diff] [blame] | 270 | } |
| Dave Borowitz | 9de6595 | 2012-08-13 16:09:45 -0700 | [diff] [blame] | 271 | } |