Support named arguments for routes in CodeLens

[zizuib]

It seems like CodeLens expects the url in the client calls to be positional arguments and ignore them if they are named, would it be possible to support both positional arguments and named

In my test file i have defined a test like this:

def test_connection_success(client):
    response = client.get(url="/v1/files/test")
    assert response.status_code == 200
    assert response.json() == {"message": "Connection Successful!"}

i do not get the link above the client.get(url="/v1/files/test")

however, if i remove the named parameter url and just provide the url directly like this:

def test_connection_success(client):
    response = client.get("/v1/files/test")
    assert response.status_code == 200
    assert response.json() == {"message": "Connection Successful!"}

The link does appear.

this issue seems to be related to this code block in src/vscode/testCodeLensProvider.ts

const args = argumentsNode.namedChildren.filter(
  (child) => child.type !== "comment",
)

if (args.length === 0) continue

const pathArg = args[0]
const path = extractPathFromNode(pathArg)

while i see in the documentation here:
https://fastapi.tiangolo.com/tutorial/testing/#extended-fastapi-app-file

that positional arguments is used and not named i think it would be beneficial if the extension could support both somehow.

[savannahostrowski]

This should be fixed in the next version of the extension, which I will release shortly!