diff: cleanup

This commit is contained in:
Aleksey Pivovarov
2016-10-13 22:38:29 +03:00
parent 5f32b5ac0d
commit 5f5cc77ddb
5 changed files with 29 additions and 31 deletions
@@ -24,35 +24,35 @@ class MergeResolveUtilTest : DiffTestCase() {
"",
"",
""
);
)
test(
"x x x",
"x x x",
"x x x",
"x x x"
);
)
test(
"x x x",
"x Y x",
"x x x",
"x Y x"
);
)
test(
"x x",
"x x",
"x Y x",
"x Y x"
);
)
test(
"x X x",
"x x",
"x X x",
"x x"
);
)
}
fun testSameModification() {
@@ -61,21 +61,21 @@ class MergeResolveUtilTest : DiffTestCase() {
"x Y x",
"x Y x",
"x Y x"
);
)
test(
"x x",
"x Y x",
"x Y x",
"x Y x"
);
)
test(
"x X x",
"x x",
"x x",
"x x"
);
)
}
fun testNonConflictingChanges() {
@@ -84,21 +84,21 @@ class MergeResolveUtilTest : DiffTestCase() {
"x Y x x",
"x x Z x",
"x Y x Z x"
);
)
test(
"x",
"x Y",
"Z x",
"Z x Y"
);
)
test(
"x x",
"x",
"Z x x",
"Z x"
);
)
}
fun testFailure() {
@@ -107,14 +107,14 @@ class MergeResolveUtilTest : DiffTestCase() {
"x Y x",
"x Z x",
null
);
)
test(
"x x",
"x Y x",
"x Z x",
null
);
)
}
fun testNonFailureConflicts() {
@@ -123,35 +123,35 @@ class MergeResolveUtilTest : DiffTestCase() {
"x x",
"x X Y x",
"x Y x"
);
)
test(
"x X x",
"x x",
"x Y X x",
"x Y x"
);
)
test(
"x X Y x",
"x X x",
"x Y x",
"x x"
);
)
test(
"x X Y Z x",
"x X x",
"x Z x",
"x x"
);
)
test(
"x A B C D E F G H K x",
"x C F K x",
"x A D H x",
"x x"
);
)
}
fun testConfusingConflicts() {
@@ -162,21 +162,21 @@ class MergeResolveUtilTest : DiffTestCase() {
"x x",
"x Z x",
"xZ x"
);
)
test(
"x X X x",
"x X Y X x",
"x x",
"x Y x"
);
)
test(
"x X x",
"x x",
"x Y x",
"xY x"
);
)
test(
@@ -184,11 +184,11 @@ class MergeResolveUtilTest : DiffTestCase() {
"x Y x",
"x X Y x",
"x Y x"
);
)
}
private fun test(base: String, left: String, right: String, expected: String?) {
val actual = MergeResolveUtil.tryResolveConflict(left, base, right);
val actual = MergeResolveUtil.tryResolveConflict(left, base, right)
assertEquals(expected, actual?.toString())
}
}
@@ -19,7 +19,7 @@ import com.intellij.diff.DiffTestCase
import com.intellij.openapi.util.text.StringUtil
class TrimUtilTest : DiffTestCase() {
private val PUNCTUATION = "(){}[],./?`~!@#$%^&*-=+|\\;:'\"<>";
private val PUNCTUATION = "(){}[],./?`~!@#$%^&*-=+|\\;:'\"<>"
fun testPunctuation() {
for (c in Character.MIN_VALUE..Character.MAX_VALUE) {
@@ -80,8 +80,8 @@ class MergeAutoTest : MergeTestBase() {
private fun TestBuilder.doReplace(): Unit {
val length = document.textLength
var index1: Int = 0;
var index2: Int = 0;
var index1: Int = 0
var index2: Int = 0
if (length != 0) {
index1 = RNG.nextInt(length)
index2 = index1 + RNG.nextInt(length - index1)
@@ -51,8 +51,8 @@ class `DiffUtil getSortedIndexes() Test` : DiffTestCase() {
private fun <T> doTest(vararg values: T, comparator: (T, T) -> Int) {
val list = values.toList()
val sortedIndexes = DiffUtil.getSortedIndexes(list, comparator);
val expected = ContainerUtil.sorted(list, comparator);
val sortedIndexes = DiffUtil.getSortedIndexes(list, comparator)
val expected = ContainerUtil.sorted(list, comparator)
val actual = (0..values.size - 1).map { values[sortedIndexes[it]] }
assertOrderedEquals(actual, expected)
@@ -18,10 +18,8 @@ package com.intellij.openapi.diff.impl.settings
import com.intellij.diff.DiffTestCase
import com.intellij.diff.tools.simple.SimpleThreesideDiffViewer
import com.intellij.diff.util.TextDiffType
import com.intellij.openapi.diff.impl.settings.DiffPreviewPanel
import com.intellij.openapi.editor.ex.EditorEx
import com.intellij.openapi.editor.markup.RangeHighlighter
import com.intellij.openapi.util.Disposer
import com.intellij.testFramework.fixtures.IdeaProjectTestFixture
import com.intellij.testFramework.fixtures.IdeaTestFixtureFactory
@@ -40,7 +38,7 @@ class DiffColorSettingsTest : DiffTestCase() {
}
fun testChanges() {
var panel : DiffPreviewPanel? = null;
var panel : DiffPreviewPanel? = null
try {
panel = DiffPreviewPanel()
val viewer = panel.testGetViewer()