Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
remove rule that skips test
Browse files Browse the repository at this point in the history
We did a partial removal in an earlier commit and unintientionally
made the TestHarness skip its tests.

This commit removes the rule altogether and fixes the setUp/tearDown
calls by removing the annotation (annotation not needed), see
https://github.com/JetBrains/intellij-community/blob/master/platform/testFramework/src/com/intellij/testFramework/UsefulTestCase.java#L91
davissuber committed Oct 17, 2023

Verified

This commit was signed with the committer’s verified signature.
1 parent 10d30d5 commit a7ee27d
Showing 3 changed files with 0 additions and 67 deletions.
13 changes: 0 additions & 13 deletions intellij/src/test/kotlin/motif/intellij/TestHarness.kt
Original file line number Diff line number Diff line change
@@ -30,39 +30,26 @@ import kotlin.reflect.KClass
import motif.Scope
import motif.core.ResolvedGraph
import motif.errormessage.ErrorMessage
import motif.intellij.testing.IntelliJRule
import motif.viewmodel.TestRenderer
import org.junit.After
import org.junit.Before
import org.junit.Rule
import org.junit.Test
import org.junit.runner.RunWith

@RunWith(Parameterized::class)
class TestHarness : LightJavaCodeInsightFixtureTestCase() {

@get:Rule val rule = IntelliJRule()

@org.junit.runners.Parameterized.Parameter(0) lateinit var testDir: File

override fun getProjectDescriptor() = DefaultLightProjectDescriptor {
JavaAwareProjectJdkTableImpl.getInstanceEx().internalJdk
}

@Before
public override fun setUp() {
super.setUp()

addLibrary(Inject::class)
addLibrary(Scope::class)
addLibrary(Nullable::class)
}

@After
public override fun tearDown() {
super.tearDown()
}

private fun addLibrary(clazz: KClass<*>) {
val fileUri =
clazz
21 changes: 0 additions & 21 deletions intellij/testing/build.gradle

This file was deleted.

This file was deleted.

0 comments on commit a7ee27d

Please sign in to comment.