Simplify application of Trait in DebugServiceTests

Author: andyleejordan

Diff for: test/PowerShellEditorServices.Test/Debugging/DebugServiceTests.cs

@@ -19,6 +19,7 @@
 using Xunit;
 namespace Microsoft.PowerShell.EditorServices.Test.Debugging
 {
+    [Trait("Category", "DebugService")]
     public class DebugServiceTests : IDisposable
     {
         private readonly PsesInternalHost psesHost;
@@ -133,7 +134,6 @@ private Task<IReadOnlyList<LineBreakpoint>> GetConfirmedBreakpoints(ScriptFile s
                 CancellationToken.None);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         // This regression test asserts that `ExecuteScriptWithArgsAsync` works for both script
         // files and, in this case, in-line scripts (commands). The bug was that the cwd was
@@ -162,7 +162,6 @@ await debugService.SetCommandBreakpointsAsync(
             Assert.Equal("[ArrayList: 0]", var.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerAcceptsScriptArgs()
         {
@@ -224,7 +223,6 @@ public async Task DebuggerAcceptsScriptArgs()
             Assert.Equal("\"Extra1\"", childVars[0].ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerSetsAndClearsFunctionBreakpoints()
         {
@@ -250,7 +248,6 @@ public async Task DebuggerSetsAndClearsFunctionBreakpoints()
             Assert.Empty(breakpoints);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerStopsOnFunctionBreakpoints()
         {
@@ -283,7 +280,6 @@ public async Task DebuggerStopsOnFunctionBreakpoints()
             Assert.Equal("2", i.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerSetsAndClearsLineBreakpoints()
         {
@@ -317,7 +313,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Empty(remainingBreakpoints);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerStopsOnLineBreakpoints()
         {
@@ -334,7 +329,6 @@ await debugService.SetLineBreakpointsAsync(
             AssertDebuggerStopped(debugScriptFile.FilePath, 7);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerStopsOnConditionalBreakpoints()
         {
@@ -374,7 +368,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal($"{breakpointValue2}", i.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerStopsOnHitConditionBreakpoint()
         {
@@ -399,7 +392,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal($"{hitCount}", i.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerStopsOnConditionalAndHitConditionBreakpoint()
         {
@@ -423,7 +415,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal("10", i.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerProvidesMessageForInvalidConditionalBreakpoint()
         {
@@ -452,7 +443,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Contains("Unexpected token '-ez'", breakpoints[0].Message);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerFindsParseableButInvalidSimpleBreakpointConditions()
         {
@@ -475,7 +465,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Contains("Use '-gt' instead of '>'", breakpoints[1].Message);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerBreaksWhenRequested()
         {
@@ -487,7 +476,6 @@ public async Task DebuggerBreaksWhenRequested()
             AssertDebuggerPaused();
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerRunsCommandsWhileStopped()
         {
@@ -502,7 +490,6 @@ public async Task DebuggerRunsCommandsWhileStopped()
             Assert.Equal(17, (await executeTask.ConfigureAwait(true))[0]);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariableStringDisplaysCorrectly()
         {
@@ -522,7 +509,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.False(var.IsExpandable);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerGetsVariables()
         {
@@ -573,7 +559,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal("$false", falseVar.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact(Skip = "Variable setting is broken")]
         public async Task DebuggerSetsVariablesNoConversion()
         {
@@ -634,7 +619,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal(newGlobalIntValue, intGlobalVar.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact(Skip = "Variable setting is broken")]
         public async Task DebuggerSetsVariablesWithConversion()
         {
@@ -698,7 +682,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal(newGlobalValue, globalVar.ValueString);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariableEnumDisplaysCorrectly()
         {
@@ -719,7 +702,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.False(var.IsExpandable);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariableHashtableDisplaysCorrectly()
         {
@@ -756,7 +738,6 @@ await debugService.SetLineBreakpointsAsync(
             }
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariableNullStringDisplaysCorrectly()
         {
@@ -777,7 +758,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.True(nullStringVar.IsExpandable);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariablePSObjectDisplaysCorrectly()
         {
@@ -805,7 +785,6 @@ await debugService.SetLineBreakpointsAsync(
             Assert.Equal("\"John\"", childVars["Name"]);
         }
 
-        [Trait("Category", "DebugService")]
         [Fact]
         public async Task DebuggerVariablePSCustomObjectDisplaysCorrectly()
         {
@@ -835,7 +814,6 @@ await debugService.SetLineBreakpointsAsync(
 
         // Verifies fix for issue #86, $proc = Get-Process foo displays just the ETS property set
         // and not all process properties.
-        [Trait("Category", "DebugService")]
         [Fact(Skip = "Length of child vars is wrong now")]
         public async Task DebuggerVariableProcessObjDisplaysCorrectly()
         {