-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[browserstack-service] Sending skipped test details for beforeEach, beforeAll and afterEach hooks in mocha #10258
[browserstack-service] Sending skipped test details for beforeEach, beforeAll and afterEach hooks in mocha #10258
Conversation
added observability and browserstack check
@christian-bromann Please review this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we update the branch and see if the test pass? If so I am happy to merge.
let parentTitle = test.parent | ||
// Sometimes parent will be an object instead of a string | ||
if (typeof test.parent === 'object') { | ||
// @ts-ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we resolve the type issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, we are getting the test object from the context of the test that is passed to the service (test.ctx.test.parent
) in some places. And this object doesn't match with Framework.Test
type which has parent has string, but in the context object, the parent can be an object. Also we don't have any exported types to type for this context.
Currently removed the ts-ignore and casted it to any
. Let me know if you have any better solution
@christian-bromann the tests have passed. Please merge if it looks good to you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Proposed changes
Related v7 PR #10259
Types of changes
Checklist
Further comments
Reviewers: @webdriverio/project-committers