-
Notifications
You must be signed in to change notification settings - Fork 183
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
[tests-only][full-ci]Update expected to failure based on correct issue #7211
Conversation
80a5cf9
to
54b49c2
Compare
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
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.
let's wait till decision from dev team
54b49c2
to
b47366d
Compare
9ea846d
to
8cdc362
Compare
This PR is put on hold since this issue is being looked by the development team. So this PR can be waited till some notice from them. |
8cdc362
to
41218a0
Compare
let's merge this PR. as this is not adding new test but updating the issue only. The test will probably be removed in the PR where the bug is fixed |
Kudos, SonarCloud Quality Gate passed! |
Description
Since this issue #4393 was closed so that needs to be updated according to this issue linke for it. #4393 (comment). But New issue is created according to the actual behavior since the expected behavior is unknown. So This PR updates the expected to failure with the new issue created for the same failing test.
Related Issue:
#4393 (comment)