Drop java OpenProject subcommand

Author: bstaletic

ycmd/completers/java/java_completer.py

@@ -358,9 +358,6 @@ def GetCustomSubcommands( self ):
       'OrganizeImports': (
         lambda self, request_data, args: self.OrganizeImports( request_data )
       ),
-      'OpenProject': (
-        lambda self, request_data, args: self._OpenProject( request_data, args )
-      ),
       'WipeWorkspace': (
         lambda self, request_data, args: self._WipeWorkspace( request_data,
                                                               args )
@@ -418,25 +415,6 @@ def _WipeWorkspace( self, request_data, args ):
                          wipe_config = with_config )
 
 
-  def _OpenProject( self, request_data, args ):
-    if len( args ) != 1:
-      raise ValueError( "Usage: OpenProject <project directory>" )
-
-    project_directory = args[ 0 ]
-
-    # If the dir is not absolute, calculate it relative to the working dir of
-    # the client (if supplied).
-    if not os.path.isabs( project_directory ):
-      if 'working_dir' not in request_data:
-        raise ValueError( "Project directory must be absolute" )
-
-      project_directory = os.path.normpath( os.path.join(
-        request_data[ 'working_dir' ],
-        project_directory ) )
-
-    self._RestartServer( request_data, project_directory = project_directory )
-
-
   def _Reset( self ):
     if self._workspace_path and self._use_clean_workspace:
       try:

ycmd/tests/java/server_management_test.py

@@ -18,7 +18,6 @@
 import functools
 import os
 import psutil
-import requests
 import time
 
 from unittest.mock import patch
@@ -36,12 +35,10 @@
 from ycmd.tests.java import ( PathToTestFile,
                               isolated_app,
                               IsolatedYcmd,
-                              SharedYcmd,
                               StartJavaCompleterServerInDirectory,
                               StartJavaCompleterServerWithFile )
 from ycmd.tests.test_utils import ( BuildRequest,
                                     CompleterProjectDirectoryMatcher,
-                                    ErrorMatcher,
                                     MockProcessTerminationTimingOut,
                                     TemporaryTestDir,
                                     WaitUntilCompleterServerReady )
@@ -299,45 +296,6 @@ def test_ServerManagement_ProjectDetection_MavenParent_Submodule( self, app ):
                  CompleterProjectDirectoryMatcher( project ) )
 
 
-  @SharedYcmd
-  def test_ServerManagement_OpenProject_RelativePathNoWD( self, app ):
-    response = app.post_json(
-      '/run_completer_command',
-      BuildRequest(
-        filetype = 'java',
-        command_arguments = [
-          'OpenProject',
-          os.path.join( '..', 'simple_maven_project' ),
-        ],
-      ),
-      expect_errors = True,
-    )
-    assert_that( response.status_code,
-                 equal_to( requests.codes.internal_server_error ) )
-    assert_that( response.json,
-                 ErrorMatcher( ValueError,
-                               'Project directory must be absolute' ) )
-
-
-  @SharedYcmd
-  def test_ServerManagement_OpenProject_RelativePathNoPath( self, app ):
-    response = app.post_json(
-      '/run_completer_command',
-      BuildRequest(
-        filetype = 'java',
-        command_arguments = [
-          'OpenProject',
-        ],
-      ),
-      expect_errors = True,
-    )
-    assert_that( response.status_code,
-                 equal_to( requests.codes.internal_server_error ) )
-    assert_that( response.json,
-                 ErrorMatcher( ValueError,
-                               'Usage: OpenProject <project directory>' ) )
-
-
   def test_ServerManagement_ProjectDetection_NoParent( self ):
     with TemporaryTestDir() as tmp_dir:
       with isolated_app() as app:

ycmd/tests/java/subcommands_test.py

@@ -178,7 +178,6 @@ def test_Subcommands_DefinedSubcommands( self, app ):
                    'GoToReferences',
                    'GoToType',
                    'GoToSymbol',
-                   'OpenProject',
                    'OrganizeImports',
                    'RefactorRename',
                    'RestartServer',