Normalize ply.source_roots at LSP didChangeConfiguration method

This is useful to specify source roots in relative path, if pyls
process might run in other than the root of the target workspace(s).

Author: flying-foozy

pyls/config/pyls_conf.py

@@ -70,6 +70,9 @@ def project_config(self, document_path):
 
         return settings
 
+    def lsp_config(self, config):
+        self.normalize(config, self._norm_root_path)
+
     def normalize(self, config, basedir):
         for config_path, inside_only in NORMALIZED_CONFIGS:
             paths = get_config_by_path(config, config_path)

test/test_workspace.py

@@ -143,6 +143,7 @@ def _make_paths_dir_relative(paths, dirname):
     'tox.ini',
     'service/foo/setup.cfg',
     'service/foo/tox.ini',
+    None,
 ])
 def test_source_roots_config(tmpdir, metafile):
     """Examine that source_roots config is intentionaly read in.
@@ -178,6 +179,12 @@ def test_source_roots_config(tmpdir, metafile):
         initializationOptions={}
     )
 
+    if not metafile:
+        # configured by client via LSP after pyls startup
+        pyls.m_workspace__did_change_configuration({
+            'pyls': {'source_roots': source_roots},
+        })
+
     # put new document under ROOT/service/foo
     test_uri = uris.from_fs_path(os.path.join(root_path, doc_root, 'hello/test.py'))
     pyls.workspace.put_document(test_uri, 'assert true')