fix: don't search copybooks if no paths are specified

Author: ishche

clients/cobol-lsp-vscode-extension/src/services/Settings.ts

@@ -244,18 +244,6 @@ export class SettingsService {
     ];
     const wsFolders = SettingsUtils.getWorkspaceFoldersPath(true);
 
-    // if no configuration for local or remote copybook paths is provided
-    // use pattern for workspace folder and subfolders as a default value
-    if (
-      paths.length === 0 &&
-      this.getCopybookConfigValues(PATHS_DSN, documentUri, dialectType)
-        .length === 0 &&
-      this.getCopybookConfigValues(PATHS_USS, documentUri, dialectType)
-        .length === 0
-    ) {
-      paths.push("**");
-    }
-
     return SettingsService.prepareLocalSearchFolders(paths, wsFolders);
   }
 

clients/cobol-lsp-vscode-extension/src/test/suite/lsp.spec.copybooks.test.ts

@@ -173,38 +173,6 @@ suite("Integration Test Suite: Copybooks", function () {
     .slow(1000);
 
   suite("Default local copybooks paths configuration", () => {
-    suite("local or remote copybook paths not set", () => {
-      suiteSetup(async () => {
-        await helper.updateConfig("default.json");
-        await helper.activate();
-      });
-
-      test("Local copybooks are resolved from workspace subfolders if no configuration is provided", async () => {
-        const editor = await helper.showDocument("USERC1N1.cbl");
-
-        let diagnostics: vscode.Diagnostic[] = [];
-        await helper.waitFor(() => {
-          diagnostics = vscode.languages.getDiagnostics(editor.document.uri);
-
-          return (
-            diagnostics.length > 0 &&
-            diagnostics.some(
-              (d) => d.message === "Errors inside the copybook",
-            ) &&
-            diagnostics.filter(
-              (d) => d.message === "BOOK1N: Copybook not found",
-            ).length === 0
-          );
-        });
-
-        assert.strictEqual(
-          diagnostics.filter((d) => d.message === "BOOK1N: Copybook not found")
-            .length,
-          0,
-        );
-      });
-    });
-
     suite("local copybook path is configured", () => {
       suiteSetup(async () => {
         await helper.updateConfig("testing.json");