Skip to content

Commit 852c1ca

Browse files
authored
feat: enhance sv keyword support and add related tests (#69)
1 parent 468a529 commit 852c1ca

4 files changed

Lines changed: 20 additions & 4 deletions

File tree

examples/app.jac

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,4 +173,8 @@ test "mongodb user operations" {
173173
with MongoDbContainer("mongo:7.0") as mongo {
174174

175175
}
176-
}
176+
}
177+
178+
# Test: sv import keyword highlighting
179+
sv import from endpoints { AddTodo, GetTodos }
180+
sv import from auth { login, logout }

src/__tests__/inspectTokenScopes.test.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -895,4 +895,16 @@ describe('inspectTokenScopesHandler - Location Based Tests', () => {
895895
});
896896
});
897897

898+
describe('sv keyword highlighting (lines 179-180)', () => {
899+
test('sv keyword is storage modifier like cl (line 179)', () => {
900+
// sv import from endpoints { AddTodo, GetTodos }
901+
expectToken(result, 179, 1, 3, 'sv', ['source.jac', 'storage.modifier.declaration.jac']);
902+
});
903+
904+
test('sv keyword is storage modifier like cl (line 180)', () => {
905+
// sv import from auth { login, logout }
906+
expectToken(result, 180, 1, 3, 'sv', ['source.jac', 'storage.modifier.declaration.jac']);
907+
});
908+
});
909+
898910
});

syntaxes/jac.tmLanguage.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@
110110
{
111111
"name": "meta.semstring.jac",
112112
"begin": "(?x)\\b(sem)\\b(?=\\s*[[:alpha:]_]\\w*|\\s*\\n)",
113-
"end": ";|(?=^\\s*(?:cl|def|impl|with|sem|import|glob|let|enum|obj|class|node|edge|walker|test)\\b)",
113+
"end": ";|(?=^\\s*(?:cl|def|impl|with|sem|sv|import|glob|let|enum|obj|class|node|edge|walker|test)\\b)",
114114
"beginCaptures": {
115115
"1": {
116116
"name": "storage.type.semstring.jac"
@@ -948,7 +948,7 @@
948948
},
949949
{
950950
"name": "storage.modifier.declaration.jac",
951-
"match": "(?x)\n \\b(?<!\\.)(\n priv|protect|pub|static|override|let|abs|glob|global|nonlocal|cl\n )\\b\n"
951+
"match": "(?x)\n \\b(?<!\\.)(\n priv|protect|pub|static|override|let|abs|glob|global|nonlocal|cl|sv\n )\\b\n"
952952
},
953953
{
954954
"name": "storage.type.class.jac",

tsconfig.tsbuildinfo

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
{"root":["./src/constants.ts","./src/extension.ts","./src/utils.ts","./src/commands/index.ts","./src/commands/inspecttokenscopes.ts","./src/environment/manager.ts","./src/lsp/lsp_manager.ts","./src/test/platformhelper.ts","./src/test/runtest.ts","./src/test/suite/cleanup.integration.test.ts","./src/test/suite/environment.integration.test.ts","./src/test/suite/index.ts","./src/test/suite/lsp.integration.test.ts","./src/test/suite/testutils.ts","./src/utils/envdetection.ts","./src/utils/envversion.ts","./src/utils/jacrunner.ts","./src/visual_debugger/visdbg.ts","./src/webview/visualdebugger.ts"],"version":"5.9.3"}
1+
{"root":["./src/constants.ts","./src/extension.ts","./src/utils.ts","./src/commands/index.ts","./src/commands/inspectTokenScopes.ts","./src/environment/manager.ts","./src/lsp/lsp_manager.ts","./src/test/platformHelper.ts","./src/test/runTest.ts","./src/test/suite/commands.integration.test.ts","./src/test/suite/environment.integration.test.ts","./src/test/suite/index.ts","./src/test/suite/lsp.integration.test.ts","./src/test/suite/testUtils.ts","./src/utils/envDetection.ts","./src/utils/envVersion.ts","./src/utils/jacRunner.ts","./src/visual_debugger/visdbg.ts","./src/webview/visualDebugger.ts"],"version":"5.9.3"}

0 commit comments

Comments
 (0)