-
Notifications
You must be signed in to change notification settings - Fork 33
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
Release 2.3: Cannot read properties of undefined (reading 'message') #44
Comments
Getting the same issue. Seems to be caused by this change: #40 |
Is there any update to this, please? @gstokkink I wonder if your PR (#40) requires a specific requirements in the xml files - which not all will have. |
Seeing this as well. Sample Junit file is generated by https://storybook.js.org/ with the |
Thanks for adding a repro case @austinpray-mixpanel - I'll take a look. I think this should be unrelated to #40 ? But I'm not sure yet, I'll dig in. |
Hello @ethomson is there any update on this issue? I've just updated my project from version |
We're running into this open Issue which seems to have been introduced in version 2.3: test-summary/action#44 Signed-off-by: Jan Martens <jan@martens.eu.org>
We're running into this open Issue which seems to have been introduced in version 2.3: test-summary/action#44 Signed-off-by: Jan Martens <jan@martens.eu.org>
@JanMa - apologies, looking now. 👀 Unrelated - I notice that you're pinning your actions to SHAs. (Which is a good practice.) Curious if you're doing this manually or using a tool to help? |
Thanks for the quick fix @ethomson, I really appreciate it 🙂 |
@JanMa Good to know. I just published v2.4, let me know if this doesn't resolve the issue! |
Hey,
Using the new Release creates a error message, when the tests fails:
Cannot read properties of undefined (reading 'message')
Using 2.2 works as expected.
Bye :)
The text was updated successfully, but these errors were encountered: