From e79a7d3e94d18fb4503e0ee0a91d673276e253c6 Mon Sep 17 00:00:00 2001
From: Alexey Sedunov <alexey.sedunov@jetbrains.com>
Date: Mon, 17 Apr 2017 13:05:15 +0300
Subject: [PATCH] Change Signature: Update test data

---
 .../JavaMethodOverridesReplaceParamMessages.txt        |  2 ++
 .../changeSignature/KotlinChangeSignatureTest.kt       | 10 +++++++---
 2 files changed, 9 insertions(+), 3 deletions(-)
 create mode 100644 idea/testData/refactoring/changeSignature/JavaMethodOverridesReplaceParamMessages.txt

diff --git a/idea/testData/refactoring/changeSignature/JavaMethodOverridesReplaceParamMessages.txt b/idea/testData/refactoring/changeSignature/JavaMethodOverridesReplaceParamMessages.txt
new file mode 100644
index 00000000000..47f3dfb37f0
--- /dev/null
+++ b/idea/testData/refactoring/changeSignature/JavaMethodOverridesReplaceParamMessages.txt
@@ -0,0 +1,2 @@
+Parameter s is used in method body
+Parameter s is used in method body
\ No newline at end of file
diff --git a/idea/tests/org/jetbrains/kotlin/idea/refactoring/changeSignature/KotlinChangeSignatureTest.kt b/idea/tests/org/jetbrains/kotlin/idea/refactoring/changeSignature/KotlinChangeSignatureTest.kt
index 34a62b106fd..0426c6c6d55 100644
--- a/idea/tests/org/jetbrains/kotlin/idea/refactoring/changeSignature/KotlinChangeSignatureTest.kt
+++ b/idea/tests/org/jetbrains/kotlin/idea/refactoring/changeSignature/KotlinChangeSignatureTest.kt
@@ -125,9 +125,9 @@ class KotlinChangeSignatureTest : KotlinLightCodeInsightFixtureTestCase() {
         compareEditorsWithExpectedData()
     }
 
-    private fun doTestConflict(configure: KotlinChangeInfo.() -> Unit = {}) {
+    private fun runAndCheckConflicts(testAction: () -> Unit) {
         try {
-            doTest(configure)
+            testAction()
             TestCase.fail("No conflicts found")
         }
         catch (e: Throwable) {
@@ -142,6 +142,8 @@ class KotlinChangeSignatureTest : KotlinLightCodeInsightFixtureTestCase() {
         }
     }
 
+    private fun doTestConflict(configure: KotlinChangeInfo.() -> Unit = {}) = runAndCheckConflicts { doTest(configure) }
+
     private fun doTestUnmodifiable(configure: KotlinChangeInfo.() -> Unit = {}) {
         try {
             doTest(configure)
@@ -217,6 +219,8 @@ class KotlinChangeSignatureTest : KotlinLightCodeInsightFixtureTestCase() {
         compareEditorsWithExpectedData()
     }
 
+    private fun doJavaTestConflict(configure: JavaRefactoringConfiguration.() -> Unit) = runAndCheckConflicts { doJavaTest(configure) }
+
     private fun compareEditorsWithExpectedData() {
         //noinspection ConstantConditions
         val checkErrorsAfter = InTextDirectivesUtils.isDirectiveDefined(file!!.text, "// CHECK_ERRORS_AFTER")
@@ -671,7 +675,7 @@ class KotlinChangeSignatureTest : KotlinLightCodeInsightFixtureTestCase() {
 
     fun testParameterToReceiverImplicitReceivers() = doTest { receiverParameterInfo = newParameters[0] }
 
-    fun testJavaMethodOverridesReplaceParam() = doJavaTest {
+    fun testJavaMethodOverridesReplaceParam() = doJavaTestConflict {
         newReturnType = stringPsiType
         newParameters[0] = ParameterInfoImpl(-1, "x", PsiType.INT, "1")
     }