-
Notifications
You must be signed in to change notification settings - Fork 48
Bad values in xml [mocha] #29
Comments
Duplicate of #26 |
I'd like to clear some points before moving the conversation to #26. The resulting error is the same, but reasons of it can be different. As I see, here is step stop attribute has empty value @danielsaf could you please specify, which test framework are you using and what Also, I don't understand the step "change my Repository URL for bitbucket"? Do you mean that the same source code but run from bitbucket produces the error? How did it work before? |
@just-boris framework: 'mocha',
|
@just-boris |
@just-boris Hi! Do you still need a demo project to figure out this bug ? The bug is still here. |
I too am seeing this issue with I'm using Mocha and here are my wdio-* packages: |
Just to add, as it may help track down the issue: I had this issue, but don't anymore since disabling wdio-sync and using Promises to write my tests. |
I'm also seeing this issue using cucumber, specifically when I have a test with multiple assertions and the first one fails and the second one passes. I've put a repo together that demonstrates the issue, here - https://bitbucket.org/vividcloud/wdio-allure-reporter-bug A NumberFormatException occurs when generating the report because of empty stop and status attributes on one of the test-case elements, as per this gist. |
Update here! I could reproduce the issue in test-cases that we have in our repository itself. See PR #33
When we don't return promise from some handlers, they will be recorded in a wrong order. So, the easiest solution will be fixing test codebase to make sure, that all command results are handled. This is what we did in webdriverio/webdriverio#1461, but we can't fix something in users' code. Dear thread, could you please check your code and look for unhandled calls, like in the code example above. Does problem appear again, even when you handled all commands properly? |
@just-boris: Makes sense. I wonder could there be a bug somewhere in wdio-sync then, since I was seeing errors with that, but not with Promises? EDIT: Ok, I must have been doing something wrong. Re-enabled sync, and reworked my tests/commands to use sync, and all is well now. EDIT 2: Further info: my wdio-sync is sorted (I think - I don't actually know what I did, but it works now!). I've encountered this when doing |
Hey guys, i've figure out the problem related to this. It's inside the allure-js-common package. Here's the PR for you guys checkout: If any of you won't wait for them accept the pr to make the allure-report worl just clone my forked version of the wdio-allure-reporter. I've forked both repositories to fix this, the wdio-allure-reporter needed to be forked to change the package reference and use my edited one. Allure reporter with updated package: https://github.com/wallynm/wdio-allure-reporter Just install it, (in my case i need to install it globally) and the XML's should be generated correctly.
|
Hey guys, I am experiencing unstable allure reporter using mocha (sometimes the report fails to generate). The tests, app, jenkins, allure etc are all consistent. It feels like some wdio commands return a result that sometimes affects the report. If I am following these threads correctly (allure-framework/allure-js-commons#25 (comment)), a fix is needed somewhere in webdriverio. Is there any progress here? I'm not sure where to start looking, any pointers and I'm happy to try help.
|
@kevinmcdonnell
|
Please, try v2 and check whether this error happens or not. |
@just-boris is there anything that has to be updated in this repo? |
No, it is all fine. |
No simular errors have been reported since may 2017. Looks like it was fixed. I am closing this issue. |
Hi, I need help. After change my Repository URL for bitbucket - allure report stopped work. I see this error on Jenkins log.
Can one help me whit this?
The text was updated successfully, but these errors were encountered: