Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Release/2.0.0 - Add tests for EventArgs serialization (#25940) #26116

Merged
merged 3 commits into from
Jan 10, 2018

Conversation

ViktorHofer
Copy link
Member

Relates to https://github.com/dotnet/corefx/issues/25930

  • Add tests for EventArgs serialization

  • Add EventArgs equality check as the type doesn't provide its own check

@ViktorHofer ViktorHofer added this to the 2.0.x milestone Jan 2, 2018
@ViktorHofer ViktorHofer self-assigned this Jan 2, 2018
@ViktorHofer ViktorHofer requested a review from danmoseley January 2, 2018 19:51
@ViktorHofer ViktorHofer changed the title Release 2.0.0 - Add tests for EventArgs serialization (#25940) Release/2.0.0 - Add tests for EventArgs serialization (#25940) Jan 2, 2018
@danmoseley
Copy link
Member

Tests pending on dotnet/coreclr#15686

@ViktorHofer
Copy link
Member Author

Triggered a coreclr 2.0 build. waiting for maestro PR...

@ViktorHofer
Copy link
Member Author

@dotnet-bot test this please

* Add tests for EventArgs serialization

* Add EventArgs equality check as the type doesn't provide its own check
@ViktorHofer ViktorHofer force-pushed the release/2.0.0-EventArgs branch from a401bdd to 5e42ac4 Compare January 9, 2018 23:42
* Add tests for EventArgs serialization

* Add EventArgs equality check as the type doesn't provide its own check
@ViktorHofer
Copy link
Member Author

ViktorHofer commented Jan 10, 2018

@dotnet/dnceng some of these legs show that they succeeded but have failed tests. wrong status shown? E.g. Windows x86 Release Build

@danmoseley
Copy link
Member

@dotnet-bot test this please

@MattGal
Copy link
Member

MattGal commented Jan 10, 2018

@ViktorHofer thanks for bringing this to @dnceng 's attention, I'll take a look and file an internal issue if needed if I find anything actionable and broken.

@MattGal
Copy link
Member

MattGal commented Jan 10, 2018

@ViktorHofer I spent some time looking at this. Unfortunately, as you've restarted the build I can't see what the results view (mc.dot.net) looked like when you first raised the issue, but in either case the windows tests are reliably failing in the binary serialization tests, i.e.
https://mc.dot.net/#/user/ViktorHofer/pr~2Fjenkins~2Fdotnet~2Fcorefx~2Frelease~2F2.0.0~2F/test~2Ffunctional~2Fcli~2F/5e42ac4b84d5061eba00ebd45ccc34338e713724/workItem/System.Runtime.Serialization.Formatters.Tests .

The test fails on every platform actually. If you can capture a screen shot of what you're seeing if this happens again that'd be awesome, as I'd expect to have seen this test fail uniformly (every run in your PR seems to have it failing)

@ViktorHofer
Copy link
Member Author

ViktorHofer commented Jan 10, 2018

Yes I expected nearly all legs to fail as there was a bug in the test code. Unfortunately the build was restarted today but I still have the link to the leg: https://ci3.dot.net/job/dotnet_corefx/job/release_2.0.0/job/windows-TGroup_netfx+CGroup_Release+AGroup_x86+TestOuter_false_prtest/66/

Is that enough?

@ViktorHofer
Copy link
Member Author

ViktorHofer commented Jan 10, 2018

@dotnet/dnceng this time a failing OSX x64 Debug Build leg:

https://ci3.dot.net/job/dotnet_corefx/job/release_2.0.0/job/osx-TGroup_netcoreapp+CGroup_Debug+AGroup_x64+TestOuter_false_prtest/73/consoleText

ERROR: Error cloning remote repo 'origin'
hudson.plugins.git.GitException: Could not init /Users/dotnet-bot/j/workspace/dotnet_corefx/release_2.0.0/osx-TGroup_netcoreapp+CGroup_Debug+AGroup_x64+TestOuter_false_prtest

@ViktorHofer
Copy link
Member Author

@dotnet-bot test OSX x64 Debug Build

@ViktorHofer ViktorHofer merged commit 320a39e into dotnet:release/2.0.0 Jan 10, 2018
@ViktorHofer ViktorHofer deleted the release/2.0.0-EventArgs branch January 10, 2018 23:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants