Skip to content

Commit

Permalink
Add missing problemApiSupportEnabled argument in unit tests
Browse files Browse the repository at this point in the history
Set the new constructor parameter of WorkspaceConfiguration introduced
with #1296 to false.
  • Loading branch information
oleosterhagen committed Jul 14, 2024
1 parent 06828cf commit aa35611
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class BuildConfigurationTest extends ProjectSynchronizationSpecification {
WorkspaceConfiguration orignalConfiguration = configurationManager.loadWorkspaceConfiguration()

when:
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, workspaceGradleUserHome, workspaceJavaHome, offlineMode, buildScansEnabled, autoSync, workspaceArguments, workspaceJvmArguments, showConsole, showExecutions, false))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, workspaceGradleUserHome, workspaceJavaHome, offlineMode, buildScansEnabled, autoSync, workspaceArguments, workspaceJvmArguments, showConsole, showExecutions, false, false))
BuildConfiguration configuration = createInheritingBuildConfiguration(projectDir)

then:
Expand Down Expand Up @@ -203,7 +203,7 @@ connection.gradle.distribution=MODIFIED_DISTRO"""
when:
configurationManager.saveBuildConfiguration(buildConfig)
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, workspaceGradleUserHome, workspaceJavaHome, offlineMode, buildScansEnabled, autoSync, workspaceArguments, workspaceJvmArguments, showConsole, showExecutions, false))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, workspaceGradleUserHome, workspaceJavaHome, offlineMode, buildScansEnabled, autoSync, workspaceArguments, workspaceJvmArguments, showConsole, showExecutions, false, false))
buildConfig = configurationManager.loadBuildConfiguration(projectDir)
then:
Expand Down Expand Up @@ -242,7 +242,7 @@ connection.gradle.distribution=MODIFIED_DISTRO"""
when:
configurationManager.saveBuildConfiguration(buildConfig)
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(GradleDistribution.fromBuild(), null, null, !buildScansEnabled, !offlineMode, !autoSync, [], [], false, false, false))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(GradleDistribution.fromBuild(), null, null, !buildScansEnabled, !offlineMode, !autoSync, [], [], false, false, false, false))
buildConfig = configurationManager.loadBuildConfiguration(projectDir)
then:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ class ProjectConfigurationTest extends ProjectSynchronizationSpecification {

when:
configurationManager.saveProjectConfiguration(projectConfig)
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, gradleUserHome, javaHome, offlineMode, buildScansEnabled, autoSync, arguments, jvmArguments, showConsole, showExecutions, false))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, gradleUserHome, javaHome, offlineMode, buildScansEnabled, autoSync, arguments, jvmArguments, showConsole, showExecutions, false, false))
projectConfig = configurationManager.loadProjectConfiguration(project)

then:
Expand Down Expand Up @@ -224,7 +224,7 @@ class ProjectConfigurationTest extends ProjectSynchronizationSpecification {

when:
configurationManager.saveProjectConfiguration(projectConfig)
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(GradleDistribution.fromBuild(), null, null, !buildScansEnabled, !offlineMode, !autoSync, [], [], false, false, false))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(GradleDistribution.fromBuild(), null, null, !buildScansEnabled, !offlineMode, !autoSync, [], [], false, false, false, false))
projectConfig = configurationManager.loadProjectConfiguration(project)

then:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ class WorkspaceConfigurationTest extends WorkspaceSpecification {
when:
File gradleUserHomeDir = dir(gradleUserHome)
File javaHomeDir = dir(javaHome)
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, gradleUserHomeDir, javaHomeDir, offlineMode, buildScansEnabled, autoSync, args, jvmArgs, showConsole, showExecutions, moduleSupportEnabled))
configurationManager.saveWorkspaceConfiguration(new WorkspaceConfiguration(distribution, gradleUserHomeDir, javaHomeDir, offlineMode, buildScansEnabled, autoSync, args, jvmArgs, showConsole, showExecutions, moduleSupportEnabled, false))
WorkspaceConfiguration updatedConfiguration = configurationManager.loadWorkspaceConfiguration()

then:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,7 @@ class SynchronizingBuildScriptUpdateListenerTest extends ProjectSynchronizationS
workspaceConfig.jvmArguments,
workspaceConfig.showConsoleView,
workspaceConfig.showExecutionsView,
false,
false)
configurationManager.saveWorkspaceConfiguration(workspaceConfig)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class AddBuildshipNatureHandlerTest extends WorkspaceSpecification {
def "Uses configuration from workspace settings"() {
setup:
WorkspaceConfiguration originalWorkspaceConfig = configurationManager.loadWorkspaceConfiguration()
WorkspaceConfiguration config = new WorkspaceConfiguration(GradleDistribution.forVersion("3.0"), dir('custom-gradle-home'), null, false, false, false, [], [], false, false, false)
WorkspaceConfiguration config = new WorkspaceConfiguration(GradleDistribution.forVersion("3.0"), dir('custom-gradle-home'), null, false, false, false, [], [], false, false, false, false)
configurationManager.saveWorkspaceConfiguration(config)

IProject project = EclipseProjects.newProject('add-buildship-nature')
Expand Down

0 comments on commit aa35611

Please sign in to comment.