Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Remove tests previously ported to CoreFX repo #19527

Conversation

AaronRobinsonMSFT
Copy link
Member

Ported tests in dotnet/corefx#30690

cc @luqunl

@AaronRobinsonMSFT AaronRobinsonMSFT added area-Interop test enhancement Improvements of test source code labels Aug 17, 2018
@AaronRobinsonMSFT AaronRobinsonMSFT added this to the 3.0 milestone Aug 17, 2018
@@ -124,9 +124,6 @@ CoreMangLib/system/collections/generic/hashset/Regression_Dev10_609271/Regressio
CoreMangLib/system/collections/generic/hashset/Regression_Dev10_624201/Regression_Dev10_624201.sh
GC/Coverage/smalloom/smalloom.sh
Interop/COM/NETClients/Primitives/NETClientPrimitives/NETClientPrimitives.sh
Interop/MarshalAPI/GetNativeVariantForObject/GetNativeVariantForObject/GetNativeVariantForObject.sh
Interop/MarshalAPI/GetObjectForNativeVariant/GetObjectForNativeVariant/GetObjectForNativeVariant.sh
Interop/MarshalAPI/GetObjectsForNativeVariants/GetObjectsForNativeVariants/GetObjectsForNativeVariants.sh
Interop/MarshalAPI/IUnknown/IUnknownTest/IUnknownTest.sh
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still need this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, because it has native test assets - we might reconsider in the future?

@luqunl
Copy link

luqunl commented Aug 17, 2018

Maybe later we should add some interop corefx testcases into coreclr CI, similar to https://ci.dot.net/job/dotnet_coreclr/job/master/job/x64_release_windows_nt_corefx_innerloop_prtest/796/console

@AaronRobinsonMSFT AaronRobinsonMSFT merged commit ff9b943 into dotnet:master Aug 17, 2018
@AaronRobinsonMSFT AaronRobinsonMSFT deleted the remove_duplicate_interop_tests branch August 17, 2018 18:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Interop test enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants