Skip to content
This repository has been archived by the owner on Nov 23, 2021. It is now read-only.

--skip-pre option does not skip pre-request scripts at collection or folder level #105

Closed
MopsiMauser opened this issue Jun 2, 2021 · 3 comments

Comments

@MopsiMauser
Copy link

Hi,

when using the --skip-pre option the converter does not remove the pre-request scripts at the collection or folder level. I attached a postman collection and the converted k6 script. I just had to change the extensions because github refuses to upload json and js files.

Command Line: postman-to-k6 TestCollection.postman_collection.json -o TestCollection.js --skip-pre

Best Regards

TestCollection.js.txt
TestCollection.postman_collection.json.txt

@thim81
Copy link
Contributor

thim81 commented Aug 18, 2021

Hi @MopsiMauser

I have provided a forked NPM package version that potential could fix your current issue plus I created a PR on the original postman-to-k6 package. Could you try it and give feedback if your problems is resolved?

To use the fix, you need to replace in your packages.json:

"dependencies": {
    "postman-to-k6": "^1.5.0"
  }

to

"dependencies": {
    "@apideck/postman-to-k6": "^1.6.1"
  }

This is a forked NPM package, to help overcome the issue. The current maintainers are quite overloaded by the acquisition of K6 by Grafana. By forking the repo & package, users can keep using new PR's & (security) fixes. At a later stage the changes can potentially be merged back in the original postman-to-k6 repo.

The changelog contains the differences between the original package & the forked version.

@MopsiMauser
Copy link
Author

Hi @thim81

thank you for your effort!! I just tested it and it works now <3

@ppcano
Copy link
Collaborator

ppcano commented Nov 22, 2021

Apologies for the inactivity on this project.

Due to the inability to properly support this project, the k6 team has decided not to continue its development. The primary reason is that k6 scope has grown significantly with the launch of k6 extensions. The team is prioritizing extending k6 capabilities over the converters.

We suggest you post your issue at api-deck/postman-to-k6. The project is active and maintained on this fork.

@ppcano ppcano closed this as completed Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants