Tests for workspace symbols

Author: wkillerud

extension/test/electron/symbols/document-symbols.test.js

@@ -28,7 +28,7 @@ async function findDocumentSymbols(documentUri) {
   return result;
 }
 
-test('gets CSS selectors', async () => {
+test('gets document symbols', async () => {
   const result = await findDocumentSymbols(stylesUri);
 
   assert.ok(

extension/test/electron/symbols/fixtures/other.scss

@@ -0,0 +1 @@
+$from-other: 'hello';

extension/test/electron/symbols/workspace-symbols.test.js

@@ -0,0 +1,47 @@
+const assert = require('node:assert');
+const path = require('node:path');
+const vscode = require('vscode');
+const { showFile, sleepCI } = require('../util');
+
+const stylesUri = vscode.Uri.file(
+  path.resolve(__dirname, 'fixtures', 'styles.sass')
+);
+
+before(async () => {
+  await showFile(stylesUri);
+  await sleepCI();
+});
+
+after(async () => {
+  await vscode.commands.executeCommand('workbench.action.closeAllEditors');
+});
+
+/**
+ * @param {string} [query='']
+ * @returns {Promise<import('vscode').SymbolInformation[]>}
+ */
+async function findWorkspaceSymbols(query = '') {
+  const result = await vscode.commands.executeCommand(
+    'vscode.executeWorkspaceSymbolProvider',
+    query
+  );
+  return result;
+}
+
+test('gets workspace symbols with empty query', async () => {
+  const result = await findWorkspaceSymbols();
+
+  assert.ok(result.find((s) => s.name === '.hello'));
+  assert.ok(result.find((s) => s.name === '$from-other'));
+});
+
+test('gets workspace symbols with query', async () => {
+  const result = await findWorkspaceSymbols('other');
+
+  assert.equal(
+    result.find((s) => s.name === '.hello'),
+    undefined
+  );
+
+  assert.ok(result.find((s) => s.name === '$from-other'));
+});

pkgs/sass_language_server/lib/src/language_server.dart

@@ -292,6 +292,10 @@ class LanguageServer {
           var query =
               (params.value as Map<String, Object?>)['query'] as String?;
 
+          if (initialScan != null) {
+            await initialScan;
+          }
+
           var result = _ls.findWorkspaceSymbols(query);
           return result;
         } on Exception catch (e) {