PR comments fix

vladimir-shcherbakov · vladimir-shcherbakov · commit 72be768149fa · 2019-02-10T18:40:08.000-08:00
diff --git a/src/Automation/Automation.Test/ScenarioTests/AccountTests.cs b/src/Automation/Automation.Test/ScenarioTests/AccountTests.cs
@@ -12,7 +12,6 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
 using Microsoft.WindowsAzure.Commands.ScenarioTest;
 using Xunit;
 
diff --git a/src/Automation/Automation.Test/ScenarioTests/AutomationTestRunner.cs b/src/Automation/Automation.Test/ScenarioTests/AutomationTestRunner.cs
@@ -1,21 +1,15 @@
-﻿using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-using Microsoft.Azure.Commands.TestFx;
-using Microsoft.WindowsAzure.Commands.ScenarioTest;
+﻿using Microsoft.Azure.Commands.TestFx;
 using Xunit.Abstractions;
 
-namespace Microsoft.Azure.Commands.Network.Test.ScenarioTests
+namespace Commands.Automation.Test
 {
     public class AutomationTestRunner
     {
         protected readonly ITestRunner TestRunner;
 
         protected AutomationTestRunner(ITestOutputHelper output)
         {
-            TestRunner = TestFx.TestManager.CreateInstance (output)
+            TestRunner = TestManager.CreateInstance (output)
                 .WithNewPsScriptFilename ($"{GetType().Name}.ps1")
                 .WithProjectSubfolderForTests ("ScenarioTests")
                 .WithCommonPsScripts (new[]
diff --git a/src/Automation/Automation.Test/ScenarioTests/AutomationTests.cs b/src/Automation/Automation.Test/ScenarioTests/AutomationTests.cs
@@ -12,11 +12,10 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
 using Microsoft.WindowsAzure.Commands.ScenarioTest;
 using Xunit;
 
-namespace Microsoft.Azure.Commands.Automation.Test
+namespace Commands.Automation.Test
 {
     public class AutomationTests : AutomationTestRunner
     {
diff --git a/src/Automation/Automation.Test/ScenarioTests/CertificateTests.cs b/src/Automation/Automation.Test/ScenarioTests/CertificateTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class CertificateTests : AutomationTestRunner
     {
         public CertificateTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/ConnectionTests.cs b/src/Automation/Automation.Test/ScenarioTests/ConnectionTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class ConnectionTests : AutomationTestRunner
     {
         public ConnectionTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/CredentialTests.cs b/src/Automation/Automation.Test/ScenarioTests/CredentialTests.cs
@@ -12,14 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.Azure.ServiceManagement.Common.Models;
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class CredentialTests : AutomationTestRunner
     {
         public CredentialTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/HybridWorkerGroupTests.cs b/src/Automation/Automation.Test/ScenarioTests/HybridWorkerGroupTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class HybridWorkerGroupTests : AutomationTestRunner
     {
         public HybridWorkerGroupTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/JobScheduleTests.cs b/src/Automation/Automation.Test/ScenarioTests/JobScheduleTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class JobScheduleTests : AutomationTestRunner
     {
         public JobScheduleTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/ModuleTests.cs b/src/Automation/Automation.Test/ScenarioTests/ModuleTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class ModuleTests : AutomationTestRunner
     {
         public ModuleTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/RunbookJobTests.cs b/src/Automation/Automation.Test/ScenarioTests/RunbookJobTests.cs
@@ -12,14 +12,12 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using System.IO;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using System.IO;
-    using Xunit;
-
     public class RunbookJobTests : AutomationTestRunner
     {
         public RunbookJobTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/ScheduleTests.cs b/src/Automation/Automation.Test/ScenarioTests/ScheduleTests.cs
@@ -12,18 +12,13 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.Azure.ServiceManagement.Common.Models;
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class ScheduleTests : AutomationTestRunner
     {
-        public XunitTracingInterceptor logger;
-
         public ScheduleTests(Xunit.Abstractions.ITestOutputHelper output)
             : base(output)
         {
diff --git a/src/Automation/Automation.Test/ScenarioTests/SourceControlTests.cs b/src/Automation/Automation.Test/ScenarioTests/SourceControlTests.cs
@@ -24,13 +24,11 @@
 are recorded. See example in SourceControlTests.ps1 where $testReposInfo is defined.
 */
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class SourceControlTests : AutomationTestRunner
     {
         public SourceControlTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/UpdateManagementTests.cs b/src/Automation/Automation.Test/ScenarioTests/UpdateManagementTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class UpdateManagementTests : AutomationTestRunner
     {
         public UpdateManagementTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/VariableTests.cs b/src/Automation/Automation.Test/ScenarioTests/VariableTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class VariableTests : AutomationTestRunner
     {
         public VariableTests(Xunit.Abstractions.ITestOutputHelper output)
diff --git a/src/Automation/Automation.Test/ScenarioTests/WebhookTests.cs b/src/Automation/Automation.Test/ScenarioTests/WebhookTests.cs
@@ -12,13 +12,11 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;
+using Microsoft.WindowsAzure.Commands.ScenarioTest;
+using Xunit;
 
 namespace Commands.Automation.Test
 {
-    using Microsoft.WindowsAzure.Commands.ScenarioTest;
-    using Xunit;
-
     public class WebhookTests : AutomationTestRunner
     {
         public WebhookTests(Xunit.Abstractions.ITestOutputHelper output)

Original file line number	Diff line number	Diff line change
`@@ -12,11 +12,10 @@`
`12`	`12`	`// limitations under the License.`
`13`	`13`	`// ----------------------------------------------------------------------------------`
`14`	`14`
`15`		`-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;`
`16`	`15`	`using Microsoft.WindowsAzure.Commands.ScenarioTest;`
`17`	`16`	`using Xunit;`
`18`	`17`
`19`		`-namespace Microsoft.Azure.Commands.Automation.Test`
	`18`	`+namespace Commands.Automation.Test`
`20`	`19`	`{`
`21`	`20`	`public class AutomationTests : AutomationTestRunner`
`22`	`21`	`{`
Original file line number	Diff line number	Diff line change
`@@ -12,13 +12,11 @@`
`12`	`12`	`// limitations under the License.`
`13`	`13`	`// ----------------------------------------------------------------------------------`
`14`	`14`
	`15`	`+using Microsoft.WindowsAzure.Commands.ScenarioTest;`
	`16`	`+using Xunit;`
`15`	`17`
`16`	`18`	`namespace Commands.Automation.Test`
`17`	`19`	`{`
`18`		`- using Microsoft.Azure.Commands.Network.Test.ScenarioTests;`
`19`		`- using Microsoft.WindowsAzure.Commands.ScenarioTest;`
`20`		`- using Xunit;`
`21`		`-`
`22`	`20`	`public class CertificateTests : AutomationTestRunner`
`23`	`21`	`{`
`24`	`22`	`public CertificateTests(Xunit.Abstractions.ITestOutputHelper output)`
Original file line number	Diff line number	Diff line change
`@@ -12,18 +12,13 @@`
`12`	`12`	`// limitations under the License.`
`13`	`13`	`// ----------------------------------------------------------------------------------`
`14`	`14`
`15`		`-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;`
	`15`	`+using Microsoft.WindowsAzure.Commands.ScenarioTest;`
	`16`	`+using Xunit;`
`16`	`17`
`17`	`18`	`namespace Commands.Automation.Test`
`18`	`19`	`{`
`19`		`- using Microsoft.Azure.ServiceManagement.Common.Models;`
`20`		`- using Microsoft.WindowsAzure.Commands.ScenarioTest;`
`21`		`- using Xunit;`
`22`		`-`
`23`	`20`	`public class ScheduleTests : AutomationTestRunner`
`24`	`21`	`{`
`25`		`- public XunitTracingInterceptor logger;`
`26`		`-`
`27`	`22`	`public ScheduleTests(Xunit.Abstractions.ITestOutputHelper output)`
`28`	`23`	`: base(output)`
`29`	`24`	`{`
Original file line number	Diff line number	Diff line change
`@@ -24,13 +24,11 @@`
`24`	`24`	`are recorded. See example in SourceControlTests.ps1 where $testReposInfo is defined.`
`25`	`25`	`*/`
`26`	`26`
`27`		`-using Microsoft.Azure.Commands.Network.Test.ScenarioTests;`
	`27`	`+using Microsoft.WindowsAzure.Commands.ScenarioTest;`
	`28`	`+using Xunit;`
`28`	`29`
`29`	`30`	`namespace Commands.Automation.Test`
`30`	`31`	`{`
`31`		`- using Microsoft.WindowsAzure.Commands.ScenarioTest;`
`32`		`- using Xunit;`
`33`		`-`
`34`	`32`	`public class SourceControlTests : AutomationTestRunner`
`35`	`33`	`{`
`36`	`34`	`public SourceControlTests(Xunit.Abstractions.ITestOutputHelper output)`