Skip to content

Commit 669f4a6

Browse files
committed
Add .model suffix to model filenames
1 parent b7b4302 commit 669f4a6

File tree

2 files changed

+17
-13
lines changed
  • extensions/ql-vscode

2 files changed

+17
-13
lines changed

extensions/ql-vscode/src/data-extensions-editor/yaml.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,11 @@ export function createDataExtensionYamlsPerLibrary(
124124
const semverRegex =
125125
/[v=\s]*([0-9]+)\.([0-9]+)\.([0-9]+)(?:-?((?:[0-9]+|\d*[a-zA-Z-][a-zA-Z0-9-]*)(?:\.(?:[0-9]+|\d*[a-zA-Z-][a-zA-Z0-9-]*))*))?(?:\+([0-9A-Za-z-]+(?:\.[0-9A-Za-z-]+)*))?/;
126126

127-
export function createFilenameForLibrary(library: string, prefix = "models/") {
127+
export function createFilenameForLibrary(
128+
library: string,
129+
prefix = "models/",
130+
suffix = ".model",
131+
) {
128132
let libraryName = basename(library);
129133
const extension = extname(libraryName);
130134
libraryName = libraryName.slice(0, -extension.length);
@@ -153,7 +157,7 @@ export function createFilenameForLibrary(library: string, prefix = "models/") {
153157
// Remove any duplicate dots
154158
libraryName = libraryName.replaceAll(/\.{2,}/g, ".");
155159

156-
return `${prefix}${libraryName}.yml`;
160+
return `${prefix}${libraryName}${suffix}.yml`;
157161
}
158162

159163
export function loadDataExtensionYaml(

extensions/ql-vscode/test/unit-tests/data-extensions-editor/yaml.test.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ describe("createDataExtensionYamlsPerLibrary", () => {
292292
);
293293

294294
expect(yaml).toEqual({
295-
"models/sql2o.yml": `extensions:
295+
"models/sql2o.model.yml": `extensions:
296296
- addsTo:
297297
pack: codeql/java-all
298298
extensible: sourceModel
@@ -315,7 +315,7 @@ describe("createDataExtensionYamlsPerLibrary", () => {
315315
extensible: neutralModel
316316
data: []
317317
`,
318-
"models/spring-boot.yml": `extensions:
318+
"models/spring-boot.model.yml": `extensions:
319319
- addsTo:
320320
pack: codeql/java-all
321321
extensible: sourceModel
@@ -402,38 +402,38 @@ describe("loadDataExtensionYaml", () => {
402402

403403
describe("createFilenameForLibrary", () => {
404404
const testCases = [
405-
{ library: "sql2o.jar", filename: "models/sql2o.yml" },
405+
{ library: "sql2o.jar", filename: "models/sql2o.model.yml" },
406406
{
407407
library: "sql2o-1.6.0.jar",
408-
filename: "models/sql2o.yml",
408+
filename: "models/sql2o.model.yml",
409409
},
410410
{
411411
library: "spring-boot-3.0.2.jar",
412-
filename: "models/spring-boot.yml",
412+
filename: "models/spring-boot.model.yml",
413413
},
414414
{
415415
library: "spring-boot-v3.0.2.jar",
416-
filename: "models/spring-boot.yml",
416+
filename: "models/spring-boot.model.yml",
417417
},
418418
{
419419
library: "spring-boot-3.0.2-alpha1.jar",
420-
filename: "models/spring-boot.yml",
420+
filename: "models/spring-boot.model.yml",
421421
},
422422
{
423423
library: "spring-boot-3.0.2beta2.jar",
424-
filename: "models/spring-boot.yml",
424+
filename: "models/spring-boot.model.yml",
425425
},
426426
{
427427
library: "rt.jar",
428-
filename: "models/rt.yml",
428+
filename: "models/rt.model.yml",
429429
},
430430
{
431431
library: "System.Runtime.dll",
432-
filename: "models/system.runtime.yml",
432+
filename: "models/system.runtime.model.yml",
433433
},
434434
{
435435
library: "System.Runtime.1.5.0.dll",
436-
filename: "models/system.runtime.yml",
436+
filename: "models/system.runtime.model.yml",
437437
},
438438
];
439439

0 commit comments

Comments
 (0)