Merged PR 950795: Remove RunProcessAsTask workaround from smoke tests

Remove workaround for jamesmanning/RunProcessAsTask#20
Azure · Jul 10, 2018 · 20143fa · 20143fa
1 parent 91c5612
commit 20143fa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/smoke/IotEdgeQuickstart/IotEdgeQuickstart.csproj b/smoke/IotEdgeQuickstart/IotEdgeQuickstart.csproj
@@ -25,7 +25,7 @@
     <PackageReference Include="Microsoft.Azure.Devices" Version="1.16.0" />
     <PackageReference Include="Microsoft.Azure.EventHubs" Version="1.1.0" />
     <PackageReference Include="Microsoft.CodeCoverage" Version="1.0.3" />
-    <PackageReference Include="RunProcessAsTask" Version="1.2.1" />
+    <PackageReference Include="RunProcessAsTask" Version="1.2.3" />
     <PackageReference Include="YamlDotNet" Version="4.3.2" />
   </ItemGroup>
 

diff --git a/smoke/IotEdgeQuickstart/details/Iotedged.cs b/smoke/IotEdgeQuickstart/details/Iotedged.cs
@@ -74,10 +74,7 @@ public Iotedged(string archivePath, Option<RegistryCredentials> credentials, Opt
 
         public async Task VerifyNotActive()
         {
-            // 'sleep' before 'systemctl' works around a problem in
-            // RunProcessAsTask seen on Raspberry Pi 3. See
-            // https://github.com/jamesmanning/RunProcessAsTask/issues/20
-            string[] result = await Process.RunAsync("bash", "-c \"sleep .5 && systemctl --no-pager show iotedge | grep ActiveState=\"");
+            string[] result = await Process.RunAsync("bash", "-c \"systemctl --no-pager show iotedge | grep ActiveState=\"");
             if (result.First().Split("=").Last() == "active")
             {
                 throw new Exception("IoT Edge Security Daemon is already active. If you want this test to overwrite the active configuration, please run `systemctl stop iotedged` first.");