on 4th time this check is evaluated, it throws an undefined

[minoseah629]

1

2

3

4

[thim81]

Hi @minoseah629

Thanks for reporting this.

Could you share which version of K6 you are using?

[minoseah629]

ill share this was a problem prior to running any k6 command:

k6.exe v0.51.0 (commit/33d3caa7d1, go1.22.3, windows/amd64)

[thim81]

Would it be possible to share how to reproduce the issue?
I want to figure why exec becomes undefined.

[thim81]

Your PR looks good since it improves the JavaScript safety with the optional chaining, but it is strange that exec becomes undefined.

[minoseah629]

i cannot share the exact postman collection json. I have exported my postman collection yesterday. this is what I observed:

1. A good test that has an exec but with an list of 1 string that's empty.

2. Postman has been updated to incorporate a package concept: https://learning.postman.com/docs/tests-and-scripts/write-scripts/package-library/

3. The specific request I see is somehow the usage of packages (even though the script is empty and not using a package), the script does not have an empty exec as expected:

4. This specific request is the only one like this.

[thim81]

@minoseah629 Thanks again for sharing this detailed info and the screenshots. It will allow us to rebuild a Postman collection to reproduce the mentioned cases and add some tests to cover this from the future.

Give me a bit of time to process this all and reproduce it, that will allow your PR to land properly and covered.

[minoseah629]

sounds good.

sorry for not generating unit tests. i am not good with javascript compared to c# experience.

[thim81]

@minoseah629 I was not expecting unit tests, I'm already very happy with all your input and the PR.

[thim81]

I have not been able to reproduce it, but the PR changes are useful for a safer JS.

Did it solve your problem?

[thim81]

hi @minoseah629

We just released v1.10.0 of postman-to-k6 cli.
It includes your PR and some more improvements and additional tests (WIP).

Thanks for your contribution. Let me know if your issue is resolved or still occurring.