From 6bfbec8c8811988f54b4f387bded04ab43441c3a Mon Sep 17 00:00:00 2001 From: Natalia Ukhorskaya Date: Wed, 16 Dec 2015 15:45:19 +0300 Subject: [PATCH] ConvertAssertToIf may produce code with error // TODO: we could generate anonymous function instead of lambda expression when there is a return --- .../convertAssertToIf/lambdaMessageOutsideParentheses.kt | 2 ++ .../convertAssertToIf/lambdaMessageOutsideParentheses.kt.after | 2 ++ .../lambdaMultiStatementMessageOutsideParentheses.kt | 2 ++ .../lambdaMultiStatementMessageOutsideParentheses.kt.after | 2 ++ 4 files changed, 8 insertions(+) diff --git a/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt b/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt index edac359413b..488d43c09f1 100644 --- a/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt +++ b/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt @@ -1,4 +1,6 @@ // WITH_RUNTIME +// SKIP_ERRORS_AFTER +// TODO: 'return' is not allowed here fun foo() { assert(true) { return diff --git a/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt.after b/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt.after index 861c3701762..80f6055d1cc 100644 --- a/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt.after +++ b/idea/testData/intentions/convertAssertToIf/lambdaMessageOutsideParentheses.kt.after @@ -1,4 +1,6 @@ // WITH_RUNTIME +// SKIP_ERRORS_AFTER +// TODO: 'return' is not allowed here fun foo() { if (!true) { throw AssertionError({ diff --git a/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt b/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt index dfddb0b1a8e..9d2cf2ca7fe 100644 --- a/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt +++ b/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt @@ -1,4 +1,6 @@ // WITH_RUNTIME +// SKIP_ERRORS_AFTER +// TODO: 'return' is not allowed here fun foo() { assert(true) { if (false) return diff --git a/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt.after b/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt.after index 33c4a0667a6..8a8ddcc8527 100644 --- a/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt.after +++ b/idea/testData/intentions/convertAssertToIf/lambdaMultiStatementMessageOutsideParentheses.kt.after @@ -1,4 +1,6 @@ // WITH_RUNTIME +// SKIP_ERRORS_AFTER +// TODO: 'return' is not allowed here fun foo() { if (!true) { throw AssertionError({