Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Issues with using a websocket k6 typescript enum type #4623

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Fernando-hub527
Copy link

What?

Ensure that all enums that can be used in the WebSocket module are correctly mapped.

Why?

According to issue #4540 , when using 'EventName.Open' in the script, an error was returned indicating that 'open' was undefined.

Checklist

  • I have performed a self-review of my code.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have added tests for my changes.
  • I have run linter and tests locally (make check) and all pass.

Checklist: Documentation (only for k6 maintainers and if relevant)

Please do not merge this PR until the following items are filled out.

  • I have added the correct milestone and labels to the PR.
  • I have updated the k6-documentation: grafana/k6-docs#PR-NUMBER
  • I have updated the TypeScript definitions: grafana/k6-DefinitelyTyped#PR-NUMBER
  • I have updated the release notes: link

Related PR(s)/Issue(s)

#4540

@Fernando-hub527 Fernando-hub527 requested a review from a team as a code owner March 12, 2025 00:54
@Fernando-hub527 Fernando-hub527 requested review from oleiade and joanlopez and removed request for a team March 12, 2025 00:54
@CLAassistant
Copy link

CLAassistant commented Mar 12, 2025

CLA assistant check
All committers have signed the CLA.

@oleiade
Copy link
Member

oleiade commented Mar 26, 2025

Hey @Fernando-hub527 👋🏻

Thanks for putting the time and effort in doing this. I've commented on the original issue, and will wait for an answer there before considering reviewing and merging this 🙇🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants