From 9c50041284b77edbb6c068bdcc391b3d01160c81 Mon Sep 17 00:00:00 2001 From: Kirill Likhodedov Date: Sat, 28 Sep 2013 16:14:03 +0400 Subject: [PATCH] [log] Remove tests on the CommitParser, because it is a test-only parser --- .../vcs/log/parser/CommitDataParserTest.java | 67 ------------------- .../vcs/log/parser/CommitParserTest.java | 53 --------------- .../parser/TimestampVcsCommitParserTest.java | 55 --------------- 3 files changed, 175 deletions(-) delete mode 100644 platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitDataParserTest.java delete mode 100644 platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitParserTest.java delete mode 100644 platform/vcs-log/graph/test/com/intellij/vcs/log/parser/TimestampVcsCommitParserTest.java diff --git a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitDataParserTest.java b/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitDataParserTest.java deleted file mode 100644 index e052f11e2096..000000000000 --- a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitDataParserTest.java +++ /dev/null @@ -1,67 +0,0 @@ -package com.intellij.vcs.log.parser; - -import com.intellij.vcs.log.VcsShortCommitDetails; -import junit.framework.Assert; -import org.jetbrains.annotations.NotNull; -import org.junit.Test; - -import static junit.framework.Assert.assertEquals; - -/** - * @author erokhins - */ -public class CommitDataParserTest { - - private static String toStr(@NotNull VcsShortCommitDetails commitData) { - StringBuilder s = new StringBuilder(); - s.append(commitData.getHash()).append("|-"); - s.append(commitData.getAuthorName()).append("|-"); - s.append(commitData.getAuthorTime()).append("|-"); - s.append(commitData.getSubject()); - return s.toString(); - } - - private void runTest(@NotNull String inputStr) { - VcsShortCommitDetails commitData = CommitParser.parseCommitData(inputStr); - assertEquals(inputStr, toStr(commitData)); - } - - @Test - public void simple1() { - runTest("af56|-author|-1435|-message"); - } - - @Test - public void emptyMessage() { - runTest("12|-author|-1435|-"); - } - - @Test - public void longAuthor() { - runTest("af56|-author skdhfb j 2353246|-1435|-message"); - } - - @Test - public void strangeMessage() { - runTest("af56|-author |-1435|-m|-|-es dfsage"); - } - - @Test - public void bigTimestamp() { - runTest("af56|-author |-143523623|-m|-|-es dfsage"); - } - - @Test - public void emptyAuthor() { - runTest("af56|-|-143523623|-"); - } - - @Test - public void emptyTimestamp() { - VcsShortCommitDetails commitData = CommitParser.parseCommitData("af56|-author |-|-message"); - Assert.assertEquals("author ", commitData.getAuthorName()); - Assert.assertEquals(0, commitData.getAuthorTime()); - Assert.assertEquals("message", commitData.getSubject()); - } - -} diff --git a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitParserTest.java b/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitParserTest.java deleted file mode 100644 index 61e40283fecc..000000000000 --- a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/CommitParserTest.java +++ /dev/null @@ -1,53 +0,0 @@ -package com.intellij.vcs.log.parser; - -import com.intellij.vcs.log.Hash; -import com.intellij.vcs.log.VcsCommit; -import org.junit.Test; - -import static junit.framework.Assert.assertEquals; - -/** - * @author erokhins - */ -public class CommitParserTest { - private String toStr(VcsCommit commitParentHashes) { - StringBuilder s = new StringBuilder(); - s.append(commitParentHashes.getHash().asString()).append("|-"); - for (int i = 0; i < commitParentHashes.getParents().size(); i++) { - Hash hash = commitParentHashes.getParents().get(i); - if (i != 0) { - s.append(" "); - } - s.append(hash.asString()); - } - return s.toString(); - } - - private void runTest(String inputStr) { - VcsCommit vcsCommit = CommitParser.parseCommitParents(inputStr); - assertEquals(inputStr, toStr(vcsCommit)); - } - - @Test - public void simple1() { - runTest("a312|-"); - } - - @Test - public void parent() { - runTest("a312|-23"); - } - - @Test - public void twoParent() { - runTest("a312|-23 a54"); - } - - - @Test - public void moreParent() { - runTest("a312|-23 a54 abcdef34 034f 00af 00000"); - } - - -} diff --git a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/TimestampVcsCommitParserTest.java b/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/TimestampVcsCommitParserTest.java deleted file mode 100644 index 1cb316019d67..000000000000 --- a/platform/vcs-log/graph/test/com/intellij/vcs/log/parser/TimestampVcsCommitParserTest.java +++ /dev/null @@ -1,55 +0,0 @@ -package com.intellij.vcs.log.parser; - -import com.intellij.vcs.log.Hash; -import com.intellij.vcs.log.TimedVcsCommit; -import org.junit.Test; - -import static junit.framework.Assert.assertEquals; - -/** - * @author erokhins - */ -public class TimestampVcsCommitParserTest { - - private String toStr(TimedVcsCommit commitParents) { - StringBuilder s = new StringBuilder(); - s.append(commitParents.getAuthorTime()).append("|-"); - s.append(commitParents.getHash().asString()).append("|-"); - for (int i = 0; i < commitParents.getParents().size(); i++) { - Hash hash = commitParents.getParents().get(i); - if (i != 0) { - s.append(" "); - } - s.append(hash.asString()); - } - return s.toString(); - } - - - private void runTest(String inputStr) { - TimedVcsCommit commitParents = CommitParser.parseTimestampParentHashes(inputStr); - assertEquals(inputStr, toStr(commitParents)); - } - - @Test - public void simple() { - runTest("1|-af|-"); - } - - @Test - public void parents() { - runTest("12314|-af|-12 fd"); - } - - - @Test - public void parent() { - runTest("12314|-af|-12"); - } - - - @Test - public void longTest() { - runTest("123142412423412|-af|-12"); - } -}