Skip to content

Commit 15c2a86

Browse files
committed
Use Argument[this] instead of Argument[-1]
1 parent e14b4c3 commit 15c2a86

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

extensions/ql-vscode/src/data-extensions-editor/auto-model.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ export function createAutoModelRequest(
5353
argumentIndex < numberOfArguments;
5454
argumentIndex++
5555
) {
56+
const argumentInput: string =
57+
argumentIndex === -1 ? "Argument[this]" : `Argument[${argumentIndex}]`;
5658
const method: Method = {
5759
package: externalApiUsage.packageName,
5860
type: externalApiUsage.typeName,
@@ -63,7 +65,7 @@ export function createAutoModelRequest(
6365
? undefined
6466
: toMethodClassification(modeledMethod),
6567
usages: usagesForMethod.slice(0, 6), // At most 6 usages per argument
66-
input: `Argument[${argumentIndex}]`,
68+
input: argumentInput,
6769
};
6870

6971
// A method that is supported is modeled outside of the model file, so it is not a candidate.

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

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ describe("createAutoModelRequest", () => {
266266
explanation: "",
267267
},
268268
usages: usages["org.sql2o.Sql2o#open()"],
269-
input: "Argument[-1]",
269+
input: "Argument[this]",
270270
},
271271
{
272272
package: "org.sql2o",
@@ -279,7 +279,7 @@ describe("createAutoModelRequest", () => {
279279
explanation: "",
280280
},
281281
usages: usages["org.sql2o.Sql2o#Sql2o(String)"],
282-
input: "Argument[-1]",
282+
input: "Argument[this]",
283283
},
284284
{
285285
package: "org.sql2o",
@@ -302,7 +302,7 @@ describe("createAutoModelRequest", () => {
302302
name: "createQuery",
303303
signature: "(String)",
304304
usages: usages["org.sql2o.Connection#createQuery(String)"],
305-
input: "Argument[-1]",
305+
input: "Argument[this]",
306306
classification: undefined,
307307
},
308308
{
@@ -320,7 +320,7 @@ describe("createAutoModelRequest", () => {
320320
name: "executeScalar",
321321
signature: "(Class)",
322322
usages: usages["org.sql2o.Query#executeScalar(Class)"],
323-
input: "Argument[-1]",
323+
input: "Argument[this]",
324324
classification: undefined,
325325
},
326326
{
@@ -341,7 +341,7 @@ describe("createAutoModelRequest", () => {
341341
usages[
342342
"org.springframework.boot.SpringApplication#run(Class,String[])"
343343
],
344-
input: "Argument[-1]",
344+
input: "Argument[this]",
345345
classification: undefined,
346346
},
347347
{
@@ -374,7 +374,7 @@ describe("createAutoModelRequest", () => {
374374
name: "println",
375375
signature: "(String)",
376376
usages: usages["java.io.PrintStream#println(String)"],
377-
input: "Argument[-1]",
377+
input: "Argument[this]",
378378
classification: undefined,
379379
},
380380
{
@@ -392,7 +392,7 @@ describe("createAutoModelRequest", () => {
392392
name: "Sql2o",
393393
signature: "(String,String,String)",
394394
usages: usages["org.sql2o.Sql2o#Sql2o(String,String,String)"],
395-
input: "Argument[-1]",
395+
input: "Argument[this]",
396396
classification: undefined,
397397
},
398398
{

0 commit comments

Comments
 (0)