Rebased to latest changes on upstream preview& reenabling test that should now work

panchagnula · panchagnula · commit 85bd2c9de491 · 2018-09-13T14:07:49.000-07:00
diff --git a/src/ResourceManager/Websites/Commands.Websites.Test/ScenarioTests/AppServicePlanTests.cs b/src/ResourceManager/Websites/Commands.Websites.Test/ScenarioTests/AppServicePlanTests.cs
@@ -38,7 +38,7 @@ public void TestCreateNewAppServicePlan()
             WebsitesController.NewInstance.RunPsTest(_logger, "Test-CreateNewAppServicePlan");
         }
 
-        [Fact(Skip = "This passes locally on playback mode. Should be enabled before merging & after common targets starts using the expected Client Runtime, this to test if CI passes without this test")]
+        [Fact]
         [Trait(Category.AcceptanceType, Category.CheckIn)]
         public void TestSetAppServicePlan()
         {

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public void TestCreateNewAppServicePlan()`
`38`	`38`	`WebsitesController.NewInstance.RunPsTest(_logger, "Test-CreateNewAppServicePlan");`
`39`	`39`	`}`
`40`	`40`
`41`		`- [Fact(Skip = "This passes locally on playback mode. Should be enabled before merging & after common targets starts using the expected Client Runtime, this to test if CI passes without this test")]`
	`41`	`+ [Fact]`
`42`	`42`	`[Trait(Category.AcceptanceType, Category.CheckIn)]`
`43`	`43`	`public void TestSetAppServicePlan()`
`44`	`44`	`{`