Brought back test reporter, this time phoenix-actions, a revamped dor…

…ny/test-reporter with test summary
oskardudycz · Dec 3, 2022 · b21fa5a · b21fa5a
1 parent 82ce320
commit b21fa5a
Show file tree

Hide file tree

Showing 49 changed files with 350 additions and 232 deletions.
diff --git a/.github/workflows/build.dotnet.testreport.yml b/.github/workflows/build.dotnet.testreport.yml
@@ -0,0 +1,23 @@
+name: Test Report
+
+on:
+    workflow_run:
+        workflows: ['Build and Test']
+        types:
+            - completed
+
+jobs:
+    build:
+        runs-on: ubuntu-latest
+
+        steps:
+            - name: Publish test results
+              uses: phoenix-actions/test-reporting@v10
+              if: success() || failure()
+              with:
+                  name: Tests Results
+                  artifact: test-results
+                  reporter: dotnet-trx
+                  path: '**/test-results.trx'
+                  output-to: 'step-summary'
+                  fail-on-error: 'false'
diff --git a/.github/workflows/build.dotnet.yml b/.github/workflows/build.dotnet.yml
@@ -29,7 +29,14 @@ jobs:
               run: dotnet build --configuration Release --no-restore
 
             - name: Run tests
-              run: dotnet test --configuration Release --no-build --filter Category!=SkipCI
+              run: dotnet test --configuration Release --no-build --filter Category!=SkipCI --logger "trx;LogFileName=test-results.trx
+
+            - name: Upload test results
+              uses: actions/upload-artifact@v2
+              if: success() || failure()
+              with:
+                name: test-results
+                path: '**/test-results.trx'
 
             - name: Stop containers
               if: always()

diff --git a/Core.EventStoreDB.Tests/Core.EventStoreDB.Tests.csproj b/Core.EventStoreDB.Tests/Core.EventStoreDB.Tests.csproj
@@ -11,17 +11,19 @@
     </ItemGroup>
 
     <ItemGroup>
-        <ProjectReference Include="..\Core.EventStoreDB\Core.EventStoreDB.csproj" />
+        <ProjectReference Include="..\Core.EventStoreDB\Core.EventStoreDB.csproj"/>
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
         </PackageReference>
     </ItemGroup>
 
+    <Import Project="..\Tests.Build.props"/>
+
 </Project>
diff --git a/Core.Kafka.Tests/Core.Kafka.Tests.csproj b/Core.Kafka.Tests/Core.Kafka.Tests.csproj
@@ -24,4 +24,6 @@
         </PackageReference>
     </ItemGroup>
 
+    <Import Project="..\Tests.Build.props"/>
+
 </Project>
diff --git a/Core.Tests/Core.Tests.csproj b/Core.Tests/Core.Tests.csproj
@@ -11,17 +11,20 @@
     </ItemGroup>
 
     <ItemGroup>
-        <ProjectReference Include="..\Core\Core.csproj" />
+        <ProjectReference Include="..\Core\Core.csproj"/>
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Marten" Version="6.0.0-alpha.3" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Marten" Version="6.0.0-alpha.3"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
         </PackageReference>
     </ItemGroup>
+
+    <Import Project="..\Tests.Build.props"/>
+
 </Project>
diff --git a/EventSourcing.Integration.Tests/EventSourcing.Integration.Tests.csproj b/EventSourcing.Integration.Tests/EventSourcing.Integration.Tests.csproj
@@ -11,13 +11,16 @@
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Marten" Version="6.0.0-alpha.3" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Marten" Version="6.0.0-alpha.3"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
         </PackageReference>
     </ItemGroup>
+
+    <Import Project="..\Tests.Build.props"/>
+
 </Project>
diff --git a/EventSourcing.NetCore.sln b/EventSourcing.NetCore.sln
@@ -19,6 +19,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution
 		Directory.Build.props = Directory.Build.props
 		Dockerfile = Dockerfile
 		Core.Build.props = Core.Build.props
+		Tests.Build.props = Tests.Build.props
 	EndProjectSection
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Core.Tests", "Core.Tests\Core.Tests.csproj", "{E1B97A7B-97C3-4C14-9BE6-ACE0AF45CE61}"
@@ -202,6 +203,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "CI", "CI", "{F078A982-331A-
 ProjectSection(SolutionItems) = preProject
 	.github\workflows\build.dotnet.yml = .github\workflows\build.dotnet.yml
 	.github\workflows\publish.docker.yml = .github\workflows\publish.docker.yml
+	.github\workflows\build.dotnet.testreport.yml = .github\workflows\build.dotnet.testreport.yml
 EndProjectSection
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Marten.Integration.Tests", "Marten.Integration.Tests\Marten.Integration.Tests.csproj", "{D9D67A6C-FE95-4864-B8F1-BD2689134590}"

diff --git a/Marten.Integration.Tests/Marten.Integration.Tests.csproj b/Marten.Integration.Tests/Marten.Integration.Tests.csproj
@@ -11,22 +11,24 @@
     </ItemGroup>
 
     <ItemGroup>
-        <ProjectReference Include="..\Core\Core.csproj" />
+        <ProjectReference Include="..\Core\Core.csproj"/>
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Marten" Version="6.0.0-alpha.3" />
-        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Marten" Version="6.0.0-alpha.3"/>
+        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
         </PackageReference>
     </ItemGroup>
 
     <ItemGroup>
-        <Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />
+        <Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}"/>
     </ItemGroup>
+
+    <Import Project="..\Tests.Build.props"/>
 </Project>
diff --git a/Sample/ECommerce/Carts/Carts.Api.Tests/Carts.Api.Tests.csproj b/Sample/ECommerce/Carts/Carts.Api.Tests/Carts.Api.Tests.csproj
@@ -24,4 +24,6 @@
       <ProjectReference Include="..\Carts.Api\Carts.Api.csproj" />
     </ItemGroup>
 
+    <Import Project="..\..\..\..\Tests.Build.props"/>
+
 </Project>
diff --git a/Sample/ECommerce/Carts/Carts.Tests/Builders/CartBuilder.cs b/Sample/ECommerce/Carts/Carts.Tests/Builders/CartBuilder.cs
@@ -5,7 +5,7 @@ namespace Carts.Tests.Builders;
 
 internal class CartBuilder
 {
-    private Func<ShoppingCart> build  = () => new ShoppingCart();
+    private Func<ShoppingCart> build = () => new ShoppingCart();
 
     public CartBuilder Opened()
     {

diff --git a/Sample/ECommerce/Carts/Carts.Tests/Carts.Tests.csproj b/Sample/ECommerce/Carts/Carts.Tests/Carts.Tests.csproj
@@ -5,21 +5,23 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="NSubstitute" Version="4.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="NSubstitute" Version="4.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0" />
+        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0"/>
     </ItemGroup>
 
 
     <ItemGroup>
-      <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj" />
-      <ProjectReference Include="..\Carts\Carts.csproj" />
+        <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj"/>
+        <ProjectReference Include="..\Carts\Carts.csproj"/>
     </ItemGroup>
 
+    <Import Project="..\..\..\..\Tests.Build.props"/>
+
 </Project>
diff --git a/Sample/ECommerce/Carts/Carts.Tests/Carts/InitializingCart/InitializeCartTests.cs b/Sample/ECommerce/Carts/Carts.Tests/Carts/InitializingCart/InitializeCartTests.cs
@@ -31,4 +31,4 @@ public void ForValidParams_ShouldCreateCartWithPendingStatus()
                 clientId
             );
     }
-}
+}
diff --git a/Sample/ECommerce/Carts/Carts.Tests/Extensions/Reservations/CartExtensions.cs b/Sample/ECommerce/Carts/Carts.Tests/Extensions/Reservations/CartExtensions.cs
@@ -12,7 +12,6 @@ internal static class CartExtensions
         Guid id,
         Guid clientId)
     {
-
         shoppingCart.Id.Should().Be(id);
         shoppingCart.ClientId.Should().Be(clientId);
         shoppingCart.Status.Should().Be(ShoppingCartStatus.Pending);

diff --git a/Sample/ECommerce/Carts/Carts.Tests/Stubs/Ids/FakeIdGenerator.cs b/Sample/ECommerce/Carts/Carts.Tests/Stubs/Ids/FakeIdGenerator.cs
@@ -2,8 +2,8 @@
 
 namespace Carts.Tests.Stubs.Ids;
 
-public class FakeIdGenerator : IIdGenerator
+public class FakeIdGenerator: IIdGenerator
 {
     public Guid? LastGeneratedId { get; private set; }
     public Guid New() => (LastGeneratedId = Guid.NewGuid()).Value;
-}
+}
diff --git a/Sample/ECommerce/Carts/Carts.Tests/Stubs/Products/FakeProductPriceCalculator.cs b/Sample/ECommerce/Carts/Carts.Tests/Stubs/Products/FakeProductPriceCalculator.cs
@@ -6,11 +6,12 @@ namespace Carts.Tests.Stubs.Products;
 internal class FakeProductPriceCalculator: IProductPriceCalculator
 {
     public const decimal FakePrice = 13;
+
     public IReadOnlyList<PricedProductItem> Calculate(params ProductItem[] productItems)
     {
         return productItems
             .Select(pi =>
                 PricedProductItem.Create(pi, FakePrice))
             .ToList();
     }
-}
+}
diff --git a/Sample/ECommerce/Carts/Carts.Tests/Stubs/Repositories/FakeRepository.cs b/Sample/ECommerce/Carts/Carts.Tests/Stubs/Repositories/FakeRepository.cs
@@ -3,13 +3,13 @@
 
 namespace Carts.Tests.Stubs.Repositories;
 
-public class FakeRepository<T> : IMartenRepository<T> where T : class, IAggregate
+public class FakeRepository<T>: IMartenRepository<T> where T : class, IAggregate
 {
     public Dictionary<Guid, T> Aggregates { get; private set; }
 
     public FakeRepository(params T[] aggregates)
     {
-        Aggregates = aggregates.ToDictionary(ks=> ks.Id, vs => vs);
+        Aggregates = aggregates.ToDictionary(ks => ks.Id, vs => vs);
     }
 
     public Task<T?> Find(Guid id, CancellationToken cancellationToken)
@@ -23,13 +23,15 @@ public async Task<long> Add(T aggregate, CancellationToken cancellationToken = d
         return await Task.FromResult(aggregate.Version);
     }
 
-    public async Task<long> Update(T aggregate, long? expectedVersion = null, CancellationToken cancellationToken = default)
+    public async Task<long> Update(T aggregate, long? expectedVersion = null,
+        CancellationToken cancellationToken = default)
     {
         Aggregates[aggregate.Id] = aggregate;
         return await Task.FromResult(aggregate.Version);
     }
 
-    public async Task<long> Delete(T aggregate, long? expectedVersion = null, CancellationToken cancellationToken = default)
+    public async Task<long> Delete(T aggregate, long? expectedVersion = null,
+        CancellationToken cancellationToken = default)
     {
         Aggregates.Remove(aggregate.Id);
         return await Task.FromResult(aggregate.Version);

diff --git a/Sample/ECommerce/Orders/Orders.Api.Tests/Orders.Api.Tests.csproj b/Sample/ECommerce/Orders/Orders.Api.Tests/Orders.Api.Tests.csproj
@@ -5,23 +5,25 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="7.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="NSubstitute" Version="4.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
-        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="NSubstitute" Version="4.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
+        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
-          <PrivateAssets>all</PrivateAssets>
-          <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+            <PrivateAssets>all</PrivateAssets>
+            <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
     </ItemGroup>
 
     <ItemGroup>
-      <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj" />
-      <ProjectReference Include="..\Orders.Api\Orders.Api.csproj" />
+        <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj"/>
+        <ProjectReference Include="..\Orders.Api\Orders.Api.csproj"/>
     </ItemGroup>
 
+    <Import Project="..\..\..\..\Tests.Build.props"/>
+
 </Project>
diff --git a/Sample/ECommerce/Orders/Orders.Tests/Orders.Tests.csproj b/Sample/ECommerce/Orders/Orders.Tests/Orders.Tests.csproj
@@ -22,4 +22,6 @@
       <ProjectReference Include="..\Orders\Orders.csproj" />
     </ItemGroup>
 
+    <Import Project="..\..\..\..\Tests.Build.props"/>
+
 </Project>
diff --git a/Sample/ECommerce/Payments/Payments.Api.Tests/Payments.Api.Tests.csproj b/Sample/ECommerce/Payments/Payments.Api.Tests/Payments.Api.Tests.csproj
@@ -5,23 +5,25 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="FluentAssertions" Version="6.8.0" />
-        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="7.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0" />
-        <PackageReference Include="NSubstitute" Version="4.4.0" />
-        <PackageReference Include="xunit" Version="2.4.2" />
-        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0" />
+        <PackageReference Include="FluentAssertions" Version="6.8.0"/>
+        <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0"/>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.0"/>
+        <PackageReference Include="NSubstitute" Version="4.4.0"/>
+        <PackageReference Include="xunit" Version="2.4.2"/>
+        <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="7.0.0"/>
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
-          <PrivateAssets>all</PrivateAssets>
-          <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+            <PrivateAssets>all</PrivateAssets>
+            <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
     </ItemGroup>
 
     <ItemGroup>
-      <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj" />
-      <ProjectReference Include="..\Payments.Api\Payments.Api.csproj" />
+        <ProjectReference Include="..\..\..\..\Core.Testing\Core.Testing.csproj"/>
+        <ProjectReference Include="..\Payments.Api\Payments.Api.csproj"/>
     </ItemGroup>
 
+    <Import Project="..\..\..\..\Tests.Build.props"/>
+
 </Project>