-
Notifications
You must be signed in to change notification settings - Fork 12.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
Fix: verification of incremental correctness that was not working because of using wrote writeFile #48751
Conversation
…ile so we know buildInfo was written Also baseline these so its easy to verify the changes
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.
So the tests used to call the unpatched writeFile
and that was always wrong? They were, apparently, doing this because the assert was too strict?
So the tests used to call the unpatched writeFile and that was always wrong? They were, apparently, doing this because the assert was too strict? |
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, assuming I have correctly understood why calling the original writeFile
was a mistake.
This ensures that we use sys.writeFile for baseline buildinfo so that buildInfo correctness is verified. It was missing when we started passing the writeFileCallback which didnt track written files.
This also baselines those differences so its easy to view, read and maintain.
This is on top of #48703. Actual change is only 386786c and f45e9ca