Skip to content
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

AbstractAcceptanceTest integrationtest fix for vs2019 #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vip32
Copy link
Contributor

@vip32 vip32 commented May 29, 2019

the new vs2019 (16.2.0) test explorer has an issue with GetAssemblies when it tries to load internal test assemblies
microsoft/vstest#2008

with this fix the Microsoft assemblies are not loaded when an integration test runs (FunctionMonkey.Testing.AbstractAcceptanceTest)

the new vs2019 (16.2.0) test explorer has an issue with GetAssemblies when it tries to load internal test assemblies
microsoft/vstest#2008

with this fix the Microsoft assemblies are not loaded when an integration test runs (FunctionMonkey.Testing.AbstractAcceptanceTest)
@@ -40,7 +40,7 @@ public static IFunctionAppConfiguration FindConfiguration()

private static bool Scan(out MethodInfo linkBackInfo, out IFunctionAppConfiguration findConfiguration)
{
Assembly[] assemblies = AppDomain.CurrentDomain.GetAssemblies();
Assembly[] assemblies = AppDomain.CurrentDomain.GetAssemblies().Where(a => !a.GetName().Name.StartsWith("Microsoft.")).ToArray();
Copy link
Owner

Choose a reason for hiding this comment

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

Is it safe to further qualify the Microsoft. prefix - Microsoft.VisualStudio perhaps?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants