Remove Azure CI and fix test timings (#31)

* Fix test timings * Fix cover reports * Fix path and removed azure ci build * Fixed to fixed path to lcov
net-daemon · Mar 23, 2020 · 6f4d155 · 6f4d155
1 parent 9f5b56a
commit 6f4d155
Show file tree

Hide file tree

Showing 5 changed files with 4,017 additions and 12 deletions.
diff --git a/.github/workflows/ci_build.yml b/.github/workflows/ci_build.yml
@@ -11,4 +11,10 @@ jobs:
             - name: Build code
               run: dotnet build
             - name: Test code
-              run: dotnet test
+              run: dotnet test -v minimal /p:CollectCoverage=true /p:CoverletOutputFormat=lcov /p:CoverletOutput=${{github.workspace}}/codecover/lcov.info
+
+            - name: Publish coverage report to coveralls.io
+              uses: coverallsapp/github-action@master
+              with:
+                github-token: ${{ secrets.GITHUB_TOKEN }}
+                path-to-lcov: ${{github.workspace}}/codecover/lcov.info
diff --git a/azure/netdaemon-ci-pipeline.yml b/azure/netdaemon-ci-pipeline.yml
@@ -3,14 +3,14 @@
 # Add steps that run tests, create a NuGet package, deploy, and more:
 # https://docs.microsoft.com/azure/devops/pipelines/languages/dotnet-core
 
-trigger:
-- master
-- dev
-
-pr:
-  branches:
-    include:
-    - '*'  # must quote since "*" is a YAML reserved character; we want a string
+#trigger:
+#- master
+#- dev
+
+#pr:
+#  branches:
+#    include:
+#    - '*'  # must quote since "*" is a YAML reserved character; we want a string
 
 pool:
   vmImage: 'ubuntu-latest'

diff --git a/tests/NetDaemon.Daemon.Tests/Demon/NetDaemonHostTests.cs b/tests/NetDaemon.Daemon.Tests/Demon/NetDaemonHostTests.cs
@@ -252,9 +252,9 @@ public async Task SpeakShouldWaitUntilMediaPlays()
             // Called once after 150 ms
             DefaultHassClientMock.Verify(n => n.CallService("tts", "google_cloud_say", expectedAttributesExpObject, true), Times.Once);
 
-            await Task.Delay(200);
+            await Task.Delay(300);
 
-            // Called twice after 250ms
+            // Called twice
             DefaultHassClientMock.Verify(n => n.CallService("tts", "google_cloud_say", expectedAttributesExpObject, true), Times.Exactly(2));
 
             await WaitUntilCanceled(daemonTask);

diff --git a/tests/NetDaemon.Daemon.Tests/FluentTests.cs b/tests/NetDaemon.Daemon.Tests/FluentTests.cs
@@ -398,7 +398,7 @@ public async Task TimerEveryWithAttributeShouldCallServiceCorrectNumberOfTimes()
             await RunDefauldDaemonUntilCanceled();
 
             // ASSERT
-            DefaultHassClientMock.VerifyCallServiceTimes("turn_on", Times.AtLeast(4));
+            DefaultHassClientMock.VerifyCallServiceTimes("turn_on", Times.AtLeast(2)); // Low value due to slow cloud builds
             DefaultHassClientMock.VerifyCallService("light", "turn_on", ("attr", "on"), ("entity_id", "light.correct_light"));
         }