Skip to content

[Host.Kafka] Fix and improve commit logic #1105

[Host.Kafka] Fix and improve commit logic

[Host.Kafka] Fix and improve commit logic #1105

Re-run triggered September 17, 2024 07:31
Status Failure
Total duration 11m 37s
Artifacts 1

build.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 11 warnings
build
Failed test were found and 'fail-on-error' option is set to ***
build
Process completed with exit code 1.
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3, actions/setup-java@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build: src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L10
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
build: src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L22
Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)
build: src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L64
Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)
build: src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L78
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L65
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaPartitionConsumer.cs#L40
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
build: src/SlimMessageBus.Host.Kafka/KafkaMessageBus.cs#L164
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L131
Either remove or fill this block of code. (https://rules.sonarsource.com/csharp/RSPEC-108)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L162
Await CancelAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
build: src/SlimMessageBus.Host.Kafka/KafkaMessageBus.cs#L117
Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

Artifacts

Produced during runtime
Name Size
test-results
228 KB