From 482bd26568f68260c1c6f707b8c0cae757ca6ad4 Mon Sep 17 00:00:00 2001 From: Jason Malinowski Date: Tue, 11 Jul 2017 17:34:53 -0700 Subject: [PATCH] Disable xUnit1013 in one file where it doesn't make sense Right now this code is doing a pattern where it overrides the public methods in derived types, and those have facts. That forces the base type to also be public, but we can't put Facts there. This can be cleaned up in a better way, but I want to keep this first round of changes as mechanical as possible. --- .../IntegrationTest/IntegrationTests/Workspace/WorkspaceBase.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/VisualStudio/IntegrationTest/IntegrationTests/Workspace/WorkspaceBase.cs b/src/VisualStudio/IntegrationTest/IntegrationTests/Workspace/WorkspaceBase.cs index 0e256cb3807..dcdc52963be 100644 --- a/src/VisualStudio/IntegrationTest/IntegrationTests/Workspace/WorkspaceBase.cs +++ b/src/VisualStudio/IntegrationTest/IntegrationTests/Workspace/WorkspaceBase.cs @@ -5,6 +5,8 @@ using Xunit; using ProjectUtils = Microsoft.VisualStudio.IntegrationTest.Utilities.Common.ProjectUtils; +#pragma warning disable xUnit1013 // currently there are public virtual methods that are overridden by derived types + namespace Roslyn.VisualStudio.IntegrationTests.Workspace { public abstract class WorkspaceBase : AbstractEditorTest -- GitLab