Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[release/9.0-staging] Backport test fixes related to BinaryFormatter removal #111508

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

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

adamsitnik
Copy link
Member

@adamsitnik adamsitnik commented Jan 16, 2025

Backport of #110550 and #111208 to release/9.0-staging

Customer Impact

  • Customer reported
  • Found internally

CI only changes.

Regression

  • Yes
  • No

Testing

Both fixes have been tested in the main branch.

Risk

Very low: for #110550 we just perform fewer test cases for large inputs, for #111208 we simply don't test Drawing-related types for Mono as it's not supported.

* don't run Drawing-related tests that do things like creating Bitmaps on Mono, as it's not supported (it does not support ComWrappers)

* re-enable the tests
@adamsitnik adamsitnik added test-enhancement Improvements of test source code binaryformatter-migration Issues related to the removal of BinaryFormatter and migrations away from it labels Jan 16, 2025
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 16, 2025
@adamsitnik adamsitnik added area-System.Formats.Nrbf and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 16, 2025
@jkotas jkotas added the Servicing-approved Approved for servicing release label Jan 17, 2025
@jkotas
Copy link
Member

jkotas commented Jan 17, 2025

Test fixes - tell mode.

@jkotas jkotas merged commit f9ffbe0 into dotnet:release/9.0-staging Jan 17, 2025
79 of 91 checks passed
@matouskozak
Copy link
Member

Thank you @adamsitnik for the backport

@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Formats.Nrbf binaryformatter-migration Issues related to the removal of BinaryFormatter and migrations away from it Servicing-approved Approved for servicing release test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants