Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: realtime reporter only set id on non null step.metaStep #119

Merged
merged 1 commit into from
Jan 23, 2021
Merged

fix: realtime reporter only set id on non null step.metaStep #119

merged 1 commit into from
Jan 23, 2021

Conversation

spiroid
Copy link
Contributor

@spiroid spiroid commented Jan 6, 2021

Launching test with the latest version of the codecept ui always terminated with an error setting an id on a null metaStep object.
Digging in the source code revealed that an id is generated and assigned to metaStep before checking that it's not null.

Here is a quick fix

@DavertMik DavertMik merged commit 8198460 into codeceptjs:master Jan 23, 2021
@DavertMik
Copy link
Contributor

Thanks!

@buckthebug
Copy link

Guys, please publish the new version of the npm package. It's a pity when it is already fixed and merged. Examples are failing in UI now. :(

@ydmitry
Copy link

ydmitry commented May 17, 2021

When this will be published?.. 5 month ago latest upgrade of the package. Can't use this package without this fix

@ChristofFritz
Copy link

Don't mind my previous (deleted) comment. Thank's for merging and releasing this fix, i looked at the wrong place earlier.

Copy link

github-actions bot commented Feb 8, 2024

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

5 participants