-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
ci: fix symbol test to handle duplicate symbols #28459
Conversation
In 7cb8396, we improved the symbol test to remove suffixes from the output, so "CIRCULAR$1" became "CIRCULAR". However, the logic that checked for extra symbols depended on each symbol being unique. If multiple symbols with the same name were added (e.g. pulled in from separate files), they would be added to the map as "extras", even if they were marked as expected in the golden file. This commit updates the symbol checking logic to take multiple symbols with the same name into account. Closes angular#28406
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!
merge-assistance: integration test failing due to #28460 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
In 7cb8396, we improved the symbol test to remove suffixes from
the output, so "CIRCULAR$1" became "CIRCULAR". However, the logic
that checked for extra symbols depended on each symbol being unique.
If multiple symbols with the same name were added (e.g. pulled in
from separate files), they would be added to the map as "extras",
even if they were marked as expected in the golden file.
This commit updates the symbol checking logic to take multiple
symbols with the same name into account.
Closes #28406