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
Bugfix/reporting png is cancelled #37513
Bugfix/reporting png is cancelled #37513
Conversation
@@ -0,0 +1,36 @@ | |||
/* |
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.
nice to see these tests... but is it supposed to catch the issue? I went back to the old code and ran the jest tests and everything still passed. Maybe I'm doing something wrong though.
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.
It is a binding issue, I'll see if I can't get a test-case for that specifically since these methods do get passed around
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.
Added a new test case, plus changed how some older tests functioned (they did explicit bindings when utilizing this util). Tests started failing when I added these news specs and adjusted older tests.
I'll get it on the list to re-enable that functional test.
💔 Build Failed |
…ts that do binding
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
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, code review and pulled down and tested, png reporting works.
Fixes an issue with the cancellation-token utility, will backport to 7.x as well.