testGetTagByNameUnannotated() test added

Author: host6

src/main/java/org/scm4j/vcs/svn/SVNVCS.java

@@ -617,10 +617,10 @@ public VCSTag createTag(String branchName, String tagName, String tagMessage, St
 
 			SVNDirEntry entry = repository.info(TAGS_PATH + tagName, -1);
 
-			SVNLogEntry copyFromEntry = revToSVNEntry(getBranchName(branchName), Long.parseLong(revisionToTag));
+			SVNLogEntry copyFromEntry = revToSVNEntry(getBranchName(branchName),
+					revisionToTag == null ? SVNRevision.HEAD.getNumber() : Long.parseLong(revisionToTag));
 
-			VCSTag tag = new VCSTag(tagName, tagMessage, entry.getAuthor(), svnLogEntryToVCSCommit(copyFromEntry));
-			return tag;
+			return new VCSTag(tagName, tagMessage, entry.getAuthor(), svnLogEntryToVCSCommit(copyFromEntry));
 		} catch (SVNException e) {
 			if (e.getErrorMessage().getErrorCode().getCode() == SVN_ITEM_EXISTS_ERROR_CODE) {
 				throw new EVCSTagExists(e);

src/test/java/org/scm4j/vcs/svn/SVNVCSTest.java

@@ -433,4 +433,23 @@ public void testListEntriesNone() throws Exception {
 		doReturn(SVNNodeKind.NONE).when(mockedRepo).checkPath(anyString(), anyLong());
 		svn.listEntries(null, null, null); // expecting no NPE 
 	}
+
+	@Test
+	public void testGetTagByNameExceptions() throws Exception {
+		doThrow(testSVNException).when(svn).revToSVNEntry(anyString(), anyLong());
+		try {
+			vcs.getTagByName("");
+			fail();
+		} catch (EVCSException e) {
+			checkEVCSException(e);
+		}
+
+		doThrow(testCommonException).when(svn).revToSVNEntry(anyString(), anyLong());
+		try {
+			vcs.getTagByName("");
+			fail();
+		} catch (RuntimeException e) {
+			checkCommonException(e);
+		}
+	}
 }