[Misc] Fix errors checking in QuickFixMultiModuleTest

This test was checking errors *before* applying action, rather than
*after*, like all other QuickFix-tests do. This commit moves
errors-check after performing quick-fix action, and changes testdata:

- for most tests errors are simply gone (as quickfix usually fixes that
error)
- also, for implementMembersInActualClassNoExpectMember we add
'DISABLE-ERROR', similarly to
implementMembersInImplClassNonImplInheritor. This is needed because
'Implement Members'-fix created 'TODO'-calls, which are not resolved in
those tests due to the lack of stdlib. Previously, there were no errors
in this test exactly because file was checked *before* applying quickfix
(i.e. when there were no 'TODO()' in file)
This commit is contained in:
Dmitry Savvinov
2019-06-07 13:20:15 +03:00
parent d458821565
commit de2944ff43
43 changed files with 6 additions and 46 deletions
@@ -1,4 +1,3 @@
// "Implement as constructor parameters" "true"
// ERROR: Class 'End' is not abstract and does not implement abstract member public abstract val a: Int defined in IFoo
actual class <caret>End : IFoo
@@ -1,4 +1,3 @@
// "Implement as constructor parameters" "true"
// ERROR: Class 'End' is not abstract and does not implement abstract member public abstract val a: Int defined in IFoo
actual class End(override val a: Int) : IFoo
@@ -1,5 +1,4 @@
// "Remove 'actual' modifier" "true"
// ERROR: Actual function 'first' has no corresponding expected declaration
actual interface ExpInterface {
actual fun first()
@@ -1,5 +1,4 @@
// "Remove 'actual' modifier" "true"
// ERROR: Actual function 'first' has no corresponding expected declaration
actual interface ExpInterface {
actual fun first()
@@ -1,4 +1,3 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
class <caret>Foo
@@ -1,4 +1,3 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
actual class Foo
@@ -1,5 +1,4 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
actual class Foo {
fun <caret>foo() {}
@@ -1,5 +1,4 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
actual class Foo {
actual fun foo() {}
@@ -1,4 +1,3 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
fun <caret>foo() {}
@@ -1,4 +1,3 @@
// "Add 'actual' modifier" "true"
// ERROR: Declaration must be marked with 'actual'
actual fun foo() {}
@@ -1,5 +1,4 @@
// "Create class 'ClassG'" "true"
// ERROR: Unresolved reference: ClassG
package a
@@ -1,5 +1,4 @@
// "Create class 'ClassG'" "true"
// ERROR: Unresolved reference: ClassG
package a
@@ -1,5 +1,4 @@
// "Create member function 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo
@@ -1,5 +1,4 @@
// "Create member function 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo {
fun foo(s: String, i: Int)
@@ -1,5 +1,4 @@
// "Create member property 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo
@@ -1,5 +1,4 @@
// "Create member property 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo {
val foo: Int
@@ -1,5 +1,4 @@
// "Create member property 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo
@@ -1,5 +1,4 @@
// "Create member property 'Foo.foo'" "true"
// ERROR: Unresolved reference: foo
expect class Foo {
var foo: Int
@@ -1,5 +1,5 @@
// "Implement members" "true"
// ERROR: Class 'ExpImpl' is not abstract and does not implement abstract member public abstract actual fun first(): Unit defined in ExpInterface
// DISABLE-ERRORS
actual interface ExpInterface {
actual fun first()
@@ -1,5 +1,5 @@
// "Implement members" "true"
// ERROR: Class 'ExpImpl' is not abstract and does not implement abstract member public abstract actual fun first(): Unit defined in ExpInterface
// DISABLE-ERRORS
actual interface ExpInterface {
actual fun first()
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
fun use(f: <caret>Foo) {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
import foo.Foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
fun use(f: <caret>Foo) {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
import foo.Foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
fun use(f: <caret>Foo) {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
import foo.Foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
fun use(f: <caret>Foo) {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
import foo.Foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: foo
package bar
fun use() {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: foo
package bar
import foo.foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
fun use(f: <caret>Foo) {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: Foo
package bar
import foo.Foo
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: foo
package bar
fun use() {
@@ -1,5 +1,4 @@
// "Import" "true"
// ERROR: Unresolved reference: foo
package bar
import foo.foo
@@ -1,4 +1,3 @@
// "Add 'inline' modifier" "true"
// ERROR: Only type parameters of inline functions can be reified
expect fun <reif<caret>ied T> inlineFun(t: T)
@@ -1,4 +1,3 @@
// "Add 'inline' modifier" "true"
// ERROR: Only type parameters of inline functions can be reified
expect inline fun <reified T> inlineFun(t: T)
@@ -1,5 +1,4 @@
// "Make 'getInternal' internal" "true"
// ERROR: 'public' function exposes its 'internal' return type Internal
internal expect class Internal
@@ -1,5 +1,4 @@
// "Make 'getInternal' internal" "true"
// ERROR: 'public' function exposes its 'internal' return type Internal
internal expect class Internal
@@ -1,5 +1,4 @@
// "Make 'One' open" "true"
// ERROR: This type is final, so it cannot be inherited from
actual class One
class Two : <caret>One()
@@ -1,5 +1,4 @@
// "Make 'One' open" "true"
// ERROR: This type is final, so it cannot be inherited from
actual open class One
class Two : <caret>One()
@@ -1,5 +1,4 @@
// "Make 'One' open" "true"
// ERROR: This type is final, so it cannot be inherited from
expect class One
expect class Two : <caret>One
@@ -1,5 +1,4 @@
// "Make 'One' open" "true"
// ERROR: This type is final, so it cannot be inherited from
expect open class One
expect class Two : <caret>One