Skip to content

Commit

Permalink
Formatting (#9)
Browse files Browse the repository at this point in the history
* Код отформатирован.
* Исправлен баг с созданием двух процессоров, которые читают из одной очереди.
В итоге тест, в некоторых случаях не проходил, так как два процессора читали ссобщения из одно очереди, при этом тест мог успешно завершитьься, только первый созlанный процессор получал сообщение HardStopCommand, а при работе двух процессоров иногда HardStopCommand ухождило в другой процессор.

* Добавлен параметр, подавляющий параллельное тестирование проектов, чтобы избкжать конфликтов доступа к одним и тем же файлов при подсчете покрытия кода

---------

Co-authored-by: etyumemntcev <[email protected]>
  • Loading branch information
etyumentcev and etyumemntcev authored Sep 5, 2023
1 parent d362af9 commit bb5709d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/dotnet-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:

- name: Test
working-directory: ${{ env.DIR }}
run: dotnet test --configuration Debug --no-build /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:MergeWith=../coverage-${{ matrix.dotnet-version }}/opencover.xml /p:CoverletOutput=../coverage-${{ matrix.dotnet-version }}/opencover.xml
run: dotnet test --configuration Debug --no-build /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:MergeWith=../coverage-${{ matrix.dotnet-version }}/opencover.json /p:CoverletOutput=../coverage-${{ matrix.dotnet-version }}/opencover.xml /maxcpucount:1

- name: Code Coverage
uses: codecov/codecov-action@v3
Expand Down
1 change: 0 additions & 1 deletion appserver/scopes.tests/scopes.tests.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
</ItemGroup>

<ItemGroup>
<ProjectReference Include="..\core\core.csproj" />
<ProjectReference Include="..\scopes\scopes.csproj" />
</ItemGroup>

Expand Down
4 changes: 4 additions & 0 deletions appserver/server.tests/server.tests.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,10 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="coverlet.msbuild" Version="6.0.0">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.5.0" />
<PackageReference Include="Moq" Version="4.20.69" />
<PackageReference Include="xunit" Version="2.4.2" />
Expand Down

0 comments on commit bb5709d

Please sign in to comment.