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

Report creation with '|' in name fails #3

Closed
dgasi opened this issue Jul 27, 2016 · 10 comments
Closed

Report creation with '|' in name fails #3

dgasi opened this issue Jul 27, 2016 · 10 comments

Comments

@dgasi
Copy link

dgasi commented Jul 27, 2016

Hi,
I noticed when using wdio-json-reporter if you have large number of files that it fails after 28 files:
.
.
.
Wrote json report to [reports].
Wrote json report to [reports].
Failed to write json report to [reports]. Error: Error: ENOENT: no such file or directory, open '%PATH%\WDIO.json.chrome.0|.json'

So reports CID it starts with '0a' and after going trough the alphabet it starts with special characters because it created: WDIO.json.chrome.0}.json and then tried WDIO.json.chrome.0|.json where it failed.

So is it possible to reformat the name or to use numbers instead of characters?

Thx.

@fijijavis
Copy link
Collaborator

Sure. I'll take a look at this later today.

@dgasi
Copy link
Author

dgasi commented Aug 1, 2016

Hi,

Any news regarding this issue?

Thx.

@fijijavis
Copy link
Collaborator

fijijavis commented Aug 1, 2016

Sorry... Had some things come up between work and home. Will look at this today.

@fijijavis
Copy link
Collaborator

fijijavis commented Aug 2, 2016

By any chance have you tried using the wdio-junit-reporter? I'm curious to see if the problem exists there too.

CID is generated from within WDIO so I'm thinking it might make more sense to address this issue at the root instead of a one off fix here.

@dgasi
Copy link
Author

dgasi commented Aug 3, 2016

Just checked with Junit reporter and it's the same thing.

@dgasi
Copy link
Author

dgasi commented Aug 22, 2016

@fijijavis - I opened the issue on webdriverio project and got response from @christian-bromann:

webdriverio/webdriverio#1518

@dgasi I think we should not include the capability id in the name then. I think this should be fixed in the json reporter then.

does this help you to fix the issue?

@fijijavis
Copy link
Collaborator

Yes, I guess that does. If the direction is to not use cid in the name of the file then I'll work on an adjustment.

@dgasi
Copy link
Author

dgasi commented Aug 22, 2016

thanks man. let me know when you have it done I can try it out really fast.

@fijijavis
Copy link
Collaborator

Published v0.0.4 with the above referenced change. I tried it with a 2 spec files and it looks good. Let me know what you think

@dgasi
Copy link
Author

dgasi commented Aug 23, 2016

Hi,

Just did a run and no fails in reports creation .Works like a charm :-D

Thanks man.

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

No branches or pull requests

2 participants