[rider] Reduce deprecated usages of APIs, base classes and features in Rider tests

GitOrigin-RevId: 9bee48ca3ce2fdd5fa6b43ea3993449f6df80591

Author: Konstantin343

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/Extensions.kt

@@ -39,8 +39,7 @@ fun withSetting(project: Project, setting: String, enterValue: String, exitValue
   }
 }
 
-context(SolutionApiFacade)
-fun withDisabledOutOfProcessTypeProviders(function: () -> Unit) {
+fun SolutionApiFacade.withDisabledOutOfProcessTypeProviders(function: () -> Unit) {
   withSetting(
     project,
     "FSharp/FSharpOptions/FSharpExperimentalFeatures/OutOfProcessTypeProviders/@EntryValue",
@@ -51,8 +50,7 @@ fun withDisabledOutOfProcessTypeProviders(function: () -> Unit) {
   }
 }
 
-context(SolutionApiFacade)
-fun withNonFSharpProjectReferences(function: () -> Unit) {
+fun SolutionApiFacade.withNonFSharpProjectReferences(function: () -> Unit) {
   withSetting(project, "FSharp/FSharpOptions/NonFSharpProjectInMemoryReferences/@EntryValue", "true", "false") {
     project.fcsHost.updateAssemblyReaderSettings.sync(Unit)
     function()

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/FileSystemShimTest.kt

@@ -5,23 +5,22 @@ import com.jetbrains.rdclient.util.idea.waitAndPump
 import com.jetbrains.rider.plugins.fsharp.test.fcsHost
 import com.jetbrains.rider.test.annotations.TestEnvironment
 import com.jetbrains.rider.test.asserts.shouldNotBeNull
-import com.jetbrains.rider.test.base.BaseTestWithSolution
 import com.jetbrains.rider.test.env.enums.SdkVersion
 import com.jetbrains.rider.test.scriptingApi.changeFileContent
 import com.jetbrains.rd.platform.util.lifetime
 import com.jetbrains.rider.test.annotations.Mute
+import com.jetbrains.rider.test.annotations.Solution
+import com.jetbrains.rider.test.base.PerTestSolutionTestBase
 import com.jetbrains.rider.test.enums.PlatformType
 import org.testng.annotations.Test
 import java.io.File
 import java.time.Duration
 
-@Test
+@Solution("CoreConsoleApp")
 @TestEnvironment(sdkVersion = SdkVersion.LATEST_STABLE)
-class FileSystemShimTest : BaseTestWithSolution() {
-  override val testSolution: String = "CoreConsoleApp"
-
+class FileSystemShimTest : PerTestSolutionTestBase() {
   @Test
-  @Mute("RIDER-111885", platforms= [PlatformType.LINUX_ALL])
+  @Mute("RIDER-111885", platforms = [PlatformType.LINUX_ALL])
   fun externalFileChange() {
     val file = activeSolutionDirectory.resolve("Program.fs")
     val stampBefore = getTimestamp(file)

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/markup/injections/InjectionEscapersTest.kt

@@ -9,22 +9,22 @@ import com.intellij.openapi.fileEditor.ex.FileEditorManagerEx
 import com.intellij.psi.impl.source.tree.injected.InjectedLanguageUtil
 import com.jetbrains.rider.editors.getPsiFile
 import com.jetbrains.rider.test.annotations.Mute
+import com.jetbrains.rider.test.annotations.Solution
 import com.jetbrains.rider.test.annotations.TestEnvironment
-import com.jetbrains.rider.test.base.BaseTestWithSolution
+import com.jetbrains.rider.test.base.PerTestSolutionTestBase
 import com.jetbrains.rider.test.enums.PlatformType
 import com.jetbrains.rider.test.env.enums.SdkVersion
 import com.jetbrains.rider.test.framework.executeWithGold
 import com.jetbrains.rider.test.scriptingApi.getHighlighters
 import com.jetbrains.rider.test.scriptingApi.typeFromOffset
-import com.jetbrains.rider.test.scriptingApi.withOpenedEditor
 import com.jetbrains.rider.test.scriptingApi.waitForDaemon
+import com.jetbrains.rider.test.scriptingApi.withOpenedEditor
 import org.testng.annotations.Test
 
 @Test
+@Solution("CoreConsoleApp")
 @TestEnvironment(sdkVersion = SdkVersion.DOT_NET_6)
-class InjectionEscapersTest : BaseTestWithSolution() {
-  override val testSolution: String = "CoreConsoleApp"
-
+class InjectionEscapersTest : PerTestSolutionTestBase() {
   private fun doTest(action: (EditorImpl, EditorImpl) -> Unit) {
     withOpenedEditor("Program.fs", "Program.fs") {
       waitForDaemon()

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/projectModel/FcsModuleReaderTest.kt

@@ -70,7 +70,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
     expectedReferencedProjects: List<String>
   ) {
     val project = project
-    withOpenedEditor(project, "FSharpProject/Library.fs") {
+    withOpenedEditor("FSharpProject/Library.fs") {
       waitForNextDaemon()
       assert(markupAdapter.hasErrors == hasErrors)
       assertFcsStampAndReferencedProjectNames(this, expectedReferencedProjects)
@@ -150,7 +150,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
 
         openFsFileDumpModuleReader(it, "2. Open F# file", false, listOf("CSharpProject"))
 
-        withOpenedEditor(project, "CSharpProject/Class1.cs") {
+        withOpenedEditor("CSharpProject/Class1.cs") {
           typeFromOffset(" ", 75)
           waitForNextDaemon()
         }
@@ -170,7 +170,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
 
         openFsFileDumpModuleReader(it, "8. Open F# file", false, listOf("CSharpProject"))
 
-        withOpenedEditor(project, "CSharpProject/Class1.cs") {
+        withOpenedEditor("CSharpProject/Class1.cs") {
           typeFromOffset(" ", 75)
         }
 
@@ -204,7 +204,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
 
         openFsFileDumpModuleReader(it, "2. Open F# file", false, listOf("CSharpProject"))
 
-        withOpenedEditor(project, "CSharpProject/Class1.cs") {
+        withOpenedEditor("CSharpProject/Class1.cs") {
           typeFromOffset(" ", 129)
         }
 
@@ -244,7 +244,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
   fun testGotoUsagesFromCSharp() {
     withNonFSharpProjectReferences {
       assertAllProjectsWereLoaded(project)
-      withOpenedEditor(project, "CSharpProject/Class1.cs", "Class1.cs") {
+      withOpenedEditor("CSharpProject/Class1.cs", "Class1.cs") {
         waitForNextDaemon()
         callAction(IdeActions.ACTION_GOTO_DECLARATION)
         waitForEditorSwitch("Library.fs")
@@ -257,7 +257,7 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
   fun testGotoUsagesFromCSharpChangeCSharp() {
     withNonFSharpProjectReferences {
       assertAllProjectsWereLoaded(project)
-      withOpenedEditor(project, "CSharpProject/Class1.cs", "Class1.cs") {
+      withOpenedEditor("CSharpProject/Class1.cs", "Class1.cs") {
         typeWithLatency("1")
         waitForNextDaemon()
         callAction(IdeActions.ACTION_GOTO_DECLARATION)
@@ -272,13 +272,13 @@ class FcsModuleReaderTest : ProjectModelBaseTest() {
     withNonFSharpProjectReferences {
       assertAllProjectsWereLoaded(project)
 
-      withOpenedEditor(project, "FSharpProject/Library.fs") {
+      withOpenedEditor("FSharpProject/Library.fs") {
         waitForNextDaemon()
       }
 
       waitForDaemonCloseAllOpenEditors(project)
 
-      withOpenedEditor(project, "CSharpProject/Class1.cs", "Class1.cs") {
+      withOpenedEditor("CSharpProject/Class1.cs", "Class1.cs") {
         typeWithLatency("1")
         waitForNextDaemon()
         callAction(IdeActions.ACTION_GOTO_DECLARATION)

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/projectModel/FcsProjectProviderTest.kt

@@ -3,7 +3,6 @@ package com.jetbrains.rider.plugins.fsharp.test.cases.projectModel
 import com.intellij.openapi.editor.impl.EditorImpl
 import com.intellij.platform.backend.workspace.WorkspaceModel
 import com.intellij.testFramework.ProjectViewTestUtil
-import com.jetbrains.rider.test.scriptingApi.waitForDaemon
 import com.jetbrains.rider.daemon.util.hasErrors
 import com.jetbrains.rider.editors.getProjectModelId
 import com.jetbrains.rider.plugins.fsharp.test.fcsHost
@@ -14,7 +13,7 @@ import com.jetbrains.rider.projectView.workspace.getProjectModelEntity
 import com.jetbrains.rider.test.annotations.Mute
 import com.jetbrains.rider.test.annotations.Solution
 import com.jetbrains.rider.test.annotations.TestEnvironment
-import com.jetbrains.rider.test.base.BaseTestWithSolution
+import com.jetbrains.rider.test.base.PerTestSolutionTestBase
 import com.jetbrains.rider.test.env.enums.SdkVersion
 import com.jetbrains.rider.test.framework.assertAllProjectsWereLoaded
 import com.jetbrains.rider.test.scriptingApi.*
@@ -23,9 +22,8 @@ import org.testng.annotations.BeforeMethod
 import org.testng.annotations.Test
 import kotlin.test.assertEquals
 
-@Test
 @TestEnvironment(sdkVersion = SdkVersion.LATEST_STABLE)
-class FcsProjectProviderTest : BaseTestWithSolution() {
+class FcsProjectProviderTest : PerTestSolutionTestBase() {
   override val traceCategories
     get() = super.traceCategories.plus("JetBrains.ReSharper.Plugins.FSharp.Checker.FcsProjectProvider")
 
@@ -54,7 +52,7 @@ class FcsProjectProviderTest : BaseTestWithSolution() {
     hasErrors: Boolean,
     expectedReferencedProjects: List<String>
   ) {
-    withOpenedEditor(project, fileName) {
+    withOpenedEditor(fileName) {
       waitForDaemon()
       project!!.fcsHost.dumpFcsModuleReader.sync(Unit)
       assertEquals(hasErrors, markupAdapter.hasErrors)
@@ -63,6 +61,7 @@ class FcsProjectProviderTest : BaseTestWithSolution() {
   }
 
   @Solution("ProjectReferencesFSharp")
+  @Test
   fun projectReferencesFSharp() {
     assertAllProjectsWereLoaded(project)
     assertHasErrorsAndProjectStampAndReferences("ReferenceFrom/Library.fs", true, emptyList())
@@ -87,6 +86,7 @@ class FcsProjectProviderTest : BaseTestWithSolution() {
   }
 
   @Mute("Broken after ProjectModelMonitor refactoring")
+  @Test
   @Solution("ProjectReferencesCSharp")
   fun projectReferencesCSharp() {
     withNonFSharpProjectReferences {
@@ -120,8 +120,9 @@ class FcsProjectProviderTest : BaseTestWithSolution() {
     }
   }
 
-  @Solution("ProjectReferencesCSharp")
   @Mute("RIDER-100270 Need to somehow set setting before solution load")
+  @Test
+  @Solution("ProjectReferencesCSharp")
   fun projectReferencesCSharpNoModuleReader() {
     assertAllProjectsWereLoaded(project)
     assertHasErrorsAndProjectStampAndReferences("FSharpProject/Library.fs", true, emptyList())

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/projectModel/ReferencesOrderTest.kt

@@ -1,20 +1,22 @@
 package com.jetbrains.rider.plugins.fsharp.test.cases.projectModel
 
 import com.jetbrains.rider.plugins.fsharp.test.fcsHost
+import com.jetbrains.rider.test.OpenSolutionParams
+import com.jetbrains.rider.test.annotations.Solution
 import com.jetbrains.rider.test.annotations.TestEnvironment
-import com.jetbrains.rider.test.base.BaseTestWithSolution
+import com.jetbrains.rider.test.base.PerTestSolutionTestBase
 import com.jetbrains.rider.test.env.enums.SdkVersion
 import org.testng.annotations.Test
 
-@Test
+@Solution("ReferencesOrder")
 @TestEnvironment(sdkVersion = SdkVersion.DOT_NET_6)
-class ReferencesOrder : BaseTestWithSolution() {
-  override val testSolution = "ReferencesOrder"
-
-  override val waitForCaches = true
-  override val restoreNuGetPackages = true
+class ReferencesOrder : PerTestSolutionTestBase() {
+  override fun modifyOpenSolutionParams(params: OpenSolutionParams) {
+    params.waitForCaches = true
+    params.restoreNuGetPackages = true
+  }
 
-  @Test()
+  @Test
   fun testReferencesOrder() {
     val references = project.fcsHost.dumpSingleProjectLocalReferences.sync(Unit)
     assert(references == listOf("Library1.dll", "Library2.dll"))

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/typeProviders/BaseTypeProvidersTest.kt

@@ -4,10 +4,14 @@ import com.jetbrains.rd.platform.diagnostics.LogTraceScenario
 import com.jetbrains.rider.plugins.fsharp.logs.FSharpLogTraceScenarios
 import com.jetbrains.rider.plugins.fsharp.rdFSharpModel
 import com.jetbrains.rider.projectView.solution
-import com.jetbrains.rider.test.base.BaseTestWithSolution
+import com.jetbrains.rider.test.OpenSolutionParams
+import com.jetbrains.rider.test.base.PerTestSolutionTestBase
+
+abstract class BaseTypeProvidersTest : PerTestSolutionTestBase() {
+  override fun modifyOpenSolutionParams(params: OpenSolutionParams) {
+    params.restoreNuGetPackages = true
+  }
 
-abstract class BaseTypeProvidersTest : BaseTestWithSolution() {
-  override val restoreNuGetPackages = true
   override val traceScenarios: Set<LogTraceScenario>
     get() = super.traceScenarios + FSharpLogTraceScenarios.FSharpTypeProviders
 

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/typeProviders/GenerativeTypeProvidersTest.kt

@@ -4,6 +4,7 @@ import com.jetbrains.rider.daemon.util.hasErrors
 import com.jetbrains.rider.plugins.fsharp.test.dumpTypeProviders
 import com.jetbrains.rider.projectView.solutionDirectoryPath
 import com.jetbrains.rider.test.annotations.Mute
+import com.jetbrains.rider.test.annotations.Solution
 import com.jetbrains.rider.test.annotations.TestEnvironment
 import com.jetbrains.rider.test.asserts.shouldBeFalse
 import com.jetbrains.rider.test.asserts.shouldBeTrue
@@ -17,18 +18,16 @@ import com.jetbrains.rider.test.scriptingApi.waitForNextDaemon
 import org.testng.annotations.Test
 import java.time.Duration
 
-@Test
+@Solution("TypeProviderLibrary")
 @TestEnvironment(sdkVersion = SdkVersion.DOT_NET_CORE_3_1, buildTool = BuildTool.FULL)
 class GenerativeTypeProvidersTest : BaseTypeProvidersTest() {
-  override val testSolution = "TypeProviderLibrary"
-
   @Test
   @Mute("RIDER-111885", platforms = [PlatformType.LINUX_ALL, PlatformType.MAC_OS_ALL])
   fun `generative type providers cross-project analysis`() {
     val generativeProviderProjectPath =
       "${project.solutionDirectoryPath}/GenerativeTypeProvider/GenerativeTypeProvider.fsproj"
 
-    withOpenedEditor(project, "GenerativeTypeLibrary/Library.fs") {
+    withOpenedEditor("GenerativeTypeLibrary/Library.fs") {
       waitForDaemon()
       markupAdapter.hasErrors.shouldBeTrue()
 
@@ -41,7 +40,7 @@ class GenerativeTypeProvidersTest : BaseTypeProvidersTest() {
     unloadProject(arrayOf("TypeProviderLibrary", "GenerativeTypeProvider"))
     reloadProject(arrayOf("TypeProviderLibrary", "GenerativeTypeProvider"))
 
-    withOpenedEditor(project, "GenerativeTypeLibrary/Library.fs") {
+    withOpenedEditor("GenerativeTypeLibrary/Library.fs") {
       waitForDaemon()
       markupAdapter.hasErrors.shouldBeFalse()
     }
@@ -51,7 +50,7 @@ class GenerativeTypeProvidersTest : BaseTypeProvidersTest() {
   @Mute("RIDER-111883", platforms = [PlatformType.LINUX_ALL, PlatformType.MAC_OS_ALL])
   fun `change abbreviation`() {
     executeWithGold(testGoldFile) {
-      withOpenedEditor(project, "GenerativeTypeProvider/Library.fs") {
+      withOpenedEditor("GenerativeTypeProvider/Library.fs") {
         waitForDaemon()
 
         it.println("Before:\n")

rider-fsharp/src/test/kotlin/com/jetbrains/rider/plugins/fsharp/test/cases/typeProviders/TypeProvidersCSharpTest.kt

@@ -12,13 +12,11 @@ import com.jetbrains.rider.test.scriptingApi.waitForNextDaemon
 import org.testng.annotations.Test
 import java.io.File
 
-@Test
+@Solution("YamlProviderCSharp")
 class TypeProvidersCSharpTest : BaseTypeProvidersTest() {
-  override val testSolution = "YamlProviderCSharp"
-
   @Test
   fun resolveTest() {
-    withOpenedEditor(project, "CSharpLibrary/CSharpLibrary.cs") {
+    withOpenedEditor("CSharpLibrary/CSharpLibrary.cs") {
       waitForDaemon()
       executeWithGold(testGoldFile) {
         dumpSevereHighlighters(it)
@@ -28,7 +26,7 @@ class TypeProvidersCSharpTest : BaseTypeProvidersTest() {
     unloadAllProjects()
     reloadAllProjects(project)
 
-    withOpenedEditor(project, "CSharpLibrary/CSharpLibrary.cs") {
+    withOpenedEditor("CSharpLibrary/CSharpLibrary.cs") {
       waitForDaemon()
       executeWithGold(testGoldFile) {
         dumpSevereHighlighters(it)
@@ -40,18 +38,18 @@ class TypeProvidersCSharpTest : BaseTypeProvidersTest() {
   @TestEnvironment(sdkVersion = SdkVersion.DOT_NET_6)
   @Solution("SwaggerProviderCSharp")
   fun changeStaticArg() {
-    withOpenedEditor(project, "CSharpLibrary/CSharpLibrary.cs", "CSharpLibrary.cs") {
+    withOpenedEditor("CSharpLibrary/CSharpLibrary.cs", "CSharpLibrary.cs") {
       waitForDaemon()
       markupAdapter.hasErrors.shouldBeFalse()
     }
 
-    withOpenedEditor(project, "SwaggerProviderLibrary/Literals.fs") {
+    withOpenedEditor("SwaggerProviderLibrary/Literals.fs") {
       waitForDaemon()
       // change schema path from "specification.json" to "specification1.json"
       typeFromOffset("1", 86)
     }
 
-    withOpenedEditor(project, "CSharpLibrary/CSharpLibrary.cs", "CSharpLibrary.cs") {
+    withOpenedEditor("CSharpLibrary/CSharpLibrary.cs", "CSharpLibrary.cs") {
       waitForNextDaemon()
       executeWithGold(File(testGoldFile.path + "_before")) {
         dumpSevereHighlighters(it)