Added real time reporting to spec reporter

[praveendvd]

Proposed changes

This pull request logs the current test status (Pass , failed ,skipped) for spec reporter .

This will fix :

Allow real time reporting of reporters using WDIO Testrunner #7532

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update

Checklist

• I have read the CONTRIBUTING doc
• I have added tests that prove my fix is effective or that my feature works
• I have added necessary documentation (if appropriate)
• I have added proper type definitions for new commands (if appropriate)

Further comments

Test will be added

Reviewers: @webdriverio/project-committers

[christian-bromann]

@praveendvd thanks for taking a stab at this.

Do you have a gif animation how this looks like? Also I believe we should make this optional as it adds additional clutter to stdout.

[praveendvd]

@praveendvd thanks for taking a stab at this.

Do you have a gif animation how this looks like?

I already added that in previous comment , could you have a look

[praveendvd]

#7532 (comment)

@christian-bromann

[praveendvd]

This is with logging set to silent in config file

[BorisOsipov]

@praveendvd @christian-bromann are you sure that for maxInstances> 1...100 the terminal logs won't become a terrible mess?

It is riduculus that we call console.log() when we have @wdio/logger...

[praveendvd]

@praveendvd @christian-bromann are you sure that for maxInstances> 1...100 the terminal logs won't become a terrible mess?

It is riduculus that we call console.log() when we have @wdio/logger...

Could you let me know how to use the logger ? i used this.write but it flushes the output only after the execution completes. Doesnt behave in a realtime manner

[praveendvd]

@christian-bromann The test fails for the sync setup and is not related to this pull request i guesss. Could you have a look Thank you 👏👍

[BorisOsipov]

Could you let me know how to use the logger ?

I don't know. Need investigation why it doesn't print "realtime". I think if logger is not worked as you desired it is not the reason to use console.log().

[christian-bromann]

@praveendvd I am working on fixing the main branch right now, I will let you know when I am done so you can rebase.

[praveendvd]

@christian-bromann and @BorisOsipov

[praveendvd]

@BorisOsipov @christian-bromann replaced console.log with logger and explicit log.level info

[praveendvd]

@christian-bromann Changed logger to event , now we send a realtimereport event that will log to console . We can throw this event from any reporter

[praveendvd]

I added the change but i strogly believe logger will be much more useful as @BorisOsipov mentioned. This implementation works well if the loglevel is silent else the status will get hidden in the sea of logs that we get from wdio. IF we use logger , then user can opt of point it to a file if required so that they can see it in a much more cleaner way .

so the new implementation works exactly the same as if we use logger, but doesnt have option to write to a file. should we implement a flag that allows to write to a file ? but for real time i am not sure if a file write is required or not

[christian-bromann]

Just added some minor formatting adjustments and renamed the new option from realTimeReporting to realtimeReporting given that realtime is one word.

[christian-bromann]

There seems to be a typing issue with Jest. I will go ahead and merge and check in main branch.

[christian-bromann]

Thanks a lot, this is awesome 👍